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

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

Issue 2343793002: Convert fast/canvas Layouttests to testharness (Closed)
Patch Set: 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/transformed-canvas-reset.html
diff --git a/third_party/WebKit/LayoutTests/fast/canvas/transformed-canvas-reset.html b/third_party/WebKit/LayoutTests/fast/canvas/transformed-canvas-reset.html
index 14c7461451ec3c085935cb84920adab4b0c8629a..6fd8b3ca216e4e318f1ea6949c74b62124661805 100644
--- a/third_party/WebKit/LayoutTests/fast/canvas/transformed-canvas-reset.html
+++ b/third_party/WebKit/LayoutTests/fast/canvas/transformed-canvas-reset.html
@@ -1,13 +1,16 @@
<!DOCTYPE html>
-<canvas id="can" width=200 height=256 style="transform: translateZ(0)"></canvas>
-Test passes if DRT doesn't crash.
+<title>Test for chromium's canvas on transform property and reset the current context</title>
sivag 2016/09/15 05:31:14 are we still resetting the current context?
Abhishek Kanike 2016/09/15 11:52:56 Sorry the title is misleading. We are not reseting
+<script src="../../resources/testharness.js"></script>
Srirama 2016/09/15 09:29:55 i think we can leave this test as it is, as it doe
Abhishek Kanike 2016/09/15 11:52:56 Yes we are not using the js-test framework. But ke
+<script src="../../resources/testharnessreport.js"></script>
+
+<canvas width=200 height=256 style="transform: translateZ(0)"></canvas>
<script>
-var can = document.querySelector("#can");
-var ctx = can.getContext("2d");
-ctx.fillStyle = "green";
-ctx.fillRect(0, 0, 300, 300);
-document.body.offsetTop;
-can.width = 300;
-if (window.testRunner)
- testRunner.dumpAsText();
+test(function() {
+ var can = document.querySelector("canvas");
+ var ctx = can.getContext("2d");
+ ctx.fillStyle = "green";
+ ctx.fillRect(0, 0, 300, 300);
+ document.body.offsetTop;
+ can.width = 300;
+});
</script>

Powered by Google App Engine
This is Rietveld 408576698