New issue
Advanced search Search tips

Issue 624192 link

Starred by 2 users
Status: Fixed
Owner:
jrumm...@chromium.org
Closed: Oct 2016
Cc:
xhw...@chromium.org
ddorwin@chromium.org
Components:
EstimatedDays: ----
NextAction: ----
OS: All
Pri: 3
Type: Bug



Sign in to add a comment

ClearKey CDM does not update session expiration time

Project Member Reported by jrumm...@chromium.org, Jun 28 2016 
On close() the EME spec states "Run the Update Expiration algorithm on the session, providing NaN." ClearKey should do this to be spec compliant (although it never sets the expiry time, so the expiry time is always NaN).
Project Member

Comment 1 by bugdroid1@chromium.org, Oct 11 2016 

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

commit beab77818bad0932063375097110e6048632a193
Author: jrummell <jrummell@chromium.org>
Date: Tue Oct 11 20:17:22 2016

EME: Implement Session Closed algorithm

Update MediaKeySession to implement the Session Closed algorithm
as outlined in the EME spec. This replaces a previous change that
implemented this as part of Clear Key key system only.

BUG= 622956 , 624192 
TEST=updated test passes

Review-Url: https://codereview.chromium.org/2406073002
Cr-Commit-Position: refs/heads/master@{#424524}

[modify] https://crrev.com/beab77818bad0932063375097110e6048632a193/media/cdm/aes_decryptor.cc
[modify] https://crrev.com/beab77818bad0932063375097110e6048632a193/media/cdm/aes_decryptor_unittest.cc
[modify] https://crrev.com/beab77818bad0932063375097110e6048632a193/third_party/WebKit/LayoutTests/media/encrypted-media/encrypted-media-keystatuses.html
[modify] https://crrev.com/beab77818bad0932063375097110e6048632a193/third_party/WebKit/Source/modules/encryptedmedia/MediaKeySession.cpp

Comment 2 by jrumm...@chromium.org, Oct 11 2016

Cc: -jrumm...@chromium.org
Owner: jrumm...@chromium.org
Status: Fixed (was: Available)
►

Sign in to add a comment