Intermittent test_plugin_mouse_coords.html | p1 mouse up Y - got 38, expected 28 | p2 mouse up Y - got 38, expected 28 | p3 mouse up Y - got 31, expected 21 followed by failures in test_selection_expanding.html

RESOLVED FIXED in Firefox 24

Status

()

RESOLVED FIXED
5 years ago
5 years ago

People

(Reporter: RyanVM, Assigned: johns)

Tracking

({intermittent-failure})

Trunk
mozilla26
x86
Windows 8
intermittent-failure
Points:
---
Bug Flags:
in-testsuite +

Firefox Tracking Flags

(firefox24 fixed, firefox25 fixed, firefox26 fixed)

Details

Attachments

(3 attachments)

(Reporter)

Description

5 years ago
https://tbpl.mozilla.org/php/getParsedLog.php?id=23236791&tree=Fx-Team

WINNT 6.2 fx-team pgo test mochitest-5 on 2013-05-21 21:13:44 PDT for push 1c2ca70f459e
slave: t-w864-ix-074

21:18:54     INFO -  1841 INFO TEST-START | /tests/layout/generic/test/test_plugin_mouse_coords.html
21:18:57     INFO -  1842 INFO Error: Unable to restore focus, expect failures and timeouts.
21:18:58     INFO -  1843 INFO TEST-PASS | /tests/layout/generic/test/test_plugin_mouse_coords.html | p1 mouse down X
21:18:58     INFO -  1844 INFO TEST-PASS | /tests/layout/generic/test/test_plugin_mouse_coords.html | p1 mouse down Y
21:18:58     INFO -  1845 INFO TEST-PASS | /tests/layout/generic/test/test_plugin_mouse_coords.html | p1 mouse move X
21:18:58     INFO -  1846 INFO TEST-PASS | /tests/layout/generic/test/test_plugin_mouse_coords.html | p1 mouse move Y
21:18:58     INFO -  1847 INFO TEST-PASS | /tests/layout/generic/test/test_plugin_mouse_coords.html | p1 mouse up X
21:18:58     INFO -  1848 ERROR TEST-UNEXPECTED-FAIL | /tests/layout/generic/test/test_plugin_mouse_coords.html | p1 mouse up Y - got 38, expected 28
21:18:58     INFO -  1849 INFO TEST-PASS | /tests/layout/generic/test/test_plugin_mouse_coords.html | p2 mouse down X
21:18:58     INFO -  1850 INFO TEST-PASS | /tests/layout/generic/test/test_plugin_mouse_coords.html | p2 mouse down Y
21:18:58     INFO -  1851 INFO TEST-PASS | /tests/layout/generic/test/test_plugin_mouse_coords.html | p2 mouse move X
21:18:58     INFO -  1852 INFO TEST-PASS | /tests/layout/generic/test/test_plugin_mouse_coords.html | p2 mouse move Y
21:18:58     INFO -  1853 INFO TEST-PASS | /tests/layout/generic/test/test_plugin_mouse_coords.html | p2 mouse up X
21:18:58     INFO -  1854 ERROR TEST-UNEXPECTED-FAIL | /tests/layout/generic/test/test_plugin_mouse_coords.html | p2 mouse up Y - got 38, expected 28
21:18:58     INFO -  1855 INFO TEST-PASS | /tests/layout/generic/test/test_plugin_mouse_coords.html | p3 mouse down X
21:18:58     INFO -  1856 INFO TEST-PASS | /tests/layout/generic/test/test_plugin_mouse_coords.html | p3 mouse down Y
21:18:58     INFO -  1857 INFO TEST-PASS | /tests/layout/generic/test/test_plugin_mouse_coords.html | p3 mouse move X
21:18:58     INFO -  1858 INFO TEST-PASS | /tests/layout/generic/test/test_plugin_mouse_coords.html | p3 mouse move Y
21:18:58     INFO -  1859 INFO TEST-PASS | /tests/layout/generic/test/test_plugin_mouse_coords.html | p3 mouse up X
21:18:58     INFO -  1860 ERROR TEST-UNEXPECTED-FAIL | /tests/layout/generic/test/test_plugin_mouse_coords.html | p3 mouse up Y - got 31, expected 21
21:18:58     INFO -  1861 INFO TEST-END | /tests/layout/generic/test/test_plugin_mouse_coords.html | finished in 4082ms

21:19:11     INFO -  1913 ERROR TEST-UNEXPECTED-FAIL | /tests/layout/generic/test/test_selection_expanding.html | The contents of div1 aren't selected (div1-div2, all boxes are overflow: visible;): Selected String: ""
21:19:11     INFO -  1914 ERROR TEST-UNEXPECTED-FAIL | /tests/layout/generic/test/test_selection_expanding.html | The contents of div2 aren't selected (div1-div2, all boxes are overflow: visible;): Selected String: ""
21:19:11     INFO -  1925 ERROR TEST-UNEXPECTED-FAIL | /tests/layout/generic/test/test_selection_expanding.html | The contents of div1 aren't selected (div1-div3, all boxes are overflow: visible;): Selected String: ""
21:19:11     INFO -  1926 ERROR TEST-UNEXPECTED-FAIL | /tests/layout/generic/test/test_selection_expanding.html | The contents of div2 aren't selected (div1-div3, all boxes are overflow: visible;): Selected String: ""
21:19:11     INFO -  1927 ERROR TEST-UNEXPECTED-FAIL | /tests/layout/generic/test/test_selection_expanding.html | The contents of div3 aren't selected (div1-div3, all boxes are overflow: visible;): Selected String: ""
21:19:11     INFO -  1937 ERROR TEST-UNEXPECTED-FAIL | /tests/layout/generic/test/test_selection_expanding.html | The contents of div1 aren't selected (div1-fixedDiv1, all boxes are overflow: visible;): Selected String: ""
21:19:11     INFO -  1938 ERROR TEST-UNEXPECTED-FAIL | /tests/layout/generic/test/test_selection_expanding.html | The contents of div2 aren't selected (div1-fixedDiv1, all boxes are overflow: visible;): Selected String: ""
21:19:11     INFO -  1939 ERROR TEST-UNEXPECTED-FAIL | /tests/layout/generic/test/test_selection_expanding.html | The contents of div3 aren't selected (div1-fixedDiv1, all boxes are overflow: visible;): Selected String: ""
21:19:12     INFO -  1941 ERROR TEST-UNEXPECTED-FAIL | /tests/layout/generic/test/test_selection_expanding.html | The contents of fixedDiv1 aren't selected (div1-fixedDiv1, all boxes are overflow: visible;): Selected String: ""
21:19:12     INFO -  1949 ERROR TEST-UNEXPECTED-FAIL | /tests/layout/generic/test/test_selection_expanding.html | The contents of div1 aren't selected (div1-xbl_child, all boxes are overflow: visible;): Selected String: ""
21:19:12     INFO -  1950 ERROR TEST-UNEXPECTED-FAIL | /tests/layout/generic/test/test_selection_expanding.html | The contents of div2 aren't selected (div1-xbl_child, all boxes are overflow: visible;): Selected String: ""
21:19:12     INFO -  1951 ERROR TEST-UNEXPECTED-FAIL | /tests/layout/generic/test/test_selection_expanding.html | The contents of div3 aren't selected (div1-xbl_child, all boxes are overflow: visible;): Selected String: ""
21:19:12     INFO -  1952 ERROR TEST-UNEXPECTED-FAIL | /tests/layout/generic/test/test_selection_expanding.html | The contents of xbl_child aren't selected (div1-xbl_child, all boxes are overflow: visible;): Selected String: ""
21:19:12     INFO -  1953 ERROR TEST-UNEXPECTED-FAIL | /tests/layout/generic/test/test_selection_expanding.html | The contents of fixedDiv1 aren't selected (div1-xbl_child, all boxes are overflow: visible;): Selected String: ""
21:19:12     INFO -  1961 ERROR TEST-UNEXPECTED-FAIL | /tests/layout/generic/test/test_selection_expanding.html | The contents of div1 aren't selected (div1-fixedDiv2, all boxes are overflow: visible;): Selected String: ""
21:19:12     INFO -  1962 ERROR TEST-UNEXPECTED-FAIL | /tests/layout/generic/test/test_selection_expanding.html | The contents of div2 aren't selected (div1-fixedDiv2, all boxes are overflow: visible;): Selected String: ""
21:19:12     INFO -  1963 ERROR TEST-UNEXPECTED-FAIL | /tests/layout/generic/test/test_selection_expanding.html | The contents of div3 aren't selected (div1-fixedDiv2, all boxes are overflow: visible;): Selected String: ""
21:19:12     INFO -  1964 ERROR TEST-UNEXPECTED-FAIL | /tests/layout/generic/test/test_selection_expanding.html | The contents of xbl_child aren't selected (div1-fixedDiv2, all boxes are overflow: visible;): Selected String: ""
21:19:12     INFO -  1965 ERROR TEST-UNEXPECTED-FAIL | /tests/layout/generic/test/test_selection_expanding.html | The contents of fixedDiv1 aren't selected (div1-fixedDiv2, all boxes are overflow: visible;): Selected String: ""
21:19:12     INFO -  1966 ERROR TEST-UNEXPECTED-FAIL | /tests/layout/generic/test/test_selection_expanding.html | The contents of fixedDiv2 aren't selected (div1-fixedDiv2, all boxes are overflow: visible;): Selected String: ""
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
(Reporter)

Comment 104

5 years ago
Roc, can you please take a look at this frequent orange or suggest someone who can? :)
Flags: needinfo?(roc)
John, are you still working on plugin bugs?
Assignee: nobody → jschoenick
Flags: needinfo?(roc)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
(Assignee)

Comment 114

5 years ago
Created attachment 778737 [details] [diff] [review]
Cleanup plugin_focus_helper test events

There's a bunch of iffy event ordering around here, and these failures all seem to start with an "Unable to restore focus" complaint. I suspect this is because we are calling SimpleTest.finish() before window.close() in these tests, but I added a bunch of strict event ordering and removed some setTimeout(1000)s that were previously just hoping for the best.
Attachment #778737 - Flags: review?(roc)
(Assignee)

Updated

5 years ago
Keywords: checkin-needed
Whiteboard: [leave open]
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
(Reporter)

Comment 120

5 years ago
https://hg.mozilla.org/integration/mozilla-inbound/rev/b1622b3acf58
Flags: in-testsuite+
Keywords: checkin-needed
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
(Assignee)

Comment 126

5 years ago
Looks like that did the trick
Status: NEW → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
Whiteboard: [leave open]
Comment hidden (Treeherder Robot)
(Reporter)

Comment 128

5 years ago
(In reply to TinderboxPushlog Robot from comment #127)

Rats.
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
(Assignee)

Comment 129

5 years ago
Created attachment 781183 [details] [diff] [review]
Fix typo in test r=typo

BUT WAIT, there was a typo that was making SimpleTest.finish race with window.close!

IT COULD STILL HOLD
Comment hidden (Treeherder Robot)
(Reporter)

Comment 132

5 years ago
https://hg.mozilla.org/mozilla-central/rev/c8bc029b8553

Excelsior!
Status: REOPENED → RESOLVED
Last Resolved: 5 years ago5 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla25
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
(Reporter)

Comment 135

5 years ago
(In reply to TinderboxPushlog Robot from comment #134)
:(
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
(Assignee)

Comment 150

5 years ago
Created attachment 785106 [details] [diff] [review]
Fix focus race in test on windows 8

I win!

https://tbpl.mozilla.org/?tree=Try&rev=4430c06ec9a5

Best theory: The test opens a popup window that has OOP child windows (plugins) that have been issued native click events. When we close that window and then call finish(), the harness sees windows hasn't returned focus to the main window yet and attempts to reclaim it. This appears to race with some OS logic and causes firefox to lose focus entirely. The waitForFocus() on the main window after closing the popup just waits for the OS to get around to returning focus to us, and seems to fix everything.

I'm not sure if the other changes are necessary, but they don't hurt.
Attachment #785106 - Flags: review?(roc)
Comment hidden (Treeherder Robot)
Comment on attachment 785106 [details] [diff] [review]
Fix focus race in test on windows 8

Review of attachment 785106 [details] [diff] [review]:
-----------------------------------------------------------------

genius
Attachment #785106 - Flags: review?(roc) → review+
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
(Assignee)

Updated

5 years ago
Attachment #778737 - Flags: checkin+
(Assignee)

Updated

5 years ago
Whiteboard: [leave open]
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
(Assignee)

Comment 163

5 years ago
Only occurrences since fix #2 are on branches without the patch, hopefully fixed for good this time.
Status: REOPENED → RESOLVED
Last Resolved: 5 years ago5 years ago
Resolution: --- → FIXED
Whiteboard: [leave open]
Comment hidden (Treeherder Robot)