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

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

Issue 965683002: Fix TextTrackMode test (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@master
Patch Set: Created 5 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
Index: LayoutTests/media/track/track-mode.html
diff --git a/LayoutTests/media/track/track-mode.html b/LayoutTests/media/track/track-mode.html
index 82e35bdccd51c47cc6b322b145e4305d7f06f9ba..07c0a11c2417e8c460f14e857163ac4a6a0f47ef 100644
--- a/LayoutTests/media/track/track-mode.html
+++ b/LayoutTests/media/track/track-mode.html
@@ -41,6 +41,13 @@
testExpected("video.textTracks[0].mode", "showing");
consoleWrite("");
+ consoleWrite("<b>*** Set to numeric value (no longer supported), should return default</b>");
jsbell 2015/02/27 16:50:35 Missing ) inside the string
jsbell 2015/02/27 16:54:12 Ignore this - the ) is there, it was just slightly
+ var value = "2";
+ run("textTrack.mode = '" + value + "'");
jsbell 2015/02/27 16:50:35 Since this is testing numeric values, this should
Paritosh Kumar 2015/03/02 04:39:35 Done. Thanks Jsbell.
+ testExpected("textTrack.mode", "showing");
+ testExpected("video.textTracks[0].mode", "showing");
+ consoleWrite("");
+
consoleWrite("<b>*** Set to known values</b>");
consoleWrite("<b>++ 'disabled'</b>");
setMode("disabled");
« no previous file with comments | « LayoutTests/media/track/track-add-remove-cue-expected.txt ('k') | LayoutTests/media/track/track-mode-expected.txt » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698