EME: Require codecs parameter for audio/videoCapabilities

third_party/WebKit/LayoutTests/media/encrypted-media/encrypted-media-requestmediakeysystemaccess.html

Index: third_party/WebKit/LayoutTests/media/encrypted-media/encrypted-media-requestmediakeysystemaccess.html
diff --git a/third_party/WebKit/LayoutTests/media/encrypted-media/encrypted-media-requestmediakeysystemaccess.html b/third_party/WebKit/LayoutTests/media/encrypted-media/encrypted-media-requestmediakeysystemaccess.html
index 9a3fca187108ed60c05be564cefeeba0779def44..150fbf71a4561a5dbf954dfb645797c771ef26a6 100644
--- a/third_party/WebKit/LayoutTests/media/encrypted-media/encrypted-media-requestmediakeysystemaccess.html
+++ b/third_party/WebKit/LayoutTests/media/encrypted-media/encrypted-media-requestmediakeysystemaccess.html
@@ -80,29 +80,44 @@
             expect_error('org.w3.clearkey', [], 'TypeError', 'Empty supportedConfigurations');
             expect_error('org.w3.clearkey', [{}], 'NotSupportedError', 'Empty configuration');
 
-            // Various combinations of supportedConfigurations.
-            // TODO(jrummell): Specifying contentType without codecs is
-            // deprecated, so this test should fail. http://crbug.com/605661.
-            expect_config('org.w3.clearkey', [{
+            // contentType without codecs is not supported.
+            expect_error('org.w3.clearkey', [{
                 initDataTypes: ['webm'],
                 audioCapabilities: [{contentType: 'audio/webm'}],
                 videoCapabilities: [{contentType: 'video/webm'}],
+            }], 'NotSupportedError', 'Basic supported configuration without codecs');

[xhwang, 2017/02/09 20:07:55]

Just in case, add a test where codecs for only audio or video is missing.

[jrummell, 2017/02/09 23:36:08]

Done. Added both.

+
+            expect_error('org.w3.clearkey', [{
+                initDataTypes: ['fakeidt', 'webm'],
+                audioCapabilities: [{contentType: 'audio/fake'}, {contentType: 'audio/webm'}],
+                videoCapabilities: [{contentType: 'video/fake'}, {contentType: 'video/webm'}],
+            }], 'NotSupportedError', 'Partially supported configuration without codecs');
+
+            // Various combinations of supportedConfigurations.
+            expect_config('org.w3.clearkey', [{
+                initDataTypes: ['webm'],
+                audioCapabilities: [{contentType: 'audio/webm; codecs=vorbis'}],
+                videoCapabilities: [{contentType: 'video/webm; codecs=vp8'}],
             }], {
                 initDataTypes: ['webm'],
-                audioCapabilities: [{contentType: 'audio/webm'}],
-                videoCapabilities: [{contentType: 'video/webm'}],
+                audioCapabilities: [{contentType: 'audio/webm; codecs=vorbis'}],
+                videoCapabilities: [{contentType: 'video/webm; codecs=vp8'}],
             }, 'Basic supported configuration');
 
-            // TODO(jrummell): Specifying contentType without codecs is
-            // deprecated, so this test should fail. http://crbug.com/605661.
             expect_config('org.w3.clearkey', [{
                 initDataTypes: ['fakeidt', 'webm'],
-                audioCapabilities: [{contentType: 'audio/fake'}, {contentType: 'audio/webm'}],
-                videoCapabilities: [{contentType: 'video/fake'}, {contentType: 'video/webm'}],
+                audioCapabilities: [
+                    {contentType: 'audio/fake; codecs=vorbis'},
+                    {contentType: 'audio/webm; codecs=vorbis'}
+                ],
+                videoCapabilities: [
+                    {contentType: 'video/fake; codecs=vp8'},
+                    {contentType: 'video/webm; codecs=vp8'}
+                ],
             }], {
                 initDataTypes: ['webm'],
-                audioCapabilities: [{contentType: 'audio/webm'}],
-                videoCapabilities: [{contentType: 'video/webm'}],
+                audioCapabilities: [{contentType: 'audio/webm; codecs=vorbis'}],
+                videoCapabilities: [{contentType: 'video/webm; codecs=vp8'}],
             }, 'Partially supported configuration');
 
             expect_config('org.w3.clearkey', [{