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

Unified Diff: media/test/data/eme_player_js/player_utils.js

Issue 1712903002: Remove prefixed EME. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Fix isRenewalMessage() in browser tests. Created 4 years, 10 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
« no previous file with comments | « media/test/data/eme_player_js/globals.js ('k') | media/test/data/eme_player_js/prefixed_clearkey_player.js » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: media/test/data/eme_player_js/player_utils.js
diff --git a/media/test/data/eme_player_js/player_utils.js b/media/test/data/eme_player_js/player_utils.js
index 81332caaee1941b6122815090e7b6ce5b3e633b6..85a3c967bd6f4c85d4f149ea81cc79a030bbd8b0 100644
--- a/media/test/data/eme_player_js/player_utils.js
+++ b/media/test/data/eme_player_js/player_utils.js
@@ -38,8 +38,8 @@ PlayerUtils.registerDefaultEventListeners = function(player) {
});
};
-// Register the necessary event handlers needed when playing encrypted content
-// using the unprefixed API. Returns a promise that resolves to the player.
+// Register the necessary event handlers needed when playing encrypted content.
+// Returns a promise that resolves to the player.
PlayerUtils.registerEMEEventListeners = function(player) {
player.video.addEventListener('encrypted', function(message) {
@@ -119,59 +119,6 @@ PlayerUtils.registerEMEEventListeners = function(player) {
.catch(function(error) { Utils.failTest(error, NOTSUPPORTEDERROR); });
};
-// Register the necessary event handlers needed when playing encrypted content
-// using the prefixed API. Even though the prefixed API is all synchronous,
-// returns a promise that resolves to the player.
-PlayerUtils.registerPrefixedEMEEventListeners = function(player) {
- player.video.addEventListener('webkitneedkey', function(message) {
- var initData = message.initData;
- if (player.testConfig.sessionToLoad) {
- Utils.timeLog('Loading session: ' + player.testConfig.sessionToLoad);
- initData =
- Utils.convertToUint8Array(PREFIXED_EME_API_LOAD_SESSION_HEADER +
- player.testConfig.sessionToLoad);
- }
- Utils.timeLog(player.testConfig.keySystem +
- ' Generate key request, initData: ' +
- Utils.getHexString(initData));
- try {
- message.target.webkitGenerateKeyRequest(player.testConfig.keySystem,
- initData);
- } catch (e) {
- Utils.failTest(e);
- }
- });
-
- player.video.addEventListener('webkitkeyadded', function(message) {
- Utils.timeLog('onWebkitKeyAdded', message);
- message.target.receivedKeyAdded = true;
- });
-
- player.video.addEventListener('webkitkeyerror', function(error) {
- Utils.timeLog('onWebkitKeyError',
- 'KeySystem: ' + error.keySystem + ', sessionId: ' +
- error.sessionId + ', errorCode: ' + error.errorCode.code +
- ', systemCode: ' + error.systemCode);
- Utils.failTest(error, PREFIXED_EME_ERROR_EVENT);
- });
-
- player.video.addEventListener('webkitkeymessage', function(message) {
- Utils.timeLog('onWebkitKeyMessage', message);
- message.target.receivedKeyMessage = true;
- if (Utils.isRenewalMessagePrefixed(message.message)) {
- Utils.timeLog('onWebkitKeyMessage - renewal', message);
- message.target.receivedRenewalMessage = true;
- }
- });
-
- // The prefixed API is all synchronous, so wrap the calls in a promise.
- return new Promise(function(resolve, reject) {
- PlayerUtils.registerDefaultEventListeners(player);
- player.video.receivedKeyMessage = false;
- resolve(player);
- });
-};
-
PlayerUtils.setVideoSource = function(player) {
if (player.testConfig.useMSE) {
Utils.timeLog('Loading media using MSE.');
@@ -197,31 +144,17 @@ PlayerUtils.initEMEPlayer = function(player) {
// Return the appropriate player based on test configuration.
PlayerUtils.createPlayer = function(video, testConfig) {
- // Update keySystem if using prefixed Clear Key since it is not available as a
- // separate key system to choose from; however it can be set in URL query.
- var usePrefixedEME = testConfig.usePrefixedEME;
- if (testConfig.keySystem == CLEARKEY && usePrefixedEME)
- testConfig.keySystem = PREFIXED_CLEARKEY;
-
function getPlayerType(keySystem) {
switch (keySystem) {
case WIDEVINE_KEYSYSTEM:
- if (usePrefixedEME)
- return PrefixedWidevinePlayer;
return WidevinePlayer;
- case PREFIXED_CLEARKEY:
- return PrefixedClearKeyPlayer;
case EXTERNAL_CLEARKEY:
case CLEARKEY:
- if (usePrefixedEME)
- return PrefixedClearKeyPlayer;
return ClearKeyPlayer;
case FILE_IO_TEST_KEYSYSTEM:
return FileIOTestPlayer;
default:
Utils.timeLog(keySystem + ' is not a known key system');
- if (usePrefixedEME)
- return PrefixedClearKeyPlayer;
return ClearKeyPlayer;
}
}
« no previous file with comments | « media/test/data/eme_player_js/globals.js ('k') | media/test/data/eme_player_js/prefixed_clearkey_player.js » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698