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

Unified Diff: third_party/WebKit/LayoutTests/fast/canvas/canvas-skia-excessive-size.html

Issue 2693193003: Use testharness.js instead of js-test.js in LayoutTests/fast/canvas tests. (Closed)
Patch Set: Corrections 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-skia-excessive-size.html
diff --git a/third_party/WebKit/LayoutTests/fast/canvas/canvas-skia-excessive-size.html b/third_party/WebKit/LayoutTests/fast/canvas/canvas-skia-excessive-size.html
index 38c3b442c2bd0ef88e4c39249b9f75eeb91c74da..b237a711c7359b8762d7fb592a32f46fb3707d86 100644
--- a/third_party/WebKit/LayoutTests/fast/canvas/canvas-skia-excessive-size.html
+++ b/third_party/WebKit/LayoutTests/fast/canvas/canvas-skia-excessive-size.html
@@ -1,37 +1,21 @@
-<head>
-<script>
-function print(message)
-{
- var paragraph = document.createElement("li");
- paragraph.appendChild(document.createTextNode(message));
- document.getElementById("console").appendChild(paragraph);
-}
-function test()
-{
- if (window.testRunner)
- testRunner.dumpAsText();
+<script src="../../resources/testharness.js"></script>
+<script src="../../resources/testharnessreport.js"></script>
+<canvas id="bigCanvas" width="134217728" height="1"></canvas>
+<script>
+function runTest() {
var canvas = document.getElementById("bigCanvas");
var width = canvas.width;
- // We need to perform a context fetch to force allocation of
- // canvas resources.
- if (canvas.getContext)
- {
- var ctx = canvas.getContext("2d");
- if (ctx == null)
- {
- print("Canvas 2d context = null!");
- }
- }
-
- print("Survived canvas creation attempt. Width = " + width);
+ // We need to perform a context fetch to force allocation of canvas resources.
+ var ctx = canvas.getContext("2d");
+ assert_false(ctx == null);
+ assert_equals(canvas.width, 134217728);
}
+
+async_test(t => {
+ window.onload = function() {
+ t.step(runTest);
+ t.done();
+ }
+}, 'Verifies if the browser survives the attempted creation of an excessively large canvas.');
</script>
-</head>
-<body onload="test();">
-<canvas id="bigCanvas" width="134217728" height="1"></canvas>
-<p>This test checks to see if the browser survives the attempted creation
-of an excessively large canvas.</p>
-<hr>
-<p><ol id=console></ol></p>
-</body>

Powered by Google App Engine
This is Rietveld 408576698