Closed Bug 1332353 Opened 3 years ago Closed 3 years ago

Introduce more clarity into whether a stylesheet is owned by a document

Categories

(Core :: CSS Parsing and Computation, defect)

defect
Not set

Tracking

()

RESOLVED FIXED
mozilla53
Tracking Status
firefox53 --- fixed

People

(Reporter: bzbarsky, Assigned: bzbarsky)

References

Details

Attachments

(1 file)

Right now we have this SetOwningDocument thing, but for editor override sheets the document doesn't actually own the sheet; the editor does.  This makes it hard to reason about sheet lifetime optimizations in the cycle collector.

I'm going to rejigger the API to make it clear what's really going on.
Note that this patch will conflict with bug 1328420, so whoever lands second will have to do some rebasing.
Thanks for the heads-up.
Attachment #8828453 - Flags: review?(cam) → review+
Pushed by bzbarsky@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/2b1fcb4b8c88
Make it clearer when a stylesheet is really owned by its mDocument.  r=heycam
https://hg.mozilla.org/mozilla-central/rev/2b1fcb4b8c88
Status: NEW → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla53
Depends on: 1358056
You need to log in before you can comment on or make changes to this bug.