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

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

Issue 2343793002: Convert fast/canvas Layouttests to testharness (Closed)
Patch Set: Convert fast/canvas Layouttests to testharness Created 4 years, 3 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/translate-text.html
diff --git a/third_party/WebKit/LayoutTests/fast/canvas/translate-text.html b/third_party/WebKit/LayoutTests/fast/canvas/translate-text.html
index 60992030f7a911a9354d3d254eb415466fe0e867..33a898f33293a3abf00a07f2e55ed7d61319ffcb 100644
--- a/third_party/WebKit/LayoutTests/fast/canvas/translate-text.html
+++ b/third_party/WebKit/LayoutTests/fast/canvas/translate-text.html
@@ -1,9 +1,41 @@
-<!DOCTYPE HTML PUBLIC "-//IETF//DTD HTML//EN">
-<html>
-<head>
-<script src="../../resources/js-test.js"></script>
-</head>
-<body>
-<script src="script-tests/translate-text.js"></script>
-</body>
-</html>
+<!DOCTYPE html>
+<title>Test for chromium's canvas bug where fillText resets the current context</title>
fs 2016/09/16 14:30:31 Maybe drop "chromium's" here.
Abhishek Kanike 2016/09/17 02:28:09 Done.
+<script src="../../resources/testharness.js"></script>
+<script src="../../resources/testharnessreport.js"></script>
+
+<body></body>
+<script>
+// Chromium Bug: https://bugs.webkit.org/show_bug.cgi?id=26436.
fs 2016/09/16 14:30:31 And "Chromium" here.
Abhishek Kanike 2016/09/17 02:28:10 Done.
+// You must see green box with message 'This text should be seen',
+// and black box below the green box.
+test(function() {
+ function pixelToString(p) {
+ return "[" + p[0] + ", " + p[1] + ", " + p[2] + ", " + p[3] + "]"
+ }
+
+ var canvas = document.createElement("canvas");
+ canvas.height = 100;
+ canvas.width = 500;
+ canvas.style.height = "100";
+ canvas.style.width = "500";
+
+ document.body.appendChild(canvas);
+
+ var ctx = canvas.getContext('2d');
+ ctx.fillStyle = 'rgb(0,255,0)';
+ ctx.fillRect(0, 0, 500, 100);
+ ctx.fillStyle = 'rgb(0,0,0)';
+ ctx.fillText("This text should be seen", 20, 20);
+ ctx.translate(0, 50);
+ ctx.font = "10pt Arial";
+ ctx.fillText("This text should NOT be seen", 20, 20);
+ // If fillText resets the current context, this rectangle won't hide the text.
+ ctx.fillStyle = 'rgb(0,0,0)';
+ ctx.fillRect(0, 0, 500, 50);
+
+ // Pixel value at 0, 0
+ var pixel = ctx.getImageData(0, 0, 1, 1).data;
+
+ assert_equals(pixelToString(pixel), "[0, 255, 0, 255]");
fs 2016/09/16 14:30:31 assert_array_equals rather than going to string?
Abhishek Kanike 2016/09/17 02:28:09 Done.
+});
+</script>

Powered by Google App Engine
This is Rietveld 408576698