Closed Bug 1337656 Opened 8 years ago Closed 8 years ago

stylo: allow layout.css.servo.enabled to be read early on in content process life

Categories

(Core :: CSS Parsing and Computation, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla54
Tracking Status
firefox54 --- fixed

People

(Reporter: heycam, Assigned: heycam)

References

Details

Attachments

(1 file)

Bug 1303096 added a white list of prefs that can be read on content process startup. Since all the other prefs read by nsLayoutUtils::Initialize are in it, I think we can just add layout.css.servo.enabled too.
Attached patch patchSplinter Review
Assignee: nobody → cam
Status: NEW → ASSIGNED
Attachment #8834755 - Flags: review?(bobbyholley)
Attachment #8834755 - Attachment is patch: true
Comment on attachment 8834755 [details] [diff] [review] patch Review of attachment 8834755 [details] [diff] [review]: ----------------------------------------------------------------- Let's get this in, but flagging blassey for retroactive f? just to double-check that this is the right thing.
Attachment #8834755 - Flags: review?(bobbyholley)
Attachment #8834755 - Flags: review+
Attachment #8834755 - Flags: feedback?(blassey.bugs)
Pushed by cmccormack@mozilla.com: https://hg.mozilla.org/integration/mozilla-inbound/rev/de301f4f1519 Allow layout.css.servo.enabled to be read at content process startup. r=bholley
Status: ASSIGNED → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla54
Attachment #8834755 - Flags: feedback?(blassey.bugs) → feedback+
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: