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

Unified Diff: third_party/WebKit/LayoutTests/imported/wpt/resources/testharnessreport.js

Issue 2500053002: Disable testharness.js timeout. (Closed)
Patch Set: Change TIMEOUT baselines to Timeout expectations Created 4 years, 1 month 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/imported/wpt/resources/testharnessreport.js
diff --git a/third_party/WebKit/LayoutTests/imported/wpt/resources/testharnessreport.js b/third_party/WebKit/LayoutTests/imported/wpt/resources/testharnessreport.js
index 8bb7dea4d27f921d45cbbd6a01ab44c6030b0648..145a0c44d0c20bbc65dea70bf634752b3ba4c2a3 100644
--- a/third_party/WebKit/LayoutTests/imported/wpt/resources/testharnessreport.js
+++ b/third_party/WebKit/LayoutTests/imported/wpt/resources/testharnessreport.js
@@ -27,7 +27,13 @@
// test results into an HTML table. When that table is dumped as text, no
// spacing between cells is preserved, and it is therefore not readable. By
// setting output to false, the HTML table will not be created.
- setup({"output":false});
+ // Also, disable timeout (except for explicit timeout), since the Blink
+ // layout test runner has its own timeout mechanism.
+ // See: https://github.com/w3c/testharness.js/blob/master/docs/api.md#setup
+ setup({
+ "output": false,
+ "explicit_timeout": true
+ });
// Function used to convert the test status code into the corresponding
// string

Powered by Google App Engine
This is Rietveld 408576698