Closed Bug 1437267 Opened 4 years ago Closed 4 years ago

Remove the svg.paint-order.enabled pref

Categories

(Core :: SVG, enhancement)

enhancement
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla60
Tracking Status
firefox60 --- fixed

People

(Reporter: jfkthame, Assigned: jfkthame)

References

Details

Attachments

(1 file)

(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.
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).
Attachment #8950016 - Flags: review?(jwatt)
Assignee: nobody → jfkthame
Status: NEW → ASSIGNED
(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? :)
Flags: needinfo?(emilio)
Attachment #8950016 - Flags: review?(jwatt) → review+
(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
Flags: needinfo?(emilio)
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.
Pushed by jkew@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/702b062142c5
Remove the svg.paint-order.enabled pref. r=jwatt
https://hg.mozilla.org/mozilla-central/rev/702b062142c5
Status: ASSIGNED → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla60
You need to log in before you can comment on or make changes to this bug.