Created attachment 278729 [details] testcase (reload to see the assertions) *Reloading* the testcase triggers: ###!!! ASSERTION: post-reflow queues not empty. This means we're leaking: 'mFirstCallbackEventRequest == nsnull && mLastCallbackEventRequest == nsnull', file /Users/jruderman/trunk/mozilla/layout/base/nsPresShell.cpp, line 1402 ###!!! ASSERTION: Some objects allocated with AllocateFrame were not freed: 'mFrameCount == 0', file /Users/jruderman/trunk/mozilla/layout/base/nsPresShell.cpp, line 673 I'm marking this bug as major because being able to test for the "Some objects allocated with AllocateFrame were not freed" assertion is important for detecting a class of memory safety bugs.
Assignee: nobody → Olli.Pettay
Created attachment 278751 [details] [diff] [review] proposed patch Sorry, my mistake. Better to post the callback when reflowing, then it gets called always and there is no need to cancel it.
Status: NEW → RESOLVED
Last Resolved: 12 years ago
Resolution: --- → FIXED
qawanted: verify this doesn't regress on the 1.8 branch after bug 387033 lands
I can't really verify this on the 1.8 branch. My debug 1.8 branch builds are from before this patch was checked in, but I don't get any assertion when reloading the testcase with those.
Crashtest checked in.
Flags: in-testsuite? → in-testsuite+
Component: XP Toolkit/Widgets: XUL → XUL
QA Contact: xptoolkit.xul → xptoolkit.widgets
You need to log in before you can comment on or make changes to this bug.