Closed
Bug 1167589
Opened 9 years ago
Closed 8 years ago
Mark the members of CSSParsingEnvironment MOZ_UNSAFE_REF
Categories
(Core :: CSS Parsing and Computation, defect)
Core
CSS Parsing and Computation
Tracking
()
RESOLVED
FIXED
mozilla41
Tracking | Status | |
---|---|---|
firefox41 | --- | fixed |
People
(Reporter: nika, Assigned: nika)
Details
Attachments
(1 file, 2 obsolete files)
Assignee | ||
Comment 1•9 years ago
|
||
Updated•9 years ago
|
Component: Layout → CSS Parsing and Computation
Comment 2•9 years ago
|
||
Why make this change?
![]() |
||
Comment 3•9 years ago
|
||
These are weak pointers for a very good reason, iirc: this is used in very performance sensitive code and the refcounting was showing up quite noticeably in profiles.
Assignee | ||
Comment 4•9 years ago
|
||
I've changed the patch to instead mark the members as MOZ_UNSAFE_REF, and reference the original bug which made the change (Bug 649163)
Attachment #8609363 -
Attachment is obsolete: true
Assignee | ||
Updated•9 years ago
|
Attachment #8610122 -
Flags: review?(dbaron)
Comment 5•8 years ago
|
||
Comment on attachment 8610122 [details] [diff] [review] Mark the members of CSSParsingEnvironment as MOZ_UNSAFE_REF Now that MOZ_NON_OWNING_REF and MOZ_UNSAFE_REF are basically the same thing underneath, I'm less confused. (Although the commit message of 229b03af6f2b was a little confusing in saying it was changing only documentation.) It's probably worth mentioning in the comment that the user of CSSParsingEnvironment is responsible for maintaining an owning reference for the lifetime of the CSSParsingEnvironment. The comment could probably otherwise be simplified, perhaps to: MOZ_UNSAFE_REF("user of CSSParsingEnviroment must hold an owning " "reference; reference counting here has unacceptable " "performance overhead (see bug 649163)") Also, please don't include the URL for the bug in the commit message; just include the bug number. r=dbaron with that or similar But please change the summary of the bug as well to match what you're doing!
Attachment #8610122 -
Flags: review?(dbaron) → review+
Assignee | ||
Comment 6•8 years ago
|
||
Attachment #8610122 -
Attachment is obsolete: true
Assignee | ||
Updated•8 years ago
|
Assignee: nobody → michael
Keywords: checkin-needed
Summary: Made the members of CSSParsingEnvironment nsCOMPtrs → Mark the members of CSSParsingEnvironment MOZ_UNSAFE_REF
Comment 8•8 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/a1558d74b4db
Status: NEW → RESOLVED
Closed: 8 years ago
status-firefox41:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla41
You need to log in
before you can comment on or make changes to this bug.
Description
•