Closed
Bug 1197022
Opened 10 years ago
Closed 10 years ago
[EME] Disable dormant mode for EME videos on all channels
Categories
(Core :: Audio/Video: Playback, defect)
Core
Audio/Video: Playback
Tracking
()
RESOLVED
FIXED
mozilla43
People
(Reporter: cpearce, Assigned: cpearce)
References
Details
Attachments
(1 file)
968 bytes,
patch
|
ajones
:
review+
Sylvestre
:
approval-mozilla-aurora+
|
Details | Diff | Splinter Review |
We don't really need dormant mode for EME, since EME doesn't use hardware decoders. Long term, we'll handle resource contention differently, so we can just disable dormant mode for EME on all channels, rather than just on Beta/Release.
Assignee | ||
Comment 1•10 years ago
|
||
Updated•10 years ago
|
Attachment #8650794 -
Flags: review?(ajones) → review+
Comment 3•10 years ago
|
||
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
status-firefox43:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla43
Assignee | ||
Comment 4•10 years ago
|
||
Comment on attachment 8650794 [details] [diff] [review]
Patch: Disable dormant for EME video
Approval Request Comment
[Feature/regressing bug #]: EME
[User impact if declined]: If the user leaves a netflix tab paused in the background for 60 seconds in an aurora build, netflix will fail and be unable to be started again when the tab is re-focused.
[Describe test coverage new/current, TreeHerder]: We have lots of EME mochitests.
[Risks and why]: Very low. This patch makes us take the path we take in Release in Aurora and Nightly too. That is, this patch disables the feature that causes the failure, and that feature is already disabled in Release.
[String/UUID change made/needed]: None.
Attachment #8650794 -
Flags: approval-mozilla-aurora?
Updated•10 years ago
|
status-firefox42:
--- → affected
Comment 5•10 years ago
|
||
Comment on attachment 8650794 [details] [diff] [review]
Patch: Disable dormant for EME video
Sure, let's take it for 42.
Attachment #8650794 -
Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
Comment 6•10 years ago
|
||
You need to log in
before you can comment on or make changes to this bug.
Description
•