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

Unified Diff: third_party/WebKit/LayoutTests/canvas/philip/tests/2d.text-custom-font-load-crash.html

Issue 1589643003: OOPIF support for testRunner.dumpAsText and similar layout dumps. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Ready for review. Created 4 years, 11 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/canvas/philip/tests/2d.text-custom-font-load-crash.html
diff --git a/third_party/WebKit/LayoutTests/canvas/philip/tests/2d.text-custom-font-load-crash.html b/third_party/WebKit/LayoutTests/canvas/philip/tests/2d.text-custom-font-load-crash.html
index 2a0984ba734577c63cd774967bfcd17a7bc6c4c1..bf9c5def7b326878ef48fc52b22a2e13d3c1627d 100644
--- a/third_party/WebKit/LayoutTests/canvas/philip/tests/2d.text-custom-font-load-crash.html
+++ b/third_party/WebKit/LayoutTests/canvas/philip/tests/2d.text-custom-font-load-crash.html
@@ -25,6 +25,7 @@ deferTest();
setTimeout(wrapFunction(function () {
ctx.fillText('AA', 0, 50);
+ _assert(true, "Test passes if it does not crash.")
dcheng 2016/01/23 05:17:33 Not sure if I missed this a comment about this, bu
Łukasz Anforowicz 2016/01/26 00:50:02 This change is not necessary in the latest patchse
if (window.testRunner)
testRunner.notifyDone();
}), 500);

Powered by Google App Engine
This is Rietveld 408576698