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)

47 Branch
defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla47
Tracking Status
firefox47 --- fixed

People

(Reporter: KWierso, Assigned: kats)

References

Details

(Keywords: intermittent-failure, Whiteboard: [gfx-noted])

Attachments

(1 file)

Whiteboard: [gfx-noted]
(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
Jerry, this might be related to bug 1238464
Flags: needinfo?(hshih)
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)
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
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 → ---
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.
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: nobody → bugmail.mozilla
Version: 44 Branch → 47 Branch
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)
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.
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.
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)
Attachment #8725259 - Flags: review?(botond) → review+
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
Status: REOPENED → RESOLVED
Closed: 9 years ago9 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla47
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: