Closed Bug 1139272 Opened 6 years ago Closed 6 years ago
put expensive ns
Style Context destructor assertions behind a pref
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: nobody → cam
Status: NEW → ASSIGNED
Attachment #8572441 - Flags: review?(dbaron)
Attachment #8572441 - Flags: review?(dbaron) → review+
You need to log in before you can comment on or make changes to this bug.