Closed
Bug 1191299
Opened 10 years ago
Closed 10 years ago
test_mousecapture.xul is going to permafail when Gecko 42 merges to Aurora
Categories
(Core :: DOM: Events, defect)
Tracking
()
VERIFIED
FIXED
mozilla42
Tracking | Status | |
---|---|---|
firefox42 | --- | verified |
People
(Reporter: RyanVM, Assigned: smaug)
References
Details
Attachments
(1 file)
827 bytes,
patch
|
RyanVM
:
review+
|
Details | Diff | Splinter Review |
The uplift is next Monday, so please look at this ASAP.
https://treeherder.mozilla.org/logviewer.html#?job_id=9856618&repo=try
08:13:35 INFO - 2993 INFO TEST-UNEXPECTED-OK | toolkit/content/tests/chrome/test_mousecapture.xul | Assertion count 6 is less than expected range 12-12 assertions.
Flags: needinfo?(alessarik)
Reporter | ||
Comment 1•10 years ago
|
||
Any idea what might be going on here, smaug?
Flags: needinfo?(bugs)
Assignee | ||
Comment 2•10 years ago
|
||
oh, yes, pointer events are enabled in nightly only.
Assignee: nobody → bugs
Flags: needinfo?(bugs)
Assignee | ||
Comment 3•10 years ago
|
||
This should work, didn't test though :)
The assertion is something which both mouse events and pointer events trigger, so
on nightly we have 2x more them.
Attachment #8644949 -
Flags: review?(ryanvm)
Reporter | ||
Comment 4•10 years ago
|
||
Comment on attachment 8644949 [details] [diff] [review]
patch
Review of attachment 8644949 [details] [diff] [review]:
-----------------------------------------------------------------
Thanks!
Attachment #8644949 -
Flags: review?(ryanvm) → review+
Comment 6•10 years ago
|
||
(In reply to Olli Pettay [:smaug] from comment #3)
> The assertion is something which both mouse events and pointer events
> trigger, so on nightly we have 2x more them.
Maybe it is the main reason, and I could not add more usefull information.
Flags: needinfo?(alessarik)
Reporter | ||
Comment 7•10 years ago
|
||
Status: NEW → RESOLVED
Closed: 10 years ago
status-firefox42:
--- → fixed
Flags: in-testsuite+
Resolution: --- → FIXED
Target Milestone: --- → mozilla42
Reporter | ||
Updated•10 years ago
|
Status: RESOLVED → VERIFIED
You need to log in
before you can comment on or make changes to this bug.
Description
•