Closed Bug 547342 Opened 12 years ago Closed 11 years ago

random orange in /tests/content/xul/templates/tests/test_bug441785.xul


(Core :: XUL, defect, P1)

Windows 7





(Reporter: ddahl, Assigned: bzbarsky)



(Keywords: intermittent-failure)


(2 files)

135 INFO Running /tests/content/xul/templates/tests/test_bug441785.xul...
++DOMWINDOW == 25 (07A6CC00) [serial = 32] [outer = 074B5100]
136 ERROR TEST-UNEXPECTED-FAIL | /tests/content/xul/templates/tests/test_bug441785.xul | first tree row count datasource 1 - got 0, expected 10
137 INFO TEST-PASS | /tests/content/xul/templates/tests/test_bug441785.xul | second tree row count datasource 1

At the time this one popped up, bz said on irc that it is a problem with the test itself - i think.
WINNT 5.2 mozilla-central debug test mochitests-2/5 [testfailed] Started 15:46, finished 16:11
Component: Layout → DOM
QA Contact: layout → general

WINNT 5.2 mozilla-central opt test mochitests-2/5 [testfailed] Started 15:18, finished 15:28

WINNT 5.2 mozilla-central opt test mochitests-2/5 [testfailed] Started 13:48, finished 14:06
Component: DOM → XP Toolkit/Widgets: XUL
QA Contact: general → xptoolkit.xul
This works around the issue, but the real problem, imo, is that scrollToRow needs to flush.  Otherwise mView can be null when someone calls it, in which case it will silently fail.  Or someone might have set an explit view, in which case it can read random numbers from mPageLength (which the constructor does not initialize)...

We should probably set mPageLength to 0 in the constructor, and have more box object methods here flushing layout.  I'm just not sure which ones.
Assignee: nobody → bzbarsky
Attachment #427906 - Flags: review?(roc)
Attachment #427906 - Flags: review?(enndeakin)
Comment on attachment 427906 [details] [diff] [review]
Temporary fix I pushed

Look ok for now. Flushing before calling looks ok too.
Attachment #427906 - Flags: review?(enndeakin) → review+
Blocks: 438871
Priority: -- → P1
Attachment #463078 - Flags: review?
Attachment #463078 - Flags: review? → review?(enndeakin)
Whiteboard: [orange] → [need review][orange]
Attachment #463078 - Flags: review?(enndeakin) → review+
Whiteboard: [need review][orange] → [need checkin][orange]
Summary: random orange in content/xul/templates/tests/bug441785.xul → random orange in /tests/content/xul/templates/tests/test_bug441785.xul
Hmm.  Looks like the patch I just checked in didn't work.  Investigating.
Yeah, ok.  That patch was wrong, because it fails to flush layout when we have a cached frame.  Pushed fix for that.  So overall, pushed:
Closed: 11 years ago
Flags: in-testsuite+
Resolution: --- → FIXED
Whiteboard: [need checkin][orange] → [orange]
Target Milestone: --- → mozilla2.0b4
Depends on: 585123
Depends on: 685457
Whiteboard: [orange]
Moving to Core:XUL per
Component: XP Toolkit/Widgets: XUL → XUL
You need to log in before you can comment on or make changes to this bug.