1437267 - Remove the svg.paint-order.enabled pref

Status: RESOLVED FIXED

(Core :: SVG, enhancement)

Description

[Jonathan Kew [:jfkthame]]

(From bug 1435684 comment 2)

This has been on by default ever since it shipped in bug 828805. I think we can consider it well-baked and eliminate the pref at this point.

Comment 1

[Jonathan Kew [:jfkthame]]

Attachment: Remove the svg.paint-order.enabled pref

This also needs to be removed from the servo code; I opened https://github.com/servo/servo/pull/20016 for that (though I don't really know if I'm following the right procedure on the servo side, that's new to me).

Comment 2

[Jonathan Watt [:jwatt]]

(In reply to Jonathan Kew (:jfkthame) from comment #1)
> This also needs to be removed from the servo code; I opened
> https://github.com/servo/servo/pull/20016 for that (though I don't really
> know if I'm following the right procedure on the servo side, that's new to
> me).

Maybe Emilio can tell us? :)

Comment 3

[Emilio Cobos Álvarez (:emilio)]

(In reply to Jonathan Watt [:jwatt] (needinfo? me) from comment #2)
> (In reply to Jonathan Kew (:jfkthame) from comment #1)
> > This also needs to be removed from the servo code; I opened
> > https://github.com/servo/servo/pull/20016 for that (though I don't really
> > know if I'm following the right procedure on the servo side, that's new to
> > me).
> 
> Maybe Emilio can tell us? :)

It is. Indeed that patch landed on autoland in:

  https://hg.mozilla.org/integration/autoland/rev/7369f0c49008b83d522c75943e5c265c8a609f9d

Comment 4

[Jonathan Kew [:jfkthame]]

Yes, bholley kindly shepherded that through. I'll push this patch to inbound once that has been merged around everywhere, so we don't risk running stylo-based tests that still expect the pref to exist.

Comment 5

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/mozilla-inbound/rev/702b062142c5
Remove the svg.paint-order.enabled pref. r=jwatt

Comment 6

[Andreea Pavel [:apavel]]

https://hg.mozilla.org/mozilla-central/rev/702b062142c5