Skip to content

Conversation

@elliette
Copy link
Member

Addresses #9398

Adds an "escape hatch" that allows a user to force the dart2js-compiled DevTools even when the dart2wasm-compiled DevTools fails to load (see for example #9118). This "escape hatch" is adding the query parameter compiler=js and forcing a reload.

FYI @mdebbar - I'm trying to figure out a good way to test this (not sure if integration_test could be made to work, or if package:webdriver is better, or...?) See #9405. Would love to hear your thoughts if you have any suggestions. Thanks!

@elliette elliette requested a review from a team as a code owner August 21, 2025 20:20
@elliette elliette requested review from bkonyi, mdebbar and srawlins and removed request for a team and bkonyi August 21, 2025 20:20
Copy link
Contributor

@mdebbar mdebbar left a comment

Choose a reason for hiding this comment

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

The change looks good to me.

As far as testing goes, ideally, you would load the app in an iframe with the ?compiler=js in the URL. Then you would poke into the HTML content of the iframe to make sure the attribute on the body is set to flt-renderer="canvaskit". But I'm not sure how to do a test like this in Dart/Flutter.

@elliette elliette merged commit f62ac70 into flutter:master Aug 21, 2025
35 of 36 checks passed
hrajwade96 pushed a commit to hrajwade96/devtools that referenced this pull request Sep 2, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants