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

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

Issue 2052643002: This change modifies testharnessreport (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Fix failing tests Created 4 years, 6 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/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 1cd3133e1eaa9f2ec698fad997f8dddc3ddb40f3..e6bf4c7b381e542a0148eb8419d6fe524a3ecbfe 100644
--- a/third_party/WebKit/LayoutTests/imported/wpt/resources/testharnessreport.js
+++ b/third_party/WebKit/LayoutTests/imported/wpt/resources/testharnessreport.js
@@ -91,15 +91,15 @@
" , harness_status.message = " +
harness_status.message +
"\n";
- } else {
- // Iterate through tests array and build string that contains
- // results for all tests.
- for (var i = 0; i < tests.length; ++i) {
- resultStr += convertResult(tests[i].status) + " " +
- sanitize(tests[i].name) + " " +
- sanitize(tests[i].message) + "\n";
- }
}
+ // Iterate through tests array and build string that contains
+ // results for all tests.
+ for (var i = 0; i < tests.length; ++i) {
+ resultStr += convertResult(tests[i].status) + " " +
+ sanitize(tests[i].name) + " " +
+ sanitize(tests[i].message) + "\n";
+ }
+
resultStr += "Harness: the test ran to completion.\n";
// Set results element's textContent to the results string.

Powered by Google App Engine
This is Rietveld 408576698