Closed Bug 1197022 Opened 5 years ago Closed 5 years ago

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

Categories

(Core :: Audio/Video: Playback, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla43
Tracking Status
firefox42 --- fixed
firefox43 --- fixed

People

(Reporter: cpearce, Assigned: cpearce)

References

(Blocks 1 open bug)

Details

Attachments

(1 file)

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: 5 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+
You need to log in before you can comment on or make changes to this bug.