1978211 - Remove layout.css.moz-broken.content.enabled pref

Status: RESOLVED FIXED

(Core :: CSS Parsing and Computation, task)

Description

[Gregory Pappas [:gregp]]

Been unshipped for about 2 years now (bug 1850342)

Comment 1

[Gregory Pappas [:gregp]]

Attachment: Bug 1978211 - Remove layout.css.moz-broken.content.enabled pref r=emilio

Comment 2

[BugBot [:suhaib / :marco]]

There is an r+ patch which didn't land and no activity in this bug for 2 weeks.
:gregp, could you have a look please?
If you still have some work to do, you can add an action "Plan Changes" in Phabricator.
For more information, please visit BugBot documentation.

Comment 3

[Emilio Cobos Álvarez [:emilio]]

Lmk if you need me to land this greg :)

Comment 4

[Gregory Pappas [:gregp]]

I did try to land this but it failed to land for some reason, probably needs to be rebased

Comment 5

[Pulsebot]

Pushed by [e-mail]:
https://github.com/mozilla-firefox/firefox/commit/56106a9006bf
https://hg.mozilla.org/integration/autoland/rev/7c88e6326156
Remove layout.css.moz-broken.content.enabled pref r=emilio

Comment 6

[Pulsebot]

Pushed by [e-mail]:
https://github.com/mozilla-firefox/firefox/commit/9114101848ae
https://hg.mozilla.org/integration/autoland/rev/f28324008675
Revert "Bug 1978211 - Remove layout.css.moz-broken.content.enabled pref r=emilio" for causing reftest failure on moz-broken-matching-1.html

Comment 7

[Narcis Beleuzu [:NarcisB]]

Backed out for causing reftest failure on moz-broken-matching-1.html

Backout link: https://hg-edge.mozilla.org/integration/autoland/rev/f28324008675872a81c811ebf62fbc3459c6e4f3
Log link: https://treeherder.mozilla.org/logviewer?job_id=521318509&repo=autoland&lineNumber=2094

Comment 8

[Emilio Cobos Álvarez [:emilio]]

Fixed the tests.

Comment 9

[Pulsebot]

Pushed by [e-mail]:
https://github.com/mozilla-firefox/firefox/commit/a52021bfebee
https://hg.mozilla.org/integration/autoland/rev/06d7f5fa1d7a
Remove layout.css.moz-broken.content.enabled pref r=emilio

Comment 10

[Pulsebot]

Pushed by [e-mail]:
https://github.com/mozilla-firefox/firefox/commit/b34dff9b92e5
https://hg.mozilla.org/integration/autoland/rev/96cd60ed91e6
Remove two tests.

Comment 11

[Pulsebot]

Pushed by [e-mail]:
https://github.com/mozilla-firefox/firefox/commit/bf01a05dc565
https://hg.mozilla.org/integration/autoland/rev/2b462c02a166
Skip some tests on Android.

Comment 12

[Pulsebot]

Pushed by [e-mail]:
https://github.com/mozilla-firefox/firefox/commit/7555a7425309
https://hg.mozilla.org/integration/autoland/rev/4940c82be630
Revert "Bug 1978211 - Skip some tests on Android."
https://github.com/mozilla-firefox/firefox/commit/ee6cbac0d8ba
https://hg.mozilla.org/integration/autoland/rev/ad575d8ba2a8
Revert "Bug 1978211 - Remove two tests."
https://github.com/mozilla-firefox/firefox/commit/11b9c3e66620
https://hg.mozilla.org/integration/autoland/rev/72c2dd8548bd
Revert "Bug 1978211 - Remove layout.css.moz-broken.content.enabled pref r=emilio" for causing multiple reftest failures

Comment 13

[rperta]

Backed out for causing multiple reftest failures
Backout link
Push with failures
Failure log(s)

Comment 14

[Pulsebot]

Pushed by [e-mail]:
https://github.com/mozilla-firefox/firefox/commit/4113f0170117
https://hg.mozilla.org/integration/autoland/rev/ca862b7f50e6
Remove layout.css.moz-broken.content.enabled pref r=emilio

Comment 15

[Cosmin Sabou [:CosminS]]

https://hg.mozilla.org/mozilla-central/rev/ca862b7f50e6