1784014 - Remove MathML preference mfrac_bevelled_attribute

Status: RESOLVED FIXED

(Core :: MathML, task)

Comment 1

[Frédéric Wang Nélar]

Attachment: Bug 1784014 - Remove MathML preference mfrac_bevelled_attribute. r=emilio

• remove flag and corresponding warning/counter.
• remove attribute from parsers, but keep the atom since it's
  used by TreeSanitizer.
• remove tests for mfrac@bevelled, there is a WPT test to check it's
  not supported.
• layout/mathml/tests/test_bug975681.html is removed, its tests are
  currently (wrongly) all disabled when the flag is off and equivalent
  tests for attributes other than bevelled exist in WPT.

Comment 2

[BugBot [:suhaib / :marco]]

There's a r+ patch which didn't land and no activity in this bug for 2 weeks.
:fredw, 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 auto_nag documentation.

Comment 3

[Frédéric Wang Nélar]

This is blocked on https://phabricator.services.mozilla.com/D154198

Comment 4

[Emilio Cobos Álvarez [:emilio]]

You should get your L3 access back Fred :)

Though please ni? me on any patches that need landing or what not, or stack them by marking as dependent and I can push them all together?

Comment 5

[Frédéric Wang Nélar]

(In reply to Emilio Cobos Álvarez (:emilio) from comment #4)

You should get your L3 access back Fred :)

I think I have it enabled again, but don't remember how to land...

Though please ni? me on any patches that need landing or what not, or stack them by marking as dependent and I can push them all together?

OK, how can I mark patches dependent on phabricator?

Comment 6

[Emilio Cobos Álvarez [:emilio]]

(In reply to Frédéric Wang (:fredw) from comment #5)

(In reply to Emilio Cobos Álvarez (:emilio) from comment #4)

You should get your L3 access back Fred :)

I think I have it enabled again, but don't remember how to land...

You can click "View stack in Lando" from phabricator, and there's a big "Land" e.g., https://lando.services.mozilla.com/D154199/ for this bug.

OK, how can I mark patches dependent on phabricator?

You can either moz-phab reorg from the command line, or do it manually with the "Edit related revisions..." menu in phabricator (in the sidebar to the right).

Comment 7

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/8386bdd056dd
Remove MathML preference mfrac_bevelled_attribute. r=emilio

Comment 8

[Marian-Vasile Laza]

https://hg.mozilla.org/mozilla-central/rev/8386bdd056dd

Comment 9

[Marian-Vasile Laza]

https://hg.mozilla.org/mozilla-central/rev/8386bdd056dd