Open Bug 668716 Opened 9 years ago Updated 10 months ago

Intermittent TEST-UNEXPECTED-PASS test_bug632379.xhtml,test_bug632379.xul | Popup should open in the same place when the menu is scrolled - 109 should equal 109 since it got moved to mochitest-chrome

Categories

(Core :: Layout, defect, P5)

All
macOS
defect

Tracking

()

Tracking Status
firefox14 --- fixed
firefox15 - fixed

People

(Reporter: philor, Unassigned)

References

()

Details

(Keywords: intermittent-failure, regression, Whiteboard: [test disabled on mac])

Attachments

(1 file, 4 obsolete files)

This test had a nice run of not failing when it was a plain mochitest, but this is the fourth or fifth time I've seen it fail since it turned chrome.

http://tinderbox.mozilla.org/showlog.cgi?log=Firefox/1309495383.1309497155.17105.gz

Rev3 MacOSX Leopard 10.5.8 mozilla-central opt test mochitest-other on 2011/06/30 21:43:03
s: talos-r3-leopard-003

10730 INFO TEST-START | chrome://mochitests/content/chrome/layout/generic/test/test_bug632379.xul
10731 ERROR TEST-UNEXPECTED-FAIL | chrome://mochitests/content/chrome/layout/generic/test/test_bug632379.xul | Popup should open in the same place when the menu is scrolled - got 109, expected 108
10732 INFO TEST-END | chrome://mochitests/content/chrome/layout/generic/test/test_bug632379.xul | finished in 1184ms
philor pointed out that it seems like with my patches from bug 735805, this fails consistently on Mac instead of being intermittent.  I'm updating my patchset there to make the is() a todo_is() on Mac.  Let's see if that works: <https://tbpl.mozilla.org/?tree=Try&rev=124f225b25a1>
Blocks: 663844
Depends on: 735805
Keywords: regression
Hardware: x86 → All
(In reply to Aryeh Gregor from comment #86)
> philor pointed out that it seems like with my patches from bug 735805, this
> fails consistently on Mac instead of being intermittent.

Ftr, waitForExplicitFinish(), that bug 735805 will re-add, had been mistakenly removed by bug 675201.
Oops -- I forgot to include the fix I mentioned in comment 86 in my push.  This patch should make it a todo() on Mac instead of permanent orange.  The try run I linked to there demonstrates that it should work.
Ms2ger said to push it with r=orange, so I did:

https://hg.mozilla.org/integration/mozilla-inbound/rev/3b06cfee48cd

Hope that was the right thing to do.
Whiteboard: [orange] → [orange][Leave open after merge]
Sadly, even a blind pig finds an UNEXPECTED-PASS every now and then.
Summary: Intermittent test_bug632379.xul | Popup should open in the same place when the menu is scrolled - got 109, expected 108 since it got moved to mochitest-chrome → Intermittent TEST-UNEXPECTED-PASS test_bug632379.xul | Popup should open in the same place when the menu is scrolled - got 109, expected 108 since it got moved to mochitest-chrome
Attachment #608064 - Attachment description: Patch to change it to a todo() → Patch to change it to a todo() [Checked in: Comment 99]
Summary: Intermittent TEST-UNEXPECTED-PASS test_bug632379.xul | Popup should open in the same place when the menu is scrolled - got 109, expected 108 since it got moved to mochitest-chrome → Intermittent TEST-UNEXPECTED-PASS test_bug632379.xul | Popup should open in the same place when the menu is scrolled - 109 should equal 109 since it got moved to mochitest-chrome