Commit 487f2f57 authored by sandersd's avatar sandersd Committed by Commit bot

Remove support for contentType from player_utils.js.

BUG=224786

Review URL: https://codereview.chromium.org/611383004

Cr-Commit-Position: refs/heads/master@{#300298}
parent 105144f9
...@@ -15,7 +15,6 @@ PlayerUtils.registerDefaultEventListeners = function(player) { ...@@ -15,7 +15,6 @@ PlayerUtils.registerDefaultEventListeners = function(player) {
// Map from event name to event listener function name. It is common for // Map from event name to event listener function name. It is common for
// event listeners to be named onEventName. // event listeners to be named onEventName.
var eventListenerMap = { var eventListenerMap = {
'needkey': 'onNeedKey',
'encrypted': 'onEncrypted', 'encrypted': 'onEncrypted',
'webkitneedkey': 'onWebkitNeedKey', 'webkitneedkey': 'onWebkitNeedKey',
'webkitkeymessage': 'onWebkitKeyMessage', 'webkitkeymessage': 'onWebkitKeyMessage',
...@@ -38,7 +37,7 @@ PlayerUtils.registerDefaultEventListeners = function(player) { ...@@ -38,7 +37,7 @@ PlayerUtils.registerDefaultEventListeners = function(player) {
}; };
PlayerUtils.registerEMEEventListeners = function(player) { PlayerUtils.registerEMEEventListeners = function(player) {
var encrypted_handler = function(message) { player.video.addEventListener('encrypted', function(message) {
function addMediaKeySessionListeners(mediaKeySession) { function addMediaKeySessionListeners(mediaKeySession) {
mediaKeySession.addEventListener('message', function(message) { mediaKeySession.addEventListener('message', function(message) {
...@@ -54,11 +53,8 @@ PlayerUtils.registerEMEEventListeners = function(player) { ...@@ -54,11 +53,8 @@ PlayerUtils.registerEMEEventListeners = function(player) {
}); });
} }
// TODO(sandersd): Stop checking contentType once we complete the switch to
// using the 'encrypted' event.
var init_data_type = message.initDataType || message.contentType;
Utils.timeLog('Creating new media key session for initDataType: ' + Utils.timeLog('Creating new media key session for initDataType: ' +
init_data_type + ', initData: ' + message.initDataType + ', initData: ' +
Utils.getHexString(new Uint8Array(message.initData))); Utils.getHexString(new Uint8Array(message.initData)));
try { try {
if (message.target.mediaKeys.createSession.length == 0) { if (message.target.mediaKeys.createSession.length == 0) {
...@@ -66,13 +62,13 @@ PlayerUtils.registerEMEEventListeners = function(player) { ...@@ -66,13 +62,13 @@ PlayerUtils.registerEMEEventListeners = function(player) {
// uses the new API. // uses the new API.
var session = message.target.mediaKeys.createSession(); var session = message.target.mediaKeys.createSession();
addMediaKeySessionListeners(session); addMediaKeySessionListeners(session);
session.generateRequest(init_data_type, message.initData) session.generateRequest(message.initDataType, message.initData)
.catch(function(error) { .catch(function(error) {
Utils.failTest(error, KEY_ERROR); Utils.failTest(error, KEY_ERROR);
}); });
} else { } else {
var session = message.target.mediaKeys.createSession( var session = message.target.mediaKeys.createSession(
init_data_type, message.initData); message.initDataType, message.initData);
session.then(addMediaKeySessionListeners) session.then(addMediaKeySessionListeners)
.catch(function(error) { .catch(function(error) {
Utils.failTest(error, KEY_ERROR); Utils.failTest(error, KEY_ERROR);
...@@ -81,12 +77,7 @@ PlayerUtils.registerEMEEventListeners = function(player) { ...@@ -81,12 +77,7 @@ PlayerUtils.registerEMEEventListeners = function(player) {
} catch (e) { } catch (e) {
Utils.failTest(e); Utils.failTest(e);
} }
} });
// TODO(sandersd): Stop registering 'needkey' after it is renamed to
// 'encrypted'.
player.video.addEventListener('encrypted', encrypted_handler);
player.video.addEventListener('needkey', encrypted_handler);
this.registerDefaultEventListeners(player); this.registerDefaultEventListeners(player);
try { try {
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment