Closed
Bug 1361276
Opened 8 years ago
Closed 7 years ago
Re-enable browser_oneOffHeader.js for OS X (gets intermittent from bug 1352364)
Categories
(Firefox :: Search, enhancement)
Firefox
Search
Tracking
()
RESOLVED
FIXED
Firefox 59
Tracking | Status | |
---|---|---|
firefox59 | --- | fixed |
People
(Reporter: johannh, Assigned: arai)
References
Details
Attachments
(1 file, 1 obsolete file)
(deleted),
patch
|
johannh
:
review+
|
Details | Diff | Splinter Review |
In bug 1352364 we change a lot of stuff but nothing particular to search AFAIK. browser_oneOffHeader still fails on try but we can't really reproduce it locally. It seems that this test is generally very brittle regarding timing. We'd like to avoid working on this in bug 1352364 since that one is big enough already and instead look at the test as a follow-up.
Reporter | ||
Comment 1•8 years ago
|
||
Forgot to mention that this is only on OSX.
Summary: browser_oneOffHeader.js gets intermittent from bug 1352364 → browser_oneOffHeader.js gets intermittent from bug 1352364 on OSX
Updated•7 years ago
|
Summary: browser_oneOffHeader.js gets intermittent from bug 1352364 on OSX → Re-enable browser_oneOffHeader.js for OS X (gets intermittent from bug 1352364)
Assignee | ||
Comment 2•7 years ago
|
||
this is sometimes reproducible for me.
Assignee | ||
Comment 3•7 years ago
|
||
(context: now I'm fixing bug 1416446 and others under bug 1414180, bug 1193394)
for me, mouseover event handler in synthesizeNativeMouseMove is not called and the test stops there, intermittently.
with this patch, the issue stops happening locally.
it delays the mouse event, and also move the cursor outside of the target element first, to make sure the cursor actually moves on the element.
Assignee | ||
Comment 4•7 years ago
|
||
Looks like, it's waiting for wrong event.
the actual event used in search UI is mousemove, so it should also wait for mousemove, but it's waiting for mouseover, and sometimes it's not fired.
switching to mousemove fixes local intermittent.
I'll ask review after some more tests on automation.
Assignee: nobody → arai.unmht
Attachment #8927637 -
Attachment is obsolete: true
Status: NEW → ASSIGNED
Assignee | ||
Comment 5•7 years ago
|
||
Assignee | ||
Comment 6•7 years ago
|
||
Comment on attachment 8929309 [details] [diff] [review]
Wait for mousemove instead of mouseover in browser/components/search/test/browser_oneOffHeader.js.
Changed to wait for mousemove, and removed skip-if
Attachment #8929309 -
Flags: review?(jhofmann)
Assignee | ||
Updated•7 years ago
|
Attachment #8929309 -
Attachment description: (WIP) Wait for mousemove instead of mouseover in browser/components/search/test/browser_oneOffHeader.js. → Wait for mousemove instead of mouseover in browser/components/search/test/browser_oneOffHeader.js.
Reporter | ||
Comment 7•7 years ago
|
||
Comment on attachment 8929309 [details] [diff] [review]
Wait for mousemove instead of mouseover in browser/components/search/test/browser_oneOffHeader.js.
Review of attachment 8929309 [details] [diff] [review]:
-----------------------------------------------------------------
Thank you for taking care of this!
Attachment #8929309 -
Flags: review?(jhofmann) → review+
Pushed by arai_a@mac.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/ae9e1ead6772
Wait for mousemove instead of mouseover in browser/components/search/test/browser_oneOffHeader.js. r=johannh
Assignee | ||
Comment 9•7 years ago
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/ae9e1ead6772ccc0e7d201f67cc786fe76e53bdd
Bug 1361276 - Wait for mousemove instead of mouseover in browser/components/search/test/browser_oneOffHeader.js. r=johannh
Comment 10•7 years ago
|
||
bugherder |
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
status-firefox59:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 59
You need to log in
before you can comment on or make changes to this bug.
Description
•