Closed
Bug 1219980
Opened 9 years ago
Closed 9 years ago
Intermittent test_basic_pan.html | Test timed out.
Categories
(Core :: Panning and Zooming, defect)
Tracking
()
RESOLVED
FIXED
mozilla47
Tracking | Status | |
---|---|---|
firefox47 | --- | fixed |
People
(Reporter: KWierso, Assigned: kats)
References
Details
(Keywords: intermittent-failure, Whiteboard: [gfx-noted])
Attachments
(1 file)
Updated•9 years ago
|
Whiteboard: [gfx-noted]
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment 5•9 years ago
|
||
it happens very often on emu-x86-kk. See log for mochitest chunk 44 https://treeherder.mozilla.org/#/jobs?repo=b2g-inbound&revision=048b60ee7fdd&exclusion_profile=false&selectedJob=3719233
Blocks: 1192121
Comment 6•9 years ago
|
||
(In reply to Hsin-Yi Tsai [:hsinyi] from comment #5)
> it happens very often on emu-x86-kk. See log for mochitest chunk 44
> https://treeherder.mozilla.org/#/jobs?repo=b2g-
> inbound&revision=048b60ee7fdd&exclusion_profile=false&selectedJob=3719233
After the second glance over the log, I learnt that bug 1238464 should be the right bug. Sorry for the noise.
No longer blocks: 1192121
Comment hidden (Intermittent Failures Robot) |
Comment 9•9 years ago
|
||
In comment 7, the failed is at emulator-ics. I think it's related to https://bugzilla.mozilla.org/show_bug.cgi?id=1231042#c49 . And we already have another fix.
Flags: needinfo?(hshih)
Assignee | ||
Comment 10•9 years ago
|
||
Sounds like this got fixed either by bug 1231042 or something else.
Status: NEW → RESOLVED
Closed: 9 years ago
Component: Graphics: Layers → Panning and Zooming
Resolution: --- → WORKSFORME
Comment hidden (Intermittent Failures Robot) |
Assignee | ||
Comment 12•9 years ago
|
||
Since this bug is the one getting starred we might as well reopen it, even though it is a different issue than before. The new failures are fennec-only, and probably a regression from enabling this test on fennec which I did recently.
Status: RESOLVED → REOPENED
Resolution: WORKSFORME → ---
Assignee | ||
Comment 13•9 years ago
|
||
Took a quick look at some of the logs, but there's not too much useful info. It looks like the events are getting synthesized at least, but somewhere between there and the "TransformEnd" message things go off the rails so the TransformEnd message never arrives. I'll do a try push with more logging.
Assignee | ||
Comment 14•9 years ago
|
||
My try push is at https://treeherder.mozilla.org/#/jobs?repo=try&revision=0324e253ae66
Assignee | ||
Comment 15•9 years ago
|
||
Ah from [1] the logging seems to indicate that the attempt at dragging turns into a long-press instead.
[1] http://mozilla-releng-blobs.s3.amazonaws.com/blobs/try/sha512/5dacfdee24188d4ed3fefcb7eb0e3ca39407e454b397a2b9ca055655510eb19b3f0848a98a9f74f56f07a2ce7357e3fea1ba0ca08feb57f9b2f4daca00d627be
Assignee | ||
Comment 16•9 years ago
|
||
The pref sets the long-press timeout to 10s but from the log it's not waiting that long. Probably because of chaos mode being enabled on the test. Disabling chaos mode should fix this.
Assignee | ||
Comment 17•9 years ago
|
||
Try push to verify: https://treeherder.mozilla.org/#/jobs?repo=try&revision=be37588c146f
Assignee: nobody → bugmail.mozilla
Version: 44 Branch → 47 Branch
Assignee | ||
Comment 18•9 years ago
|
||
Review commit: https://reviewboard.mozilla.org/r/37385/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/37385/
Attachment #8725259 -
Flags: review?(botond)
Assignee | ||
Comment 19•9 years ago
|
||
Comment on attachment 8725259 [details]
MozReview Request: Bug 1219980 - Ensure that long-press gestures don't get detected during test_basic_pan.html r?botond
Spoke too soon. There's another failure on the try push with chaos mode disabled. Seems to be a long-press still, but I'm not sure why it happened.
Attachment #8725259 -
Flags: review?(botond)
Assignee | ||
Comment 20•9 years ago
|
||
Ah, logging indicates that it's still using 500ms as the timeout. Not sure why yet but I should be able to repro that locally.
Assignee | ||
Comment 21•9 years ago
|
||
The problem was that the pref is not specified in all.js, so specialpowersAPI.js guessed at the type of the pref. It guessed a CHAR pref because the value was in quotes, but really it's an INT pref. Dropping the quotes makes it work as intended.
Assignee | ||
Comment 22•9 years ago
|
||
Comment on attachment 8725259 [details]
MozReview Request: Bug 1219980 - Ensure that long-press gestures don't get detected during test_basic_pan.html r?botond
Review request updated; see interdiff: https://reviewboard.mozilla.org/r/37385/diff/1-2/
Attachment #8725259 -
Attachment description: MozReview Request: Bug 1219980 - Disable chaos mode on test_basic_pan, because it can result in long-press gestures being detected. r?botond → MozReview Request: Bug 1219980 - Ensure that long-press gestures don't get detected during test_basic_pan.html r?botond
Attachment #8725259 -
Flags: review?(botond)
Assignee | ||
Comment 23•9 years ago
|
||
Updated•9 years ago
|
Attachment #8725259 -
Flags: review?(botond) → review+
Comment 24•9 years ago
|
||
Comment on attachment 8725259 [details]
MozReview Request: Bug 1219980 - Ensure that long-press gestures don't get detected during test_basic_pan.html r?botond
https://reviewboard.mozilla.org/r/37385/#review34921
Comment 25•9 years ago
|
||
Comment 26•9 years ago
|
||
bugherder |
Status: REOPENED → RESOLVED
Closed: 9 years ago → 9 years ago
status-firefox47:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla47
Comment hidden (Intermittent Failures Robot) |
You need to log in
before you can comment on or make changes to this bug.
Description
•