Closed Bug 1514060 Opened 6 years ago Closed 5 years ago

Intermittent toolkit/content/tests/widgets/test_videocontrols.html | Test timed out.

Categories

(Toolkit :: Video/Audio Controls, defect, P3)

Product:
Toolkit
Component:
Video/Audio Controls
Type:
defect

Tracking

()

Status:
RESOLVED FIXED
Milestone:
mozilla70
Tracking Flags:
Tracking Status
thunderbird_esr60 --- wontfix
thunderbird_esr68 --- wontfix
firefox-esr60 --- wontfix
firefox-esr68 --- wontfix
firefox69 --- wontfix
firefox70 --- fixed

People

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

References

(Depends on 1 open bug)

Details

(Keywords: intermittent-failure)

Attachments

(1 file)

Bug 1514060 - Do not assume the video covers the entire screen. r=edgar,Gijs
(deleted), text/x-phabricator-request
Details
Filed by: rgurzau [at] mozilla.com https://treeherder.mozilla.org/logviewer.html#?job_id=216873894&repo=autoland https://queue.taskcluster.net/v1/task/X906mHpJRPigiGPczjLIBA/runs/0/artifacts/public/logs/live_backing.log [task 2018-12-13T21:37:36.170Z] 21:37:36 INFO - TEST-PASS | toolkit/content/tests/widgets/test_videocontrols.html | checking event type: pause [task 2018-12-13T21:37:36.171Z] 21:37:36 INFO - AddTask.js | Leaving test ensure_fullscreen_cursor [task 2018-12-13T21:37:36.171Z] 21:37:36 INFO - AddTask.js | Entering test ensure_fullscreen_button [task 2018-12-13T21:37:36.172Z] 21:37:36 INFO - TEST-PASS | toolkit/content/tests/widgets/test_videocontrols.html | checking event type: mozfullscreenchange [task 2018-12-13T21:37:36.173Z] 21:37:36 INFO - Buffered messages finished [task 2018-12-13T21:37:36.175Z] 21:37:36 INFO - TEST-UNEXPECTED-FAIL | toolkit/content/tests/widgets/test_videocontrols.html | Test timed out. [task 2018-12-13T21:37:36.175Z] 21:37:36 INFO - SimpleTest.ok@SimpleTest/SimpleTest.js:275:7 [task 2018-12-13T21:37:36.176Z] 21:37:36 INFO - reportError@SimpleTest/TestRunner.js:121:7 [task 2018-12-13T21:37:36.176Z] 21:37:36 INFO - TestRunner._checkForHangs@SimpleTest/TestRunner.js:142:7 [task 2018-12-13T21:37:36.177Z] 21:37:36 INFO - setTimeout handler*TestRunner._checkForHangs@SimpleTest/TestRunner.js:163:5 [task 2018-12-13T21:37:36.178Z] 21:37:36 INFO - setTimeout handler*TestRunner._checkForHangs@SimpleTest/TestRunner.js:163:5 [task 2018-12-13T21:37:36.179Z] 21:37:36 INFO - setTimeout handler*TestRunner._checkForHangs@SimpleTest/TestRunner.js:163:5 [task 2018-12-13T21:37:36.180Z] 21:37:36 INFO - setTimeout handler*TestRunner._checkForHangs@SimpleTest/TestRunner.js:163:5 [task 2018-12-13T21:37:36.181Z] 21:37:36 INFO - setTimeout handler*TestRunner._checkForHangs@SimpleTest/TestRunner.js:163:5 [task 2018-12-13T21:37:36.181Z] 21:37:36 INFO - setTimeout handler*TestRunner._checkForHangs@SimpleTest/TestRunner.js:163:5 [task 2018-12-13T21:37:36.182Z] 21:37:36 INFO - setTimeout handler*TestRunner._checkForHangs@SimpleTest/TestRunner.js:163:5 [task 2018-12-13T21:37:36.183Z] 21:37:36 INFO - setTimeout handler*TestRunner._checkForHangs@SimpleTest/TestRunner.js:163:5 [task 2018-12-13T21:37:36.183Z] 21:37:36 INFO - setTimeout handler*TestRunner._checkForHangs@SimpleTest/TestRunner.js:163:5 [task 2018-12-13T21:37:36.184Z] 21:37:36 INFO - setTimeout handler*TestRunner._checkForHangs@SimpleTest/TestRunner.js:163:5 [task 2018-12-13T21:37:36.184Z] 21:37:36 INFO - setTimeout handler*TestRunner._checkForHangs@SimpleTest/TestRunner.js:163:5 [task 2018-12-13T21:37:36.184Z] 21:37:36 INFO - TestRunner.runTests/<@SimpleTest/TestRunner.js:381:9 [task 2018-12-13T21:37:36.184Z] 21:37:36 INFO - promise callback*TestRunner.runTests@SimpleTest/TestRunner.js:368:50 [task 2018-12-13T21:37:36.184Z] 21:37:36 INFO - RunSet.runtests@SimpleTest/setup.js:201:3 [task 2018-12-13T21:37:36.185Z] 21:37:36 INFO - RunSet.runall@SimpleTest/setup.js:180:5 [task 2018-12-13T21:37:36.185Z] 21:37:36 INFO - hookupTests@SimpleTest/setup.js:273:5 [task 2018-12-13T21:37:36.185Z] 21:37:36 INFO - parseTestManifest@http://mochi.test:8888/manifestLibrary.js:36:5 [task 2018-12-13T21:37:36.185Z] 21:37:36 INFO - getTestManifest/req.onload@http://mochi.test:8888/manifestLibrary.js:49:11 [task 2018-12-13T21:37:36.185Z] 21:37:36 INFO - EventHandlerNonNull*getTestManifest@http://mochi.test:8888/manifestLibrary.js:45:3 [task 2018-12-13T21:37:36.185Z] 21:37:36 INFO - hookup@SimpleTest/setup.js:253:5

Bug 1574538 turns this into a perma-failure so I'm going to debug it. It needs to be fixed before I can land bug 1574538.

Assignee: nobody → mstange
Blocks: 1574538
Status: NEW → ASSIGNED
Priority: P5 → P3
Depends on: 1575630

I've filed bug 1575630 for a fix that would guarantee the document size to cover the entire screen when the mozfullscreenchange event is fired, but in the meantime, this workaround for the test seems reasonable and unblocks me.

This might not be the only problem with the test. I'm marking this bug as leave-open because, while I'm pretty confident the patch will fix the macOS problem, I don't know whether it'll fix the Linux problem.
Another source of trouble might be the video controls fadein/fadeout behavior. I think it's possible that the video controls disappear before the button is clicked.

Keywords: leave-open
Pushed by mstange@themasta.com: https://hg.mozilla.org/integration/autoland/rev/92bb542ff11a Do not assume the video covers the entire screen. r=Gijs

The failure rate on m-c has gone to zero, so the fix worked!

Status: ASSIGNED → RESOLVED
Closed: 5 years ago
status-firefox69: --- → wontfix
status-firefox70: --- → fixed
status-firefox-esr60: --- → wontfix
status-firefox-esr68: --- → wontfix
status-thunderbird_esr60: --- → wontfix
status-thunderbird_esr68: --- → wontfix
Keywords: leave-open
OS: Unspecified → All
Hardware: Unspecified → All
Resolution: --- → FIXED
Target Milestone: --- → mozilla70
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: