test_mousecapture.xul is going to permafail when Gecko 42 merges to Aurora

VERIFIED FIXED in Firefox 42

Status

()

--
critical
VERIFIED FIXED
4 years ago
4 years ago

People

(Reporter: RyanVM, Assigned: smaug)

Tracking

Other Branch
mozilla42
Points:
---
Bug Flags:
in-testsuite +

Firefox Tracking Flags

(firefox42 verified)

Details

Attachments

(1 attachment)

(Reporter)

Description

4 years ago
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

4 years ago
Any idea what might be going on here, smaug?
Flags: needinfo?(bugs)
(Assignee)

Comment 2

4 years ago
oh, yes, pointer events are enabled in nightly only.
Assignee: nobody → bugs
Flags: needinfo?(bugs)
(Assignee)

Comment 3

4 years ago
Created attachment 8644949 [details] [diff] [review]
patch

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

4 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

4 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

4 years ago
https://hg.mozilla.org/mozilla-central/rev/2336fd9dcb91
Status: NEW → RESOLVED
Last Resolved: 4 years ago
status-firefox42: --- → fixed
Flags: in-testsuite+
Resolution: --- → FIXED
Target Milestone: --- → mozilla42
(Reporter)

Updated

4 years ago
Status: RESOLVED → VERIFIED
status-firefox42: fixed → verified
You need to log in before you can comment on or make changes to this bug.