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.
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.
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.