[EME] Video playback fails when page with EME video is moved into new window

RESOLVED FIXED in Firefox 53

Status

()

defect
P1
normal
RESOLVED FIXED
3 years ago
2 years ago

People

(Reporter: ciprian_georgiu, Assigned: bryce, Mentored)

Tracking

Trunk
mozilla53
Points:
---

Firefox Tracking Flags

(firefox48 wontfix, firefox49 wontfix, firefox50 wontfix, firefox51 wontfix, firefox52 wontfix, firefox53 fixed)

Details

Attachments

(3 attachments)

Posted image shaka player bug.gif
[Note]:
- .gif showing the issue

[Affected versions]:
- latest Aurora 50.0a2

[Affected platforms]:
- Windows 10 x86, x64
- Windows 7 x64
- Mac OS X 10.9 (Mavericks)
- Ubuntu 16.04 x64

[Steps to reproduce]:
1. Start Firefox.
2. Go to: http://shaka-player-demo.appspot.com/demo/.
3. From drop-down Asset menu select 'Angel One (multicodec, multilingual, Widevine)'.
4. Click Load.
5. Move the current window (Tab) into New window.

[Expected result]:
- The video continue to play properly in the New window.

[Actual result]:
- The video displays the following error into New window: Shaka Error MEDIA.MEDIA_SOURCE_OPERATION_THREW (InvalidStateError: An attempt was made to use an object that is not, or is no longer, usable)

[Regression range]:
- This is not a regression.
Assignee

Comment 1

3 years ago
Tested this and it appears to not be WebM specific. I can reproduce this on all released versions. Same steps can be used to reproduce with mp4 or webm media source. Updating bug to reflect broader impact.
Summary: [WebM Widevine] Video is crashing on Shaka Player Demo page when is moved into New window → [EME] Video is crashing on Shaka Player Demo page when is moved into New window
Version: 50 Branch → Trunk
I can repro this on mse-clearkey. NSPR logs show that the CDM is doing a normal shutdown for some reason.
Assignee

Updated

3 years ago
Duplicate of this bug: 1298030
Assignee

Comment 4

3 years ago
Updating the name of this bug to reflect that this issue is universal to EME videos, not just those for Shaka.
Summary: [EME] Video is crashing on Shaka Player Demo page when is moved into New window → [EME] Video playback fails when page with EME video is moved into new window
Comment hidden (mozreview-request)

Comment 6

3 years ago
mozreview-review
Comment on attachment 8805871 [details]
Bug 1298027 - Relax decoder shutdown and mediakey removal on MediaElement suspension.

https://reviewboard.mozilla.org/r/89488/#review88742
Attachment #8805871 - Flags: review?(cpearce) → review+

Comment 7

3 years ago
Pushed by bvandyk@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/fc6a63bedaec
Relax decoder shutdown and mediakey removal on MediaElement suspension. r=cpearce

Comment 8

3 years ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/fc6a63bedaec
Status: NEW → RESOLVED
Last Resolved: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla52

Comment 9

3 years ago
Backout by archaeopteryx@coole-files.de:
https://hg.mozilla.org/mozilla-central/rev/37ab1d54a08e
Backed out changeset fc6a63bedaec for timing out in test_eme_stream_capture_blocked_case3.html on Windows 7 opt and pgo. r=backout a=backout
Backed this out for timing out in test_eme_stream_capture_blocked_case3.html on Windows 7 opt and pgo:

https://hg.mozilla.org/mozilla-central/rev/37ab1d54a08e7e1431660b22377428b74dcd090a

Push with failures: https://treeherder.mozilla.org/#/jobs?repo=autoland&revision=fc6a63bedaec6e34bb3b2491b497b4f131387dab
Failure log: https://treeherder.mozilla.org/logviewer.html#?job_id=5909068&repo=autoland

22:08:19     INFO - [22:03:06.745] 400x300 as-is then 400x300 presented as 533x300-14_case3 stalled
22:08:19     INFO - [22:03:06.856] 400x225-15_case3 stalled
22:08:19     INFO - TEST-UNEXPECTED-FAIL | dom/media/test/test_eme_stream_capture_blocked_case3.html | Test timed out. 
22:08:19     INFO -     reportError@SimpleTest/TestRunner.js:114:7
22:08:19     INFO -     TestRunner._checkForHangs@SimpleTest/TestRunner.js:135:7
22:08:19     INFO -     setTimeout handler*TestRunner._checkForHangs@SimpleTest/TestRunner.js:156:5
Flags: needinfo?(bvandyk)
Should this have been re-opened ?
Assignee

Comment 13

3 years ago
Yes. I'll reopen while I investigate the test failures.
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Assignee

Updated

3 years ago
Flags: needinfo?(bvandyk)
Assignee

Comment 14

3 years ago
Whoops didn't mean to clear that need info. Will hold it while I look at tests.
Flags: needinfo?(bvandyk)
Comment hidden (mozreview-request)
Comment hidden (mozreview-request)

Comment 19

3 years ago
mozreview-review
Comment on attachment 8810270 [details]
Bug 1298027 - Shut down CDMs when a containing document becomes inactive.

https://reviewboard.mozilla.org/r/92628/#review92628
Attachment #8810270 - Flags: review?(cpearce) → review+

Comment 20

3 years ago
Pushed by bvandyk@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/f3930acd16b7
Relax decoder shutdown and mediakey removal on MediaElement suspension. r=cpearce
https://hg.mozilla.org/integration/autoland/rev/0f22052f5c48
Shut down CDMs when a containing document becomes inactive. r=cpearce

Comment 21

3 years ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/f3930acd16b7
https://hg.mozilla.org/mozilla-central/rev/0f22052f5c48
Status: REOPENED → RESOLVED
Last Resolved: 3 years ago3 years ago
Resolution: --- → FIXED
Assignee

Updated

3 years ago
Flags: needinfo?(bvandyk)
Depends on: 1317290
No longer depends on: 1317290
Target Milestone: mozilla52 → mozilla53
You need to log in before you can comment on or make changes to this bug.