Closed
Bug 1441424
Opened 7 years ago
Closed 7 years ago
dom/media/test/test_autoplay_policy_activation.html fails after bug 1193394
Categories
(Core :: Audio/Video: Playback, defect, P2)
Core
Audio/Video: Playback
Tracking
()
RESOLVED
FIXED
mozilla60
| Tracking | Status | |
|---|---|---|
| firefox60 | --- | fixed |
People
(Reporter: arai, Assigned: cpearce)
References
Details
Attachments
(2 files)
https://treeherder.mozilla.org/logviewer.html#?job_id=164510986&repo=try&lineNumber=2356
> INFO - TEST-UNEXPECTED-FAIL | dom/media/test/test_autoplay_policy_activation.html | audible playback in activated cross-origin iframe allowed - got false, expected true
> INFO - SimpleTest.is@SimpleTest/SimpleTest.js:312:5
> INFO - runTest@dom/media/test/test_autoplay_policy_activation.html:147:13
> INFO - async*@dom/media/test/test_autoplay_policy_activation.html:21:11
|
||
Comment 1•7 years ago
|
||
I should have added :cpearce.
|
|
||
Comment 2•7 years ago
|
||
:cpearce, can you please take a look at the failure in the try in comment 0? We will land bug 1193394 soon (hopefully in this week).
Flags: needinfo?(cpearce)
|
Reporter | |
Comment 3•7 years ago
|
||
here's a patch to disable the test temporarily
Attachment #8954997 -
Flags: review?(hikezoe)
|
|
Reporter | |
Updated•7 years ago
|
Keywords: leave-open
|
|
||
Updated•7 years ago
|
Attachment #8954997 -
Flags: review?(hikezoe) → review+
| Comment hidden (mozreview-request) |
|
|
Reporter | |
Comment 5•7 years ago
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/95f239098c7fedcbc92a86b5b470e651eaffac39
Bug 1441424 - Disable test_autoplay_policy_activation.html temporarily. r=hiro
|
||
Comment 6•7 years ago
|
||
| bugherder | ||
Comment on attachment 8955009 [details]
Bug 1441424 - Fix autoplay activation test.
https://reviewboard.mozilla.org/r/224188/#review230506
Attachment #8955009 -
Flags: review?(dglastonbury) → review+
Pushed by cpearce@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/f8688efe4824
Fix autoplay activation test. r=kamidphish
|
||
Comment 9•7 years ago
|
||
| bugherder | ||
|
|
Reporter | |
Updated•7 years ago
|
|
||
Comment 10•7 years ago
|
||
The fix re-enabled the test so looks like we can close this. Thanks for the report!
Assignee: nobody → cpearce
Status: NEW → RESOLVED
Closed: 7 years ago
Priority: -- → P2
Resolution: --- → FIXED
|
|
||
Comment 11•7 years ago
|
||
\o/
Clearing NI to :cpearce. Thanks for the lovely fix!
Flags: needinfo?(cpearce)
You need to log in
before you can comment on or make changes to this bug.
Description
•