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

Unified Diff: third_party/WebKit/LayoutTests/fast/canvas/set-empty-font-crash.html

Issue 2701053003: 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/set-empty-font-crash.html
diff --git a/third_party/WebKit/LayoutTests/fast/canvas/set-empty-font-crash.html b/third_party/WebKit/LayoutTests/fast/canvas/set-empty-font-crash.html
index 138ec6457ef3e0c4b9a2954de99516eb61c4aa7c..f85676dd61d2223d708922fa65bb72a4f564c824 100644
--- a/third_party/WebKit/LayoutTests/fast/canvas/set-empty-font-crash.html
+++ b/third_party/WebKit/LayoutTests/fast/canvas/set-empty-font-crash.html
@@ -1,8 +1,11 @@
-This test passes if it doesn't crash
+<script src="../../resources/testharness.js"></script>
+<script src="../../resources/testharnessreport.js"></script>
+
<canvas id="canvas"></canvas>
<script>
-canvas.getContext('2d').font = "";
+test(function(t) {
-if (window.testRunner)
- testRunner.dumpAsText();
+ canvas.getContext('2d').font = "";
+
+}, 'Test that setting canvas font to empty string does not crash.');
</script>

Powered by Google App Engine
This is Rietveld 408576698