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

Unified Diff: LayoutTests/http/tests/resources/testharnessreport.js

Issue 302093010: Update http/resources/teshharnessreport.js to match CL 109763007. (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@update-testharness
Patch Set: Created 6 years, 7 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: LayoutTests/http/tests/resources/testharnessreport.js
diff --git a/LayoutTests/http/tests/resources/testharnessreport.js b/LayoutTests/http/tests/resources/testharnessreport.js
index 8d72d9acc0922023778d2635f9cf4cdac2cd3fd4..7a9402bf4ebd859468a55ff6f0fa1565b55c1ce9 100644
--- a/LayoutTests/http/tests/resources/testharnessreport.js
+++ b/LayoutTests/http/tests/resources/testharnessreport.js
@@ -47,7 +47,7 @@ add_completion_callback(function (tests, harness_status){
var results = document.createElement("pre");
// Declare result string
- var resultStr = "\n";
+ var resultStr = "This is a testharness.js-based test.\n";
// Check harness_status. If it is not 0, tests did not
// execute correctly, output the error code and message
@@ -67,9 +67,10 @@ add_completion_callback(function (tests, harness_status){
"\n";
}
}
+ resultStr += "Harness: the test ran to completion.\n";
- // Set results element's innerHTML to the results string
- results.innerHTML = resultStr;
+ // Set results element's textContent to the results string
+ results.textContent = resultStr;
// Add results element to document
document.body.appendChild(results);

Powered by Google App Engine
This is Rietveld 408576698