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

Unified Diff: third_party/WebKit/LayoutTests/media/video-poster-scale.html

Issue 2162353002: Remove video-test.js dependency from video-[poster*, replaces*] tests. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 years, 5 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: third_party/WebKit/LayoutTests/media/video-poster-scale.html
diff --git a/third_party/WebKit/LayoutTests/media/video-poster-scale.html b/third_party/WebKit/LayoutTests/media/video-poster-scale.html
index fc19161dd8713ed33d142acbcc6f6e34e5cae8c8..72db078700e09b36e03bab2197b6a63be6adb570 100644
--- a/third_party/WebKit/LayoutTests/media/video-poster-scale.html
+++ b/third_party/WebKit/LayoutTests/media/video-poster-scale.html
@@ -1,48 +1,32 @@
-<!DOCTYPE HTML5>
-
-<html>
- <head>
- <title>'poster' aspect ratio test</title>
- <!-- TODO(foolip): Convert test to testharness.js. crbug.com/588956
- (Please avoid writing new tests using video-test.js) -->
- <script src=video-test.js></script>
- <style>
- video {
- border: 3px solid red;
- background-color: #ee00ee;
- }
- #wide { width: 320px; height: 150px; }
- #tall { width: 150px; height: 320px; }
- #big { width: 320px; height: 240px; }
- #skinny { width: 50px; height: 320px; }
- #short { width: 320px; height: 50px; }
- #just_right { width: 76px; height: 103px; }
- </style>
- <script>
- var videos;
- function start()
- {
- videos = document.getElementsByTagName('video');
- for (var ndx = 0; ndx < videos.length; ndx++ ) {
- testExpected("videos[" + ndx + "].getAttribute('poster')", "content/abe.png");
- testExpected("relativeURL(videos[" + ndx + "].poster)", "content/abe.png");
- }
-
- endTest();
- }
- </script>
- </head>
-
-
- <body onload="start()">
- <video poster="content/abe.png" id=wide></video>
- <br>
- <video poster="content/abe.png" id=just_right></video>
- <video poster="content/abe.png" id=no-style></video>
- <br>
- <video poster="content/abe.png" id=skinny></video>
- <video poster="content/abe.png" id=big></video>
- <video poster="content/abe.png" id=short></video>
- <br>
- </body>
-</html>
+<!DOCTYPE html>
+<title>"poster" aspect ratio test.</title>
+<style>
+video {
+ border: 3px solid red;
+ background-color: #ee00ee;
+}
+#wide { width: 320px; height: 150px; }
+#tall { width: 150px; height: 320px; }
+#big { width: 320px; height: 240px; }
+#skinny { width: 50px; height: 320px; }
+#short { width: 320px; height: 50px; }
+#just_right { width: 76px; height: 103px; }
+</style>
+<script>
+// TODO(srirama.m): convert this test to reference test.
+testRunner.waitUntilDone();
+function start() {
+ testRunner.notifyDone();
+}
+</script>
+<body onload="start()">
+ <video poster="content/abe.png" id="wide"></video>
+ <br>
+ <video poster="content/abe.png" id="just_right"></video>
+ <video poster="content/abe.png" id="no-style"></video>
+ <br>
+ <video poster="content/abe.png" id="skinny"></video>
+ <video poster="content/abe.png" id="big"></video>
+ <video poster="content/abe.png" id="short"></video>
+ <br>
+</body>
« no previous file with comments | « third_party/WebKit/LayoutTests/TestExpectations ('k') | third_party/WebKit/LayoutTests/media/video-replaces-poster.html » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698