Skip to content

Disable hardware acceleration for Chrome tests#21299

Merged
timvandermeij merged 1 commit into
mozilla:masterfrom
timvandermeij:disable-gpu
May 19, 2026
Merged

Disable hardware acceleration for Chrome tests#21299
timvandermeij merged 1 commit into
mozilla:masterfrom
timvandermeij:disable-gpu

Conversation

@timvandermeij
Copy link
Copy Markdown
Contributor

@timvandermeij timvandermeij commented May 19, 2026

This commit fixes the rendering issue that makes the "must update an existing annotation" ink editor integration test permafail locally in Chrome. Note that we already do this for Firefox tests, so this also improves consistency between the two browsers.

Moreover, improve how we define Chrome options to (similar to their Firefox counterparts) provide them in a single array, and document the reasoning for why these options are being set more explicitly.

Fixes #21272.

@codecov-commenter
Copy link
Copy Markdown

codecov-commenter commented May 19, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 81.28%. Comparing base (f500cff) to head (6059138).

Additional details and impacted files
@@           Coverage Diff           @@
##           master   #21299   +/-   ##
=======================================
  Coverage   81.28%   81.28%           
=======================================
  Files         256      256           
  Lines       64873    64873           
=======================================
+ Hits        52730    52732    +2     
+ Misses      12143    12141    -2     
Flag Coverage Δ
fonttest 8.68% <ø> (ø)
integrationtest 66.78% <ø> (+<0.01%) ⬆️
unittest 57.08% <ø> (-0.02%) ⬇️
unittestcli 56.31% <ø> (+0.01%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

This commit fixes the rendering issue that makes the "must update an
existing annotation" ink editor integration test permafail locally in
Chrome. Note that we already do this for Firefox tests, so this also
improves consistency between the two browsers.

Moreover, improve how we define Chrome options to (similar to their
Firefox counterparts) provide them in a single array, and document the
reasoning for why these options are being set more explicitly.

Fixes mozilla#21272.
Copy link
Copy Markdown
Contributor

@calixteman calixteman left a comment

Choose a reason for hiding this comment

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

LGTM. Thank you.

@timvandermeij
Copy link
Copy Markdown
Contributor Author

/botio-linux browsertest

@moz-tools-bot
Copy link
Copy Markdown
Collaborator

From: Bot.io (Linux m4)


Received

Command cmd_browsertest from @timvandermeij received. Current queue size: 0

Live output at: http://54.241.84.105:8877/c582eb628107ce7/output.txt

@moz-tools-bot
Copy link
Copy Markdown
Collaborator

From: Bot.io (Linux m4)


Success

Full output at http://54.241.84.105:8877/c582eb628107ce7/output.txt

Total script time: 18.42 mins

  • Regression tests: Passed

@timvandermeij timvandermeij merged commit b13ec1f into mozilla:master May 19, 2026
22 checks passed
@timvandermeij timvandermeij deleted the disable-gpu branch May 19, 2026 20:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug]: Fix permafail in the "must update an existing annotation" ink editor integration test

4 participants