[EME] Disable dormant mode for EME videos on all channels

RESOLVED FIXED in Firefox 42

Status

()

defect
RESOLVED FIXED
4 years ago
4 years ago

People

(Reporter: cpearce, Assigned: cpearce)

Tracking

(Blocks 1 bug)

unspecified
mozilla43
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox42 fixed, firefox43 fixed)

Details

Attachments

(1 attachment)

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: nobody → cpearce
Status: NEW → ASSIGNED
Attachment #8650794 - Flags: review?(ajones)
Attachment #8650794 - Flags: review?(ajones) → review+
https://hg.mozilla.org/mozilla-central/rev/4fd3f9fdf6c2
Status: ASSIGNED → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla43
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?
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+
Blocks: 1181864
You need to log in before you can comment on or make changes to this bug.