Migrate package/flutter to JS static interop.#111315
Migrate package/flutter to JS static interop.#111315joshualitt merged 3 commits intoflutter:masterfrom
Conversation
flutter-dashboard
bot
added
the
framework
|
It looks like this pull request may not have tests. Please make sure to add tests before merging. If you need an exemption to this rule, contact Hixie on the #hackers channel in Chat (don't just cc him here, he won't see it! He's on Discord!). If you are not sure if you need tests, consider this rule of thumb: the purpose of a test is to make sure someone doesn't accidentally revert the fix. Ask yourself, is there anything in your PR that you feel it is important we not accidentally revert back to how it was before your fix? Reviewers: Read the Tree Hygiene page and make sure this patch meets those guidelines before LGTMing. |
joshualitt
force-pushed
the
flutter-changes-for-wasm
branch
3 times, most recently
from
789c222 to
ce2769f
Compare
joshualitt
changed the title
flutter-dashboard
bot
added
the
c: contributor-productivity
joshualitt
force-pushed
the
flutter-changes-for-wasm
branch
3 times, most recently
from
1bc3c0f to
e97cf4d
Compare
joshualitt
requested review from
Piinks and
christopherfujino
as code owners
|
if you rebase upstream to pick up 9ba3581 this diff will be a lot smaller |
joshualitt
force-pushed
the
flutter-changes-for-wasm
branch
from
e97cf4d to
bf8b98a
Compare
|
@christopherfujino Thanks for the heads up! Much better. |
joshualitt
requested review from
ditman and
eyebrowsoffire
and removed request for
Piinks and
christopherfujino
|
@ditman @eyebrowsoffire ptal. Thoughts? We can consolidate the shims into a single file, I just wasn't sure where it should live. |
eyebrowsoffire
left a comment
eyebrowsoffire
left a comment
There was a problem hiding this comment.
Looks good! What's the plan and timing on consolidating a lot of these static JS declarations for browser APIs that are duplicated in the engine/framework/test package? These are eventually going to be directly in the dart SDK right?
|
One update here, I spoke with @eyebrowsoffire offline about the timeline here. There is a lot still up in the air unfortunately. Hopefully we will know more in a few months. |
ditman
left a comment
ditman
left a comment
There was a problem hiding this comment.
Let's do it! A few comments/nitpicks below, I don't think there's anything blocking though :)
joshualitt
force-pushed
the
flutter-changes-for-wasm
branch
from
bf8b98a to
5298d40
Compare
|
test-exempt: code refactor with no semantic change |
joshualitt
force-pushed
the
flutter-changes-for-wasm
branch
from
687b3b1 to
a4940bd
Compare
This reverts commit 6906493.
6 participants
This CL migrates a number of Flutter classes to JS static interop. There is still work to be done to migrate the rest of the framework.