Closed Bug 1608889 Opened 5 years ago Closed 5 years ago

Flip backplate preference to "true" from @IS_EARLY_BETA_OR_EARLIER@ (ship backplating in Firefox 73)

Categories

(Core :: Layout, task)

task
Not set
normal

Tracking

()

VERIFIED FIXED
mozilla74
Tracking Status
firefox73 + verified
firefox74 --- verified

People

(Reporter: morgan, Assigned: morgan)

References

Details

Attachments

(1 file)

Backplate should be live and released starting in FF73, so we'll need to change the preference accordingly and uplift.

Assignee: nobody → mreschenberg

Hooking this up to where this got implemented. Also moving to layout, as this isn't a bug relating to the implementation of the preference system. :-)

Blocks: 1539212
Component: Preferences: Backend → Layout
Summary: Flip backplate preference to "true" from @IS_EARLY_BETA_OR_EARLIER@ → Flip backplate preference to "true" from @IS_EARLY_BETA_OR_EARLIER@ (ship backplating in Firefox 73)
Status: NEW → ASSIGNED
Pushed by mreschenberg@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/712292576d96 Flip backplate preference to true to ship in 73. r=mattwoodrow

Comment on attachment 9120543 [details]
Bug 1608889: Flip backplate preference to true to ship in 73.

Beta/Release Uplift Approval Request

  • User impact if declined: Users will not experience readability backplate by default when using high-contrast mode. Users will continue to see no background images render, and may have a sub-optimal browsing experience compared to using HCM in Edge.
  • Is this code covered by automated tests?: Yes
  • Has the fix been verified in Nightly?: No
  • Needs manual test from QE?: Yes
  • If yes, steps to reproduce: 1. Navigate to about:config and verify 'permitBackplate' displays as "true" by default
  1. Navigate to cnn.com and verify images are displayed, and text overlapping images is "backplated" with a white background.
  • List of other uplifts needed: None
  • Risk to taking this patch: Medium
  • Why is the change risky/not risky? (and alternatives if risky): Though backplate has been tested pretty thoroughly by the accessibility team (and by my written tests), there is still a possibility there are problems we have not uncovered. This is a large change to how users experience HCM, and though we believe it will lead to a better overall browsing experience, we wouldn't say it comes without risk. To mitigate the possible risks, though, we've elected to keep the preference (permitBackplate) and change its default value to "true" rather than remove the pref entirely. If large issues appear that make backplate unusable, we can uplift a patch to change the default of this pref back to "false".
  • String changes made/needed: N/A
Attachment #9120543 - Flags: approval-mozilla-beta?
Flags: qe-verify+
Flags: qe-verify+ → qe-verify?

Comment on attachment 9120543 [details]
Bug 1608889: Flip backplate preference to true to ship in 73.

Enables the HCM backplate by default for Fx73. Approved for 73.0b5 \m/

Attachment #9120543 - Flags: approval-mozilla-beta? → approval-mozilla-beta+
Status: ASSIGNED → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla74
Flags: qe-verify? → qe-verify+
QA Whiteboard: [qa-triaged]

I have verified that the pref browser.display.permit_backplate is correctly set to true in Beta 73.0b5 and Nightly 74.0a1, and the background images are properly visualized.

This was tested on Windows 10 x64, macOS 10.13 and Ubuntu 18.04 x64. Note that on OS X, I didn't see any differences (pref set on true vs false) with HCM applied, the text and images were correctly displayed in both cases.

Status: RESOLVED → VERIFIED
Flags: qe-verify+
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: