"ASSERTION: post-reflow queues not empty" with xul:listboxbody and xul:text

RESOLVED FIXED

Status

()

defect
--
major
RESOLVED FIXED
12 years ago
11 years ago

People

(Reporter: jruderman, Assigned: smaug)

Tracking

(Blocks 1 bug, 5 keywords)

Trunk
x86
macOS
Points:
---
Dependency tree / graph
Bug Flags:
blocking1.8.1.8 +
in-testsuite +

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(2 attachments)

Reporter

Description

12 years ago
*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.
Flags: blocking1.9?
Assignee

Updated

12 years ago
Assignee: nobody → Olli.Pettay
Assignee

Comment 1

12 years ago
Sorry, my mistake.
Better to post the callback when reflowing, then it gets called always and there
is no need to cancel it.
Attachment #278751 - Flags: superreview?(roc)
Attachment #278751 - Flags: review?(roc)
Assignee

Updated

12 years ago
Blocks: 387033
Attachment #278751 - Flags: superreview?(roc)
Attachment #278751 - Flags: superreview+
Attachment #278751 - Flags: review?(roc)
Attachment #278751 - Flags: review+
Attachment #278751 - Flags: approval1.9+
Assignee

Updated

12 years ago
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED

Updated

12 years ago
Depends on: 368573
Flags: in-testsuite?
qawanted: verify this doesn't regress on the 1.8 branch after bug 387033 lands
Flags: blocking1.8.1.8+
Keywords: qawanted
Assignee

Updated

12 years ago
Keywords: fixed1.8.1.8
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.
Flags: blocking1.9?
Reporter

Comment 4

12 years ago
Crashtest checked in.
Flags: in-testsuite? → in-testsuite+

Updated

11 years ago
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.