Closed
Bug 1139272
Opened 10 years ago
Closed 10 years ago
put expensive nsStyleContext destructor assertions behind a pref
Categories
(Core :: CSS Parsing and Computation, defect)
Core
CSS Parsing and Computation
Tracking
()
RESOLVED
FIXED
mozilla39
| Tracking | Status | |
|---|---|---|
| firefox39 | --- | fixed |
People
(Reporter: heycam, Assigned: heycam)
References
Details
Attachments
(1 file)
The expensive assertions in the nsStyleContext destructor that verifies that destroyed style structs aren't still referenced in the tree somewhere are currently |#if 0| due to how long they can take on large pages. We have added some crashtests for bugs in the bug 931668 optimizations that are caught be these assertions. So that we can check that we don't re-introduce problems that cause those crashtests to fail, let's put the assertions behind a pref (though still DEBUG-only) that we can selectively enable for those tests in the crashtest manifest.
|
Assignee | |
Comment 1•10 years ago
|
||
Attachment #8572441 -
Flags: review?(dbaron) → review+
|
|
Assignee | |
Comment 2•10 years ago
|
||
|
||
Comment 3•10 years ago
|
||
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
status-firefox39:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla39
You need to log in
before you can comment on or make changes to this bug.
Description
•