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

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

Issue 2678493002: Use testharness.js instead of js-test.js in LayoutTests/fast/canvas tests. (Closed)
Patch Set: Addressing comments 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-fillRect.html
diff --git a/third_party/WebKit/LayoutTests/fast/canvas/canvas-fillRect.html b/third_party/WebKit/LayoutTests/fast/canvas/canvas-fillRect.html
index 7079cedfe7b8f5756d31426adbe772e0ef232bd9..b5747da3f14eb667a851244a66d823993b61eb35 100644
--- a/third_party/WebKit/LayoutTests/fast/canvas/canvas-fillRect.html
+++ b/third_party/WebKit/LayoutTests/fast/canvas/canvas-fillRect.html
@@ -1,9 +1,47 @@
-<!DOCTYPE HTML PUBLIC "-//IETF//DTD HTML//EN">
-<html>
-<head>
-<script src="../../resources/js-test.js"></script>
-</head>
+<script src="../../resources/testharness.js"></script>
+<script src="../../resources/testharnessreport.js"></script>
<body>
-<script src="script-tests/canvas-fillRect.js"></script>
+<script>
+test(function(t) {
+ var ctx = document.createElement('canvas').getContext('2d');
+
+ // Fill rect with height = width = 0.
+ ctx.fillStyle = 'red';
+ ctx.fillRect(0, 0, 0, 0);
+
+ var imageData = ctx.getImageData(0, 0, 1, 1);
+ var imgdata = imageData.data;
+ assert_equals(imgdata[0], 0);
+ assert_equals(imgdata[1], 0);
+ assert_equals(imgdata[2], 0);
+
+ ctx.clearRect(0, 0, 1, 1);
+
+ // Fill rect with height = 0, width = 1.
+
+ ctx.fillStyle = 'red';
+ ctx.fillRect(0, 0, 1, 0);
+
+ var imageData = ctx.getImageData(0, 0, 1, 1);
+ var imgdata = imageData.data;
+ assert_equals(imgdata[0], 0);
+ assert_equals(imgdata[1], 0);
+ assert_equals(imgdata[2], 0);
+
+ ctx.clearRect(0, 0, 1, 1);
+
+ // Fill rect with height = 1, width = 0.
+
+ ctx.fillStyle = 'red';
+ ctx.fillRect(0, 0, 0, 1);
+
+ var imageData = ctx.getImageData(0, 0, 1, 1);
+ var imgdata = imageData.data;
+ assert_equals(imgdata[0], 0);
+ assert_equals(imgdata[1], 0);
+ assert_equals(imgdata[2], 0);
+
+ ctx.clearRect(0, 0, 1, 1);
+}, "Series of tests to ensure correct behavior of canvas.fillRect().");
+</script>
</body>
-</html>

Powered by Google App Engine
This is Rietveld 408576698