Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(10271)

Unified Diff: content/test/data/media/encrypted_media_utils.js

Issue 11583013: Ensure keymessage response to generateKeyRequest() has keySystem set for external CDMs. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: rebase Created 8 years ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | webkit/media/crypto/ppapi/cdm_wrapper.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: content/test/data/media/encrypted_media_utils.js
diff --git a/content/test/data/media/encrypted_media_utils.js b/content/test/data/media/encrypted_media_utils.js
index 9bde89faec130ec30c581845137c7c8bd63a42f8..c10b11c07ff12b88d14c2d3a2a58d2bc503e70fd 100644
--- a/content/test/data/media/encrypted_media_utils.js
+++ b/content/test/data/media/encrypted_media_utils.js
@@ -117,8 +117,7 @@ function loadEncryptedMedia(video, mediaFile, keySystem, key) {
}
function onKeyMessage(e) {
- // TODO(ddorwin): Enable after fixing http://crbug.com/166204.
- if (!e.keySystem && false) {
+ if (!e.keySystem) {
failTest('keymessage without a keySystem: ' + e.keySystem);
return;
}
« no previous file with comments | « no previous file | webkit/media/crypto/ppapi/cdm_wrapper.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698