Skip to content

[web] Roll Chrome to 145 (framework)#182861

Merged
auto-submit[bot] merged 69 commits into
flutter:masterfrom
mdebbar:chrome_145_framework
Jul 9, 2026
Merged

[web] Roll Chrome to 145 (framework)#182861
auto-submit[bot] merged 69 commits into
flutter:masterfrom
mdebbar:chrome_145_framework

Conversation

@mdebbar

@mdebbar mdebbar commented Feb 24, 2026

Copy link
Copy Markdown
Contributor

No description provided.

@github-actions github-actions Bot added engine flutter/engine related. See also e: labels. platform-web Web applications specifically labels Feb 24, 2026
@mdebbar
mdebbar force-pushed the chrome_145_framework branch from 44c1e8c to 1a0f0d2 Compare February 26, 2026 18:50
@github-actions github-actions Bot added the tool Affects the "flutter" command-line tool. See also t: labels. label Feb 26, 2026
@Piinks

Piinks commented Apr 27, 2026

Copy link
Copy Markdown
Contributor

Greetings from stale PR triage! 👋
Is this change still on your radar?

@mdebbar
mdebbar force-pushed the chrome_145_framework branch from adf6309 to 6b20390 Compare May 18, 2026 14:58
@flutter-dashboard flutter-dashboard Bot added the CICD Run CI/CD label May 18, 2026
@github-actions github-actions Bot removed the CICD Run CI/CD label May 19, 2026
@mdebbar mdebbar added the CICD Run CI/CD label May 19, 2026
@mdebbar mdebbar added CICD Run CI/CD and removed CICD Run CI/CD labels May 28, 2026
@kevmoo
kevmoo marked this pull request as ready for review May 28, 2026 18:23

@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 and Chromedriver dependencies to version 145.0.7632.117 across CI configurations and introduces the --password-store=basic and --use-mock-keychain flags to prevent credential store prompts during testing. Additionally, on ARM macOS, the logic forcing native ARM launch is removed in favor of appending the --use-angle=metal flag to bypass SwiftShader limitations. Review feedback suggests restricting this new flag to headless mode only, as the software rendering issue with SwiftShader specifically impacts headless execution.

Comment thread packages/flutter_tools/lib/src/web/chrome.dart Outdated
@github-actions github-actions Bot removed the CICD Run CI/CD label May 28, 2026
@mdebbar mdebbar added the CICD Run CI/CD label May 29, 2026
kevmoo added 2 commits June 1, 2026 13:09
…orkaround to headless mode

- Restrict --use-mock-keychain to macOS platforms.

- Restrict the --use-angle=metal SwiftShader workaround on ARM macOS to headless executions.

- Update Chrome launcher tests to validate headed, headless, and cross-platform behaviors.
@kevmoo

kevmoo commented Jun 1, 2026

Copy link
Copy Markdown
Contributor

@mdebbar – did some more tweaks on top of your tweaks and pushed. PTAL!

Comment thread packages/flutter_tools/lib/src/web/chrome.dart
kevmoo added 18 commits June 29, 2026 19:34
…vent CanvasKit hangs

Under Chrome 145+, passing --disable-dev-shm-usage unconditionally forces Chrome to write all shared memory IPC buffers to /tmp. While this is required for Wasm/SkWasm multi-threaded tests to avoid crashing on the 64MB /dev/shm limit in containerized environments, it degrades disk IPC performance and causes standard CanvasKit tests to hang/timeout on CI.

Restores the previous behavior where --disable-dev-shm-usage is passed only when useWasm is true.

CONV=b0179f43-b48a-45ed-8b0f-df9a00890861
TAG=agy
# Conflicts:
#	.ci.yaml
#	packages/flutter_tools/test/commands.shard/hermetic/build_macos_test.dart
Increase reloadRestartTimeout and log check timeouts to 30s to reduce flakiness on CI.

Format modified files to resolve linux_unopt failures.
This ignores the lint for the debug print we added to capture browser logs. Both should be reverted before merging if CI goes green.
Resolve template change detection failure by formatting the template to match the formatted implementation file.
Clean up the PR by reverting whitespace-only, formatting-only, and other unrelated changes in 17 files to match origin/master.
Comment thread packages/flutter_test/lib/src/_test_selector_web.dart Outdated
Comment thread packages/flutter_tools/lib/src/test/flutter_web_platform.dart Outdated
Comment thread packages/flutter_tools/lib/src/test/flutter_web_platform.dart
Comment thread packages/flutter_tools/lib/src/test/flutter_web_platform.dart Outdated
Comment thread packages/flutter_tools/lib/src/test/flutter_web_platform.dart Outdated
Comment thread packages/flutter_tools/lib/src/web/chrome.dart
Comment thread packages/flutter_tools/lib/src/web/chrome.dart Outdated
Comment thread packages/flutter_tools/lib/src/web/chrome.dart Outdated
Comment thread packages/flutter_tools/test/web.shard/test_data/web_server_test_common.dart Outdated

@harryterkelsen harryterkelsen 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.

Thanks, Kevin! A couple more small things I missed on my first pass

Comment thread packages/flutter_tools/lib/src/test/flutter_web_platform.dart Outdated
Comment thread packages/flutter_tools/lib/src/test/flutter_web_platform.dart Outdated

@harryterkelsen harryterkelsen 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

@auto-submit

auto-submit Bot commented Jul 9, 2026

Copy link
Copy Markdown
Contributor

autosubmit label was removed for flutter/flutter/182861, because - The status or check suite Google testing has failed. Please fix the issues identified (or deflake) before re-applying this label.

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

Labels

CICD Run CI/CD engine flutter/engine related. See also e: labels. platform-web Web applications specifically team-web Owned by Web platform team tool Affects the "flutter" command-line tool. See also t: labels. will affect goldens Changes to golden files

Projects

Status: ✅ Done

Development

Successfully merging this pull request may close these issues.

8 participants