New issue
Advanced search Search tips

Issue 595898 link

Starred by 2 users

Issue metadata

Status: Fixed
Owner:
Closed: Mar 2016
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: All
Pri: 1
Type: Bug



Sign in to add a comment

EME does not survive Suspend/Resume

Project Member Reported by sande...@chromium.org, Mar 17 2016

Issue description

When Resume() constructs a new decoder, it does not call SetCdm() again, and so EME playback fails.
 

Comment 1 by xhw...@chromium.org, Mar 17 2016

Cc: xhw...@chromium.org
Labels: M-51
Cc: ddorwin@chromium.org
Will resume work on Android and other platforms where the DRM resources might have been reclaimed by the system?

Comment 3 by xhw...@chromium.org, Mar 17 2016

Today when we suspend we only destroy the media Renderer. The media element will stay alive and keep holding a ref to the MediaKeys/CDM.
Labels: -Pri-2 ReleaseBlock-Dev Pri-1
This should block next dev release.
 Issue 595578  has been merged into this issue.
Project Member

Comment 6 by bugdroid1@chromium.org, Mar 18 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/7655364f9ae672fdbd05e5751006387207b0bf51

commit 7655364f9ae672fdbd05e5751006387207b0bf51
Author: sandersd <sandersd@chromium.org>
Date: Fri Mar 18 22:14:35 2016

Restore CDM context on Resume().

Also converts all uses of |weak_factory_| to use a |weak_this_|, as GetWeakPtr() is not threadsafe.

BUG= 595898 

Review URL: https://codereview.chromium.org/1815013002

Cr-Commit-Position: refs/heads/master@{#382109}

[modify] https://crrev.com/7655364f9ae672fdbd05e5751006387207b0bf51/media/base/pipeline_impl.cc
[modify] https://crrev.com/7655364f9ae672fdbd05e5751006387207b0bf51/media/base/pipeline_impl.h

Please remove the blocker label if its resolved.
Labels: -ReleaseBlock-Dev
Status: Fixed (was: Started)

Sign in to add a comment