Skip to content

web_compiler_test: Enable SwiftShader on headless VMs - #10056

Merged
srawlins merged 1 commit into
flutter:masterfrom
srawlins:web-compiler-tests
Oct 26, 2026
Merged

srawlins merged 1 commit into
flutter:masterfrom
srawlins:web-compiler-tests

Conversation

@srawlins

Copy link
Copy Markdown
Contributor

Fixes #10032

  • Added 'binary': ?Platform.environment['CHROME_EXECUTABLE'] to Capabilities.chromeOptions when CHROME_EXECUTABLE is set in the environment.
  • Added the SwiftShader flags (--use-gl=angle, --use-angle=swiftshader, --enable-unsafe-swiftshader, --disable-gpu-sandbox) on Linux and Windows so headless Chrome on GPU-less CI VMs provides WebGL support required by
    FlutterLoader to select the skwasm renderer.

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Code Review

This pull request updates the Chrome capabilities in web_compiler_test.dart to support WebGL for the skwasm renderer on headless Linux and Windows VMs by enabling SwiftShader. It also attempts to conditionally set the Chrome binary path. However, a critical issue was identified where the syntax used for the conditional binary path is invalid in Dart and will cause a compilation error. A suggestion was provided to use a collection if statement instead.

Comment thread packages/devtools_app/webdriver_test/web_compiler_test.dart

@johnpryan johnpryan left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

@johnpryan johnpryan left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

@srawlins

Copy link
Copy Markdown
Contributor Author

Submitting past DCM findings, which are unrelated.

@srawlins
srawlins merged commit 9ac04e2 into flutter:master Oct 26, 2026
92 of 95 checks passed
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.

web_compiler_test fails on LUCI in Dart SDK

2 participants