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

Unified Diff: third_party/WebKit/LayoutTests/media/encrypted-media/encrypted-media-syntax.html

Issue 2354083003: media: Change MediaKeyMessageEventInit to require both attributes (Closed)
Patch Set: update media-key-message-event-constructor test Created 4 years, 3 months 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
Index: third_party/WebKit/LayoutTests/media/encrypted-media/encrypted-media-syntax.html
diff --git a/third_party/WebKit/LayoutTests/media/encrypted-media/encrypted-media-syntax.html b/third_party/WebKit/LayoutTests/media/encrypted-media/encrypted-media-syntax.html
index fb1e83aebac60d91b984f3c5d14db3e5908c47c3..201ee320c951fd81bdaa62695d5b7c24cf7caf35 100644
--- a/third_party/WebKit/LayoutTests/media/encrypted-media/encrypted-media-syntax.html
+++ b/third_party/WebKit/LayoutTests/media/encrypted-media/encrypted-media-syntax.html
@@ -1001,6 +1001,34 @@
});
}, 'Test MediaKeys setServerCertificate().');
+ function test_valid_message_type(message_type)
+ {
+ var event = new MediaKeyMessageEvent('eventType', { messageType: message_type, message: new ArrayBuffer(0) } );
+ assert_equals(event.messageType, message_type);
+ }
+
+ test(function(test)
+ {
+ // Valid MediaKeyMessageType values.
+ test_valid_message_type('license-request');
+ test_valid_message_type('license-renewal');
+ test_valid_message_type('license-release');
+ // TODO(jrummell): Add 'individualization-request' if the final
+ // spec includes it. http://crbug.com/628437.
+
+ // Invalid MediaKeyMessageType values should throw a TypeError.
+ assert_throws(new TypeError, function() {
+ new MediaKeyMessageEvent('eventType', { messageType: 'something-else', message: new ArrayBuffer(0) } );
+ });
+
+ // Missing required values should throw a TypeError.
+ assert_throws(new TypeError, function() {
+ new MediaKeyMessageEvent('eventType', { message: new ArrayBuffer(0) } );
+ });
+ assert_throws(new TypeError, function() {
+ new MediaKeyMessageEvent('eventType', { messageType: 'license-request' } );
+ });
+ }, 'Test MediaKeyMessageEvent.');
// FIXME: Add syntax checks for MediaKeys.IsTypeSupported().
// FIXME: Add syntax checks for MediaKeyError and MediaKeySession events.

Powered by Google App Engine
This is Rietveld 408576698