Closed Bug 652385 Opened 10 years ago Closed 10 years ago

nsLayoutStyleSheetcache has odd destructor

Categories

(Core :: CSS Parsing and Computation, defect, P5)

defect

Tracking

()

RESOLVED FIXED
mozilla6

People

(Reporter: dbaron, Assigned: dbaron)

Details

Attachments

(1 file)

Attached patch patchSplinter Review
nsLayoutStyleSheetCache has a destructor that duplicates the work that its static shutdown method does, but incorrectly (since it doesn't release the pointers).  The destructor should just be removed, since the shutdown method nulls out both pointers with its NS_RELEASE.
Attachment #527977 - Flags: review?(bzbarsky)
Comment on attachment 527977 [details] [diff] [review]
patch

r=me
Attachment #527977 - Flags: review?(bzbarsky) → review+
https://hg.mozilla.org/mozilla-central/rev/d656c5e38d83
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Priority: P4 → P5
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.