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

Unified Diff: third_party/WebKit/LayoutTests/fast/canvas/canvas-drawImage-live-video.html

Issue 2674863003: Use testharness.js instead of js-test.js in LayoutTests/fast/canvas tests. (Closed)
Patch Set: Corrections Created 3 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: third_party/WebKit/LayoutTests/fast/canvas/canvas-drawImage-live-video.html
diff --git a/third_party/WebKit/LayoutTests/fast/canvas/canvas-drawImage-live-video.html b/third_party/WebKit/LayoutTests/fast/canvas/canvas-drawImage-live-video.html
index 2e70eb46ae7beed6047de044ffcafc3c23e3b25f..5a6bbfed7b789af94f97c3ccf4f3741b70f2e36a 100644
--- a/third_party/WebKit/LayoutTests/fast/canvas/canvas-drawImage-live-video.html
+++ b/third_party/WebKit/LayoutTests/fast/canvas/canvas-drawImage-live-video.html
@@ -7,18 +7,14 @@
</style>
</head>
<body>
- <canvas id="canvas"></canvas>
- <video id="video">
- <source src="../../media/resources/test-live.webm" type='video/webm' />
- </video>
- <script src="../../resources/js-test.js"></script>
- <script>
- description('Verify that consecutive drawImage from a live video correctly propagates frame updates.');
- if (window.testRunner) {
- testRunner.dumpAsText();
- testRunner.waitUntilDone();
- }
-
+<canvas id="canvas"></canvas>
+<video id="video">
+ <source src="../../media/resources/test-live.webm" type='video/webm' />
+</video>
+<script src="../../resources/testharness.js"></script>
+<script src="../../resources/testharnessreport.js"></script>
+<script>
+test(function(t) {
Justin Novosad 2017/02/09 20:32:01 why no longer async?
zakerinasab 2017/02/13 16:25:19 Done.
var canvas = document.getElementById("canvas");
canvas.width = 100;
canvas.height = 100;
@@ -56,11 +52,9 @@
break;
}
}
- shouldBeFalse("imagesAreTheSame");
- if (window.testRunner)
- testRunner.notifyDone();
+ assert_fasle(imagesAreTheSame);
}
}
- </script>
+}, 'Verify that consecutive drawImage from a live video correctly propagates frame updates.');
+</script>
</body>
-</html>

Powered by Google App Engine
This is Rietveld 408576698