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

Unified Diff: chrome/test/data/media/eme_player.html

Issue 332323003: Revert of Integrate browser tests with new EME player. (Closed) Base URL: http://git.chromium.org/chromium/src.git@eme_player
Patch Set: Created 6 years, 6 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 | « chrome/browser/media/media_browsertest.cc ('k') | chrome/test/data/media/eme_player_js/app_loader.js » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/test/data/media/eme_player.html
diff --git a/chrome/test/data/media/eme_player.html b/chrome/test/data/media/eme_player.html
index 5d7f640adad6f12e88b735234695d8fccde9d32b..82f12448ee82e69349d8349d3b7200138af8e034 100644
--- a/chrome/test/data/media/eme_player.html
+++ b/chrome/test/data/media/eme_player.html
@@ -6,27 +6,28 @@
<body style='font-family:"Lucida Console", Monaco, monospace; font-size:14px'>
<i>Clearkey works only with content encrypted using bear key.</i><br><br>
<table>
- <tr title='URL param mediaFile=...'>
+ <tr>
<td><label for='mediaFile'>Encrypted video URL:</label></td>
<td><input id='mediaFile' type='text' size='64'></td>
</tr>
- <tr title='URL param licenseServerURL=...'>
+ <tr>
<td><label for='licenseServer'>License sever URL:</label></td>
<td><input id='licenseServer' type='text' size='64'></td>
</tr>
- <tr title='URL param keySystem=...'>
+ <tr>
<td><label for='keySystemList'>Key system:</label></td>
<td><select id='keySystemList'></select></td>
</tr>
- <tr title='URL param mediaType=...'>
+ <tr>
<td><label for='mediaTypeList'>Media type:</label></td>
<td><select id='mediaTypeList'></select></td>
</tr>
- <tr title='URL param usePrefixedEME=1|0'>
+ <tr>
<td><label for='usePrefixedEME'>EME API version:</label></td>
- <td><select id='usePrefixedEME'></select></td>
+ <td><select id='usePrefixedEME'></select>
+ </td>
</tr>
- <tr title='URL param useSRC=1|0'>
+ <tr>
<td><label for='useSRC'>Load media by:</label></td>
<td>
<select id='useSRC'>
@@ -59,37 +60,10 @@
<script src='eme_player_js/app_loader.js' type='text/javascript'></script>
<script type='text/javascript'>
TestConfig.updateDocument();
-
- function onTimeUpdate(e) {
- var video = e.target;
- if (video.currentTime < 1)
- return;
- // For loadSession() tests, addKey() will not be called after
- // loadSession() (the key is loaded internally). Do not check keyadded
- // and heartbeat for these tests.
- if (!TestConfig.sessionToLoad) {
- // keyadded may be fired around the start of playback; check for it
- // after a delay to avoid timing issues.
- if (TestConfig.usePrefixedEME && !video.receivedKeyAdded)
- Utils.failTest('Key added event not received.');
- if (TestConfig.keySystem == EXTERNAL_CLEARKEY &&
- !video.receivedHeartbeat)
- Utils.failTest('Heartbeat keymessage event not received.');
- }
- video.removeEventListener('ended', Utils.failTest);
- Utils.installTitleEventHandler(video, 'ended');
- video.removeEventListener('timeupdate', onTimeUpdate);
- }
-
- function Play() {
+ function Play () {
TestConfig.init();
- var video = TestApp.loadPlayer();
- Utils.resetTitleChange();
- // Ended should not fire before onTimeUpdate.
- video.addEventListener('ended', Utils.failTest);
- video.addEventListener('timeupdate', onTimeUpdate);
- video.play();
- }
+ TestApp.play();
+ };
function toggleDisplay(id) {
var element = document.getElementById(id);
« no previous file with comments | « chrome/browser/media/media_browsertest.cc ('k') | chrome/test/data/media/eme_player_js/app_loader.js » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698