Closed Bug 1389086 Opened 7 years ago Closed 7 years ago

Intermittent dom/events/test/pointerevents/test_bug1323158.html | Should receive pointerover before mouseover

Categories

(Core :: DOM: Events, defect)

Product:
Core
Component:
DOM: Events
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

()

Status:
RESOLVED FIXED

People

(Reporter: intermittent-bug-filer, Assigned: stone)

Details

(Keywords: intermittent-failure, Whiteboard: [stockwell fixed])

Attachments

(3 files)

Disable intermittent failed test case test_bug1323158.html
1.34 KB, patch
bevis
: review+
Details | Diff | Splinter Review
Remove windows line ending in the test file
8.50 KB, patch
smaug
: review+
Details | Diff | Splinter Review
Use once event listeners to avoid they are triggered after the test is done.
3.04 KB, patch
smaug
: review+
Details | Diff | Splinter Review 
Filed by: archaeopteryx [at] coole-files.de

https://treeherder.mozilla.org/logviewer.html#?job_id=122271519&repo=mozilla-inbound

https://queue.taskcluster.net/v1/task/FyjUUKaySxGSlIfAGADz2w/runs/0/artifacts/public/test_info/plain-chunked_errorsummary.log

12:32:43     INFO -  508 INFO TEST-PASS | dom/events/test/pointerevents/test_bug1323158.html | relatedTarget of mouseleave should be the same as the correcponding pointer event expect [object HTMLDivElement] got [object HTMLDivElement]
12:32:43     INFO -  Buffered messages finished
12:32:43    ERROR -  509 INFO TEST-UNEXPECTED-FAIL | dom/events/test/pointerevents/test_bug1323158.html | Should receive pointerover before mouseover
12:32:43     INFO -      runTests/</<@dom/events/test/pointerevents/test_bug1323158.html:53:7
12:32:43     INFO -      EventListener.handleEvent*runTests/<@dom/events/test/pointerevents/test_bug1323158.html:51:5
12:32:43     INFO -      runTests@dom/events/test/pointerevents/test_bug1323158.html:50:3
12:32:43     INFO -      Async*@dom/events/test/pointerevents/test_bug1323158.html:88:3
12:32:43     INFO -      focusedOrLoaded/<@SimpleTest/SimpleTest.js:795:59
Stone, have you noticed something changed in the past few days that caused this intermittent failure appear? It seems to me something we could consider to disable the test temporarily and then to enable in 58 (before we ship PE to release) due to 57 resource conflict (but I honestly have little idea about the impact of disabling this test), wdyt?
Flags: needinfo?(sshih)
Assignee: nobody → sshih
Flags: needinfo?(sshih)
(In reply to Hsin-Yi Tsai [:hsinyi] from comment #5)
> Stone, have you noticed something changed in the past few days that caused
> this intermittent failure appear? It seems to me something we could consider
> to disable the test temporarily and then to enable in 58 (before we ship PE
> to release) due to 57 resource conflict (but I honestly have little idea
> about the impact of disabling this test), wdyt?

Have no idea what causes this problem. Should be fine to disable it at this moment since it's PE specific.
Attachment #8896878 - Flags: review?(btseng)
Attachment #8896878 - Flags: review?(btseng) → review+
:kats - might this have a similar cause to bug 1388955?
Flags: needinfo?(bugmail)
I don't think so. It seems like this test sends mouse events, and doesn't use the native synthesization codepaths, so it's probably a different issue.
Flags: needinfo?(bugmail)
Pushed by sshih@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/2df4acb21215
Disable intermittent failed test case test_bug1323158.html. r=bevis.
Keywords: leave-open
Whiteboard: [stockwell needswork:DOM] → [stockwell disabled]
Whiteboard: [stockwell needswork:DOM] → [stockwell needswork:owner]
Whiteboard: [stockwell needswork:owner] → [stockwell disabled]
Whiteboard: [stockwell needswork:owner] → [stockwell disabled]
An unrelated mousemove is sent to target1 when the test is finished, which breaks this test. Using once event listeners to avoid receiving unrelated events.
Attachment #8903490 - Flags: review?(bugs)
Attachment #8903489 - Flags: review?(bugs)
Comment on attachment 8903489 [details] [diff] [review]
Remove windows line ending in the test file

rs+
Attachment #8903489 - Flags: review?(bugs) → review+
Attachment #8903490 - Flags: review?(bugs) → review+
Pushed by sshih@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/861bccda61e4
Part1: Remove windows line ending in the test file. r=smaug.
https://hg.mozilla.org/integration/mozilla-inbound/rev/0335f419f10b
Part2: Use once event listeners to avoid they are triggered after the test is done. r=smaug.
Keywords: leave-open
No intermittent failures happened after landing. Closed it.
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Whiteboard: [stockwell disabled] → [stockwell fixed]
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: