Skip to content

Conversation

@eyebrowsoffire
Copy link
Contributor

@eyebrowsoffire eyebrowsoffire commented Apr 3, 2024

This adds support for adding the --wasm flag to flutter run and flutter drive

  • Emits errors if you attempt to use the skwasm renderer without the --wasm flag
  • Emits errors if you try to use --wasm when not using a web device
  • Uses the skwasm renderer by default if you pass --wasm and no --web-renderer

@github-actions github-actions bot added tool Affects the "flutter" command-line tool. See also t: labels. a: tests "flutter test", flutter_test, or one of our tests framework flutter/packages/flutter repository. See also f: labels. f: integration_test The flutter/packages/integration_test plugin labels Apr 3, 2024
@github-actions github-actions bot added the d: examples Sample code and demos label Apr 8, 2024
@eyebrowsoffire eyebrowsoffire marked this pull request as ready for review April 11, 2024 16:18
@eyebrowsoffire eyebrowsoffire requested a review from ditman April 11, 2024 16:20
Copy link
Member

@ditman ditman left a comment

Choose a reason for hiding this comment

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

Ship it! I only have minor nits/questions that don't warrant blocking this.

Comment on lines 333 to 343
Copy link
Member

Choose a reason for hiding this comment

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

This method is duplicated between here and commands/run.dart, maybe the implementation could be shared to a WebRendererMode getRendererMode(String? userString, bool useWasm = false);? It could even be a static in the WebRendererMode enum itself?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yeah that's a good idea, I'll do that.

Copy link
Member

Choose a reason for hiding this comment

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

Sneaky!

engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Apr 13, 2024
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Apr 14, 2024
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Apr 14, 2024
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Apr 14, 2024
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Apr 15, 2024
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Apr 15, 2024
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Apr 15, 2024
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Apr 15, 2024
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Apr 15, 2024
auto-submit bot pushed a commit to flutter/packages that referenced this pull request Apr 15, 2024
Manual roll requested by [email protected]

flutter/flutter@53cba24...2e748e8

2024-04-15 [email protected] Implementing control flow collections (flutter/flutter#146601)
2024-04-15 [email protected] Roll Packages from 78f684c to 6698b2d (3 revisions) (flutter/flutter#146761)
2024-04-15 [email protected] test: Fix memory leak in transitions test (flutter/flutter#146747)
2024-04-15 [email protected] Fix filled text field active indicator overflows container bounds (flutter/flutter#146637)
2024-04-14 [email protected] - Fixes _DropdownMenuState leaking text controller (flutter/flutter#146571)
2024-04-13 [email protected] Fix memory leaks in `FloatingActionButton` (flutter/flutter#146711)
2024-04-12 [email protected] Roll Flutter Engine from 0e56e3dffe43 to 1a13c7d1f40e (2 revisions) (flutter/flutter#146703)
2024-04-12 [email protected] Roll pub packages (flutter/flutter#146704)
2024-04-12 49699333+dependabot[bot]@users.noreply.github.com Bump peter-evans/create-pull-request from 6.0.2 to 6.0.3 (flutter/flutter#146702)
2024-04-12 [email protected] Avoid forwarding the data after socket is disconnected. (flutter/flutter#146665)
2024-04-12 [email protected] [flutter_tools] Fix conductor for package args roll (flutter/flutter#146646)
2024-04-12 [email protected] Roll Flutter Engine from 6b37b170998e to 0e56e3dffe43 (4 revisions) (flutter/flutter#146698)
2024-04-12 [email protected] Light sliver clean up before SliverTree (flutter/flutter#146696)
2024-04-12 [email protected] Fix label text color is wrong for a focused and hovered TextField (flutter/flutter#146572)
2024-04-12 [email protected] Fix `getOffsetForCaret` crash (flutter/flutter#146669)
2024-04-12 [email protected] Update gen_keycodes templates (flutter/flutter#146481)
2024-04-12 [email protected] Support `flutter run --wasm` and `flutter drive --wasm`. (flutter/flutter#146231)
2024-04-12 [email protected] Fix curved animation memory leak for scrollbar (flutter/flutter#146670)
2024-04-12 [email protected] Roll Packages from e98839a to 78f684c (6 revisions) (flutter/flutter#146691)

If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-packages
Please CC [email protected],[email protected],[email protected] on the revert to ensure that a human
is aware of the problem.

To file a bug in Packages: https://github.com/flutter/flutter/issues/new/choose

To report a problem with the AutoRoller itself, please file a bug:
https://issues.skia.org/issues/new?component=1389291&template=1850622

Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md
gilnobrega pushed a commit to gilnobrega/flutter that referenced this pull request Apr 22, 2024
)

This adds support for adding the `--wasm` flag to `flutter run` and `flutter drive`
* Emits errors if you attempt to use the skwasm renderer without the `--wasm` flag
* Emits errors if you try to use `--wasm` when not using a web device
* Uses the skwasm renderer by default if you pass `--wasm` and no `--web-renderer`
TecHaxter pushed a commit to TecHaxter/flutter_packages that referenced this pull request May 22, 2024
)

Manual roll requested by [email protected]

flutter/flutter@53cba24...2e748e8

2024-04-15 [email protected] Implementing control flow collections (flutter/flutter#146601)
2024-04-15 [email protected] Roll Packages from 78f684c to 6698b2d (3 revisions) (flutter/flutter#146761)
2024-04-15 [email protected] test: Fix memory leak in transitions test (flutter/flutter#146747)
2024-04-15 [email protected] Fix filled text field active indicator overflows container bounds (flutter/flutter#146637)
2024-04-14 [email protected] - Fixes _DropdownMenuState leaking text controller (flutter/flutter#146571)
2024-04-13 [email protected] Fix memory leaks in `FloatingActionButton` (flutter/flutter#146711)
2024-04-12 [email protected] Roll Flutter Engine from 0e56e3dffe43 to 1a13c7d1f40e (2 revisions) (flutter/flutter#146703)
2024-04-12 [email protected] Roll pub packages (flutter/flutter#146704)
2024-04-12 49699333+dependabot[bot]@users.noreply.github.com Bump peter-evans/create-pull-request from 6.0.2 to 6.0.3 (flutter/flutter#146702)
2024-04-12 [email protected] Avoid forwarding the data after socket is disconnected. (flutter/flutter#146665)
2024-04-12 [email protected] [flutter_tools] Fix conductor for package args roll (flutter/flutter#146646)
2024-04-12 [email protected] Roll Flutter Engine from 6b37b170998e to 0e56e3dffe43 (4 revisions) (flutter/flutter#146698)
2024-04-12 [email protected] Light sliver clean up before SliverTree (flutter/flutter#146696)
2024-04-12 [email protected] Fix label text color is wrong for a focused and hovered TextField (flutter/flutter#146572)
2024-04-12 [email protected] Fix `getOffsetForCaret` crash (flutter/flutter#146669)
2024-04-12 [email protected] Update gen_keycodes templates (flutter/flutter#146481)
2024-04-12 [email protected] Support `flutter run --wasm` and `flutter drive --wasm`. (flutter/flutter#146231)
2024-04-12 [email protected] Fix curved animation memory leak for scrollbar (flutter/flutter#146670)
2024-04-12 [email protected] Roll Packages from e98839a to 78f684c (6 revisions) (flutter/flutter#146691)

If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-packages
Please CC [email protected],[email protected],[email protected] on the revert to ensure that a human
is aware of the problem.

To file a bug in Packages: https://github.com/flutter/flutter/issues/new/choose

To report a problem with the AutoRoller itself, please file a bug:
https://issues.skia.org/issues/new?component=1389291&template=1850622

Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md
JSUYA added a commit to JSUYA/flutter-tizen that referenced this pull request Aug 28, 2024
This change was made in this PR(flutter/flutter#146231),
we only remove the parameter since we don't handle wasm.
@eyebrowsoffire eyebrowsoffire deleted the wasm_drive_run branch December 12, 2025 21:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

a: tests "flutter test", flutter_test, or one of our tests autosubmit Merge PR when tree becomes green via auto submit App d: examples Sample code and demos f: integration_test The flutter/packages/integration_test plugin framework flutter/packages/flutter repository. See also f: labels. tool Affects the "flutter" command-line tool. See also t: labels.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants