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

Issues Search
    My Issues | Starred     Open | Closed | All

Unified Diff: third_party/WebKit/LayoutTests/media/track/track-css-user-settings-override-author-settings.html

Issue 1882583002: Convert track tests from video-test.js to testharness.js based (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: address review comments Created 4 years, 8 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 | « third_party/WebKit/LayoutTests/media/track/track-css-property-whitelist-expected.txt ('k') | third_party/WebKit/LayoutTests/media/track/track-css-user-settings-override-author-settings-expected.txt » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: third_party/WebKit/LayoutTests/media/track/track-css-user-settings-override-author-settings.html
diff --git a/third_party/WebKit/LayoutTests/media/track/track-css-user-settings-override-author-settings.html b/third_party/WebKit/LayoutTests/media/track/track-css-user-settings-override-author-settings.html
index 206200a5b33562396afc9d7959e029cd0a17da81..413df4e5c828757358cf1aeabe18cf499b2cc79c 100644
--- a/third_party/WebKit/LayoutTests/media/track/track-css-user-settings-override-author-settings.html
+++ b/third_party/WebKit/LayoutTests/media/track/track-css-user-settings-override-author-settings.html
@@ -1,73 +1,60 @@
<!DOCTYPE html>
-<script src=../media-file.js></script>
-<!-- TODO(philipj): Convert test to testharness.js. crbug.com/588956
- (Please avoid writing new tests using video-test.js) -->
-<script src=../video-test.js></script>
-<script src=../media-controls.js></script>
+<title>Test that WebVTT objects are being styled correctly based on user settings that should override author settings.</title>
+<script src="../media-file.js"></script>
+<script src="../media-controls.js"></script>
+<script src="../../resources/testharness.js"></script>
+<script src="../../resources/testharnessreport.js"></script>
<style>
- /* Author settings for the cues */
- video::cue(c) {
- color: red;
- background-color: green;
- text-shadow: 3px 3px #00ff00;
- font-size: 20px;
- font-family: arial;
- font-style: normal;
- font-variant: normal;
- }
+/* Author settings for the cues */
+video::cue(c) {
+ color: red;
+ background-color: green;
+ text-shadow: 3px 3px #00ff00;
+ font-size: 20px;
+ font-family: arial;
+ font-style: normal;
+ font-variant: normal;
+}
</style>
+<video>
+ <track src="captions-webvtt/styling.vtt" kind="captions" default>
+</video>
<script>
+async_test(function(t) {
+ var video = document.querySelector("video");
+ video.src = findMediaFile("video", "../content/test");
- function applyUserOverrideSettings() {
- if (window.internals) {
- internals.settings.setTextTrackTextColor("cyan");
- internals.settings.setTextTrackBackgroundColor("green");
- internals.settings.setTextTrackTextShadow("2px 2px #ff0000")
- internals.settings.setTextTrackTextSize("14px");
- internals.settings.setTextTrackFontFamily("fantasy");
- internals.settings.setTextTrackFontStyle("italic");
- internals.settings.setTextTrackFontVariant("small-caps");
- }
- }
-
- function verifyAuthorSettings() {
- consoleWrite("");
- cue = textTrackDisplayElement(video, 'cue').firstElementChild;
- testExpected("getComputedStyle(cue).color", "rgb(255, 0, 0)");
- testExpected("getComputedStyle(cue).backgroundColor", "rgb(0, 128, 0)");
- testExpected("getComputedStyle(cue).textShadow", "rgb(0, 255, 0) 3px 3px 0px");
- testExpected("getComputedStyle(cue).fontSize", "20px");
- testExpected("getComputedStyle(cue).fontFamily", "arial");
- testExpected("getComputedStyle(cue).fontStyle", "normal");
- testExpected("getComputedStyle(cue).fontVariant", "normal");
+ video.oncanplaythrough = t.step_func_done(function() {
+ var cue = textTrackDisplayElement(video, "cue").firstElementChild;
+ var cueStyle = getComputedStyle(cue);
+ assert_equals(cueStyle.color, "rgb(255, 0, 0)");
+ assert_equals(cueStyle.backgroundColor, "rgb(0, 128, 0)");
+ assert_equals(cueStyle.textShadow, "rgb(0, 255, 0) 3px 3px 0px");
+ assert_equals(cueStyle.fontSize, "20px");
+ assert_equals(cueStyle.fontFamily, "arial");
+ assert_equals(cueStyle.fontStyle, "normal");
+ assert_equals(cueStyle.fontVariant, "normal");
+
// Apply user settings and verify they override author-specified settings
- applyUserOverrideSettings();
- run("video.currentTime = 0.3");
- verifyUserOverrideSettings();
- }
-
- function verifyUserOverrideSettings() {
- consoleWrite("");
- cue = textTrackDisplayElement(video, 'cue').firstElementChild;
- testExpected("getComputedStyle(cue).color", "rgb(0, 255, 255)");
- testExpected("getComputedStyle(cue).backgroundColor", "rgb(0, 128, 0)");
- testExpected("getComputedStyle(cue).textShadow", "rgb(255, 0, 0) 2px 2px 0px");
- testExpected("getComputedStyle(cue).fontSize", "14px");
- testExpected("getComputedStyle(cue).fontFamily", "fantasy");
- testExpected("getComputedStyle(cue).fontStyle", "italic");
- testExpected("getComputedStyle(cue).fontVariant", "small-caps");
- endTest();
- }
+ internals.settings.setTextTrackTextColor("cyan");
+ internals.settings.setTextTrackBackgroundColor("green");
+ internals.settings.setTextTrackTextShadow("2px 2px #ff0000")
+ internals.settings.setTextTrackTextSize("14px");
+ internals.settings.setTextTrackFontFamily("fantasy");
+ internals.settings.setTextTrackFontStyle("italic");
+ internals.settings.setTextTrackFontVariant("small-caps");
+
+ video.currentTime = 0.3;
- window.onload = function() {
- consoleWrite("Test that WebVTT objects are being styled correctly based on user settings that should override author settings.");
- findMediaElement();
- video.src = findMediaFile('video', '../content/test');
- video.currentTime = 0.1;
- waitForEvent('canplaythrough', verifyAuthorSettings);
- }
-
-</script>
-<video>
- <track src="captions-webvtt/styling.vtt" kind="captions" default>
-</video>
+ cue = textTrackDisplayElement(video, "cue").firstElementChild;
+ cueStyle = getComputedStyle(cue);
+ assert_equals(cueStyle.color, "rgb(0, 255, 255)");
+ assert_equals(cueStyle.backgroundColor, "rgb(0, 128, 0)");
+ assert_equals(cueStyle.textShadow, "rgb(255, 0, 0) 2px 2px 0px");
+ assert_equals(cueStyle.fontSize, "14px");
+ assert_equals(cueStyle.fontFamily, "fantasy");
+ assert_equals(cueStyle.fontStyle, "italic");
+ assert_equals(cueStyle.fontVariant, "small-caps");
+ });
+});
+</script>
« no previous file with comments | « third_party/WebKit/LayoutTests/media/track/track-css-property-whitelist-expected.txt ('k') | third_party/WebKit/LayoutTests/media/track/track-css-user-settings-override-author-settings-expected.txt » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698