Closed Bug 1834133 Opened 1 year ago Closed 1 year ago

Remove layout.css.notify-of-unvisited and layout.css.always-repaint-on-unvisited prefs

Categories

(Core :: CSS Parsing and Computation, task)

task

Tracking

()

RESOLVED FIXED
115 Branch
Tracking Status
firefox115 --- fixed

People

(Reporter: dholbert, Assigned: dholbert)

References

(Blocks 1 open bug)

Details

Attachments

(2 files)

Bug 1632765 turned on two prefs by default, to activate some privacy mitigations:
layout.css.notify-of-unvisited
layout.css.always-repaint-on-unvisited

Probably we should remove these prefs at this point? They've been shipping default-enabled for years now, and I suspect they're not really useful for users to disable. (Disabling them has no user-facing change other than possibly opening themselves up to timing attacks.)

Update all of their usages to assume that they're true (which they have been, by-default, for several years).

If I'm reading the code comment correctly, this is now unreachable code,
and hence can & should be removed.

Depends on D178569

Pushed by dholbert@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/2167b80eb160
part 1: Remove about:config prefs layout.css.notify-of-unvisited and layout.css.always-repaint-on-unvisited. r=emilio
https://hg.mozilla.org/integration/autoland/rev/3ea2cc7b5d50
part 2: Remove code that's flagged as to-be-removed once layout.css.always-repaint-on-unvisited is true. r=emilio
Status: ASSIGNED → RESOLVED
Closed: 1 year ago
Resolution: --- → FIXED
Target Milestone: --- → 115 Branch
Blocks: old-prefs
Summary: Remove layout.css.notify-of-unvisited and layout.css.always-repaint-on-unvisited → Remove layout.css.notify-of-unvisited and layout.css.always-repaint-on-unvisited prefs
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: