MediaKeySession: Resolve the promise returned by close() after resolving the closed promise attribute |
||||
Issue descriptionPer https://w3c.github.io/encrypted-media/#close, the Session Closed algorithm, which resolves the MediaKeySession's `closed` attribute is to be run synchronously before resolving the promise returned by the `close()` method. See https://github.com/w3c/web-platform-tests/issues/4058 for more details. Currently, Chrome runs these steps in the opposite order, resulting in the promise resolve handlers running in the opposite order (and thus passing the incorrect test identified in the web-platform-tests issue above).
,
Oct 25 2016
,
Oct 26 2016
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/253ba303fba53bc0fa7fb1132da09fb2e3cd7dad commit 253ba303fba53bc0fa7fb1132da09fb2e3cd7dad Author: ddorwin <ddorwin@chromium.org> Date: Wed Oct 26 18:35:02 2016 EME Clear Key: Run Session Closed algorithm before resolving returned promise BUG=659374 TEST= https://www.w3c-test.org/encrypted-media/clearkey-mp4-playback-temporary-events.html after fix for https://github.com/w3c/web-platform-tests/issues/4058 Review-Url: https://codereview.chromium.org/2446993003 Cr-Commit-Position: refs/heads/master@{#427755} [modify] https://crrev.com/253ba303fba53bc0fa7fb1132da09fb2e3cd7dad/media/cdm/aes_decryptor.cc [modify] https://crrev.com/253ba303fba53bc0fa7fb1132da09fb2e3cd7dad/third_party/WebKit/Source/modules/encryptedmedia/MediaKeySession.cpp
,
Oct 26 2016
Clear Key is fixed and passing the corrected spec test passes. We need to fix Widevine so that https://www.w3c-test.org/encrypted-media/drm-mp4-playback-temporary-events.html will pass.
,
Oct 26 2016
This should also be done for remove().
,
Oct 26 2016
#5 is a separate bug.
,
Oct 26 2016
|
||||
►
Sign in to add a comment |
||||
Comment 1 by ddorwin@chromium.org
, Oct 25 2016