Bug 255447
| Summary: | ResolutionMediaFeatureEnabled is a no-op | ||
|---|---|---|---|
| Product: | WebKit | Reporter: | Sam Sneddon [:gsnedders] <gsnedders> |
| Component: | CSS | Assignee: | Tim Nguyen (:ntim) <ntim> |
| Status: | RESOLVED FIXED | ||
| Severity: | Normal | CC: | bfulgham, koivisto, ntim, webkit-bug-importer |
| Priority: | P2 | Keywords: | InRadar |
| Version: | Safari 16 | ||
| Hardware: | Unspecified | ||
| OS: | Unspecified | ||
Sam Sneddon [:gsnedders]
Since bug 248619 removed the (already dead) legacy media queries code, the ResolutionMediaFeatureEnabled setting has had no references within WebKit.
While dropping this setting is probably fine, we shouldn't have the setting hang around while it does nothing.
| Attachments | ||
|---|---|---|
| Add attachment proposed patch, testcase, etc. |
Tim Nguyen (:ntim)
I wonder if we should reinstate this pref, I assume it was added to prevent fingerprinting. Though a better approach to prevent fingerprinting is to spoof this + other relevant vectors to 1x.
Sam Sneddon [:gsnedders]
(In reply to Tim Nguyen (:ntim) from comment #1)
> I wonder if we should reinstate this pref, I assume it was added to prevent
> fingerprinting. Though a better approach to prevent fingerprinting is to
> spoof this + other relevant vectors to 1x.
No, it was added because it provided a path to enabling it in WebKit (versus the ~decade it had been behind a compile-time flag) and because we for a while had quirks that disabled it.
Radar WebKit Bug Importer
<rdar://problem/108363296>
Tim Nguyen (:ntim)
Pull request: https://github.com/WebKit/WebKit/pull/13281
EWS
Committed 263519@main (2d54e17cd623): <https://commits.webkit.org/263519@main>
Reviewed commits have been landed. Closing PR #13281 and removing active labels.