Test test_getUserMedia_playAudioTwice.html times out on Autophone android 6 debug

RESOLVED FIXED in Firefox 50

Status

()

P3
normal
RESOLVED FIXED
2 years ago
2 years ago

People

(Reporter: dminor, Assigned: esawin)

Tracking

unspecified
mozilla50
ARM
Android
Points:
---

Firefox Tracking Flags

(firefox50 fixed)

Details

Attachments

(1 attachment)

(Reporter)

Description

2 years ago
From the log:

68 INFO TEST-UNEXPECTED-FAIL | dom/media/tests/mochitest/test_getUserMedia_playAudioTwice.html | Error executing test: Error: verifyPlaying timed out timeout/<@http://mochi.test:8888/tests/dom/media/tests/mochitest/head.js:462:63 ...  
    runTestWhenReady/<@dom/media/tests/mochitest/head.js:351:7
69 ERROR [SimpleTest.finish()] this test already called finish!
70 INFO TEST-UNEXPECTED-ERROR | dom/media/tests/mochitest/test_getUserMedia_playAudioTwice.html | called finish() multiple times

First bad: https://treeherder.mozilla.org/#/jobs?repo=mozilla-central&revision=34a8be4346a9231e472fc36b1d7c0531e0fbf7c5&filter-tier=1&filter-tier=2&filter-tier=3

Last good: https://treeherder.mozilla.org/#/jobs?repo=mozilla-central&revision=7c0dce574b3f2d5e8f6e4e60caab5532f8e13fd6&filter-tier=1&filter-tier=2&filter-tier=3
(Assignee)

Comment 1

2 years ago
"Media element should be playing" is misleading (which results in the failure message "Media element should be playing - got true, expected false"), it should be "Media element should not be paused".

On Android API level 15, we set "media.autoplay.enabled" to false by default, which needs to be overridden in the test/test harness.

[1] https://dxr.mozilla.org/mozilla-central/source/mobile/android/chrome/content/browser.js#487
(Assignee)

Comment 2

2 years ago
Created attachment 8763211 [details] [diff] [review]
0001-Bug-1280209-1.1-Enabled-media-autoplay-for-Android-m.patch

Who would be the right build peer to r? for that?
Attachment #8763211 - Flags: review?(snorp)
(Assignee)

Updated

2 years ago
Assignee: nobody → esawin
Attachment #8763211 - Flags: review?(snorp) → review+

Comment 4

2 years ago
Pushed by esawin@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/ba16194f3602
[1.1] Enabled media autoplay for Android mochitests. r=snorp

Comment 5

2 years ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/ba16194f3602
Status: NEW → RESOLVED
Last Resolved: 2 years ago
status-firefox50: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla50
You need to log in before you can comment on or make changes to this bug.