Remove test annotations that pref on CSS containment (since it's already default-true)
Categories
(Core :: CSS Parsing and Computation, defect, P3)
Tracking
()
| Tracking | Status | |
|---|---|---|
| firefox77 | --- | fixed |
People
(Reporter: dholbert, Assigned: dholbert)
References
Details
Attachments
(1 file)
|
(deleted),
text/x-phabricator-request
|
Details |
I noticed today (when running WPT) that we've got a number of places where we manually pref on css containment for testing purposes. This isn't necessary at this point, since it's default-enabled & that's not going to change since we've been shipping it for multiple releases.
We can't remove the pref entirely yet, because there's a test that depends on preffing it off (temporarily) to test something else (see bug 1626458 comment 1). But we might as well remove all of the boilerplate that sets it to true for specific tests here and there.
|
Assignee | |
Comment 1•5 years ago
|
||
i.e. remove annotations that manually enable the property, and remove the
property_database.js check that tests for the property being enabled.
We've been shipping with the property enabled ever since Firefox 69, so we're
realistically not going to default-disable it at this point, which means these
annotations (and the property_database.js check) are just cruft.
|
|
Assignee | |
Comment 2•5 years ago
|
||
(For reference: this about:config pref has been default-enabled since bug 1487493, which made it to release as of Firefox 69, via a beta uplift at the time.)
|
||
Comment 4•5 years ago
|
||
| bugherder | ||
Description
•