Skip to content
This repository was archived by the owner on Feb 25, 2025. It is now read-only.

Conversation

@yjbanov
Copy link
Contributor

@yjbanov yjbanov commented Jun 3, 2020

Description

  • Add temporary null safety annotations to window.dart.
  • Move webOnlyScheduleFrameCallback from dart:ui to dart:_engine.
  • Sync _deprecatedLanguageSubtagMap and _deprecatedRegionSubtagMap from mobile into the web version.

Related Issues

flutter/flutter#53661

@yjbanov yjbanov requested a review from zanderso June 3, 2020 17:41
@auto-assign auto-assign bot requested a review from flar June 3, 2020 17:41
Copy link
Member

@zanderso zanderso left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm w/ question.

case 'yuu':
return 'yug'; // Yugh; deprecated 2014-02-28
default:
return languageCode;
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The behavior here looks like it might be different. Is it intentional? Does it matter?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What's the difference in the behavior? If I'm reading the code correctly, both versions will attempt to map a deprecated value to the new value, and default to _languageCode otherwise.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ah, you're right. I missed the ?? _languageCode on the getter.

case 'ZR':
return 'CD'; // Zaire; deprecated 1997-07-14
default:
return regionCode;
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ditto

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants