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

Unified Diff: LayoutTests/media/track/track-remove-track.html

Issue 83073006: Add "change" and "removetrack" events to TextTrackList. (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@add-track-id-again
Patch Set: Fix test results for Opera onremovetrack test Created 7 years, 1 month 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: LayoutTests/media/track/track-remove-track.html
diff --git a/LayoutTests/media/track/track-add-track.html b/LayoutTests/media/track/track-remove-track.html
similarity index 52%
copy from LayoutTests/media/track/track-add-track.html
copy to LayoutTests/media/track/track-remove-track.html
index a01cc3c9ab2234638adc7a245590191845c531ad..49c28ad765cdaba753d00ce1a52c404618a4fea7 100644
--- a/LayoutTests/media/track/track-add-track.html
+++ b/LayoutTests/media/track/track-remove-track.html
@@ -7,24 +7,19 @@
<script src=../video-test.js></script>
<script>
- var tracks = [];
var trackElement;
- function trackAdded()
+ var track;
+
+ function trackRemoved()
{
consoleWrite("EVENT(" + event.type + ")");
testExpected("event.target", video.textTracks);
testExpected("event instanceof window.TrackEvent", true);
- testExpected("event.track", tracks[video.textTracks.length - 1]);
-
- if (video.textTracks.length == 1) {
- consoleWrite("<br>** Test a TextTrack created with video.addTextTrack().");
- run("tracks.push(video.addTextTrack('captions', 'Caption Track', 'en'))");
- testExpected("video.textTracks.length", 2);
- } else {
- consoleWrite("");
- endTest();
- }
+ testExpected("event.track", track);
+
+ consoleWrite("");
+ endTest();
}
function start()
@@ -34,22 +29,21 @@
consoleWrite("** Test a TextTrack created for a &lt;track&gt; element.");
run("trackElement = document.createElement('track')");
run("video.appendChild(trackElement)");
- tracks.push(trackElement.track);
-
- // Register the 'addtrack' listener after creating the element to make sure the
- // event is dispatch asynchronously.
- video.textTracks.addEventListener("addtrack", trackAdded);
run("trackElement.src = 'captions-webvtt/tc004-webvtt-file.vtt'");
run("trackElement.track.mode = 'hidden'");
testExpected("video.textTracks.length", 1);
- testExpected("trackElement.readyState", HTMLTrackElement.NONE);
+
+ track = video.textTracks[0];
+ video.removeChild(trackElement);
+
+ video.textTracks.addEventListener("removetrack", trackRemoved);
}
</script>
</head>
<body onload="start()">
- <p>Tests that the 'addtrack' event is fired when a TextTrack is created.</p>
+ <p>Tests that the 'removetrack' event is fired when a TextTrack is removed.</p>
<video>
</video>
</body>
« no previous file with comments | « LayoutTests/media/track/track-change-event-expected.txt ('k') | LayoutTests/media/track/track-remove-track-expected.txt » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698