nsStyleContext::Release() does not need to set mRefCnt to 1 on destruction

RESOLVED FIXED

Status

()

Core
CSS Parsing and Computation
RESOLVED FIXED
15 years ago
15 years ago

People

(Reporter: Brian Ryner (not reading), Assigned: Brian Ryner (not reading))

Tracking

Trunk
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [whitebox])

Attachments

(1 attachment)

(Assignee)

Description

15 years ago
This was copied from nsISupportsImpl's Release() implementation, but after
reading the comment there about why, it sounds like it's only relevant for
nsISupports-derived classes.
(Assignee)

Comment 1

15 years ago
Created attachment 116903 [details] [diff] [review]
patch

The comment I mentioned about the refcount stabilization is at:

http://lxr.mozilla.org/seamonkey/source/xpcom/glue/nsISupportsImpl.h#208
(Assignee)

Updated

15 years ago
Attachment #116903 - Flags: superreview?(dbaron)
Attachment #116903 - Flags: review?(dbaron)
Attachment #116903 - Flags: superreview?(dbaron)
Attachment #116903 - Flags: superreview+
Attachment #116903 - Flags: review?(dbaron)
Attachment #116903 - Flags: review+
(Assignee)

Comment 2

15 years ago
checked in.
Status: NEW → RESOLVED
Last Resolved: 15 years ago
Resolution: --- → FIXED

Comment 3

15 years ago
verification can be done at code level
Whiteboard: [whitebox]
You need to log in before you can comment on or make changes to this bug.