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

Unified Diff: third_party/WebKit/LayoutTests/fast/canvas/canvas-largedraws.html

Issue 2681423002: Use testharness.js instead of js-test.js in LayoutTests/fast/canvas tests. (Closed)
Patch Set: 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-largedraws.html
diff --git a/third_party/WebKit/LayoutTests/fast/canvas/canvas-largedraws.html b/third_party/WebKit/LayoutTests/fast/canvas/canvas-largedraws.html
index a5f1bc88485dce3acab22809808e1f2c1f095de2..fc36dec498c2cd04250f25c6ee5d0d7e04abb669 100644
--- a/third_party/WebKit/LayoutTests/fast/canvas/canvas-largedraws.html
+++ b/third_party/WebKit/LayoutTests/fast/canvas/canvas-largedraws.html
@@ -1,18 +1,16 @@
-<!DOCTYPE html>
-<html>
- <body>
- <script>
- if (window.testRunner)
- testRunner.dumpAsText();
+<script src="../../resources/testharness.js"></script>
+<script src="../../resources/testharnessreport.js"></script>
- function newCanvasContext() {
+<script>
+test(function(t) {
+ function newCanvasContext() {
canvas = document.createElement("canvas");
canvas.width = 100;
canvas.height = 100;
return canvas.getContext("2d");
- }
+ }
- window.onload = function () {
+ window.onload = function () {
Justin Novosad 2017/02/16 15:41:56 onload is async. This code will end up being calle
zakerinasab 2017/02/16 18:12:40 Done.
ctx = newCanvasContext();
ctx.fillStyle = '#0f0';
ctx.fillRect(0, 0, 100, 50);
@@ -39,9 +37,7 @@
ctx.arc(105000000005, -105000000005, 105000000004, 0, 10500000000*Math.PI, false);
ctx.fill();
ctx.getImageData(50, 25, 1, 1);
- }
- </script>
- PASS: Draw commands with big numbers cause no problems.
- </body>
-</html>
+ }
+}, 'Draw commands with big numbers cause no problems.');
+</script>

Powered by Google App Engine
This is Rietveld 408576698