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

Unified Diff: LayoutTests/resources/testharnessreport.js

Issue 1151323002: testharnessreport: Hide unnecessary elements for JS tests (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Sync all of testharnessreport.js, and update test results Created 5 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/resources/testharnessreport.js
diff --git a/LayoutTests/resources/testharnessreport.js b/LayoutTests/resources/testharnessreport.js
index 3830870044d79e90b4b6b662047c50577c498ffb..6e35da5ae1715880635e00aab9953c2dfa84b657 100644
--- a/LayoutTests/resources/testharnessreport.js
+++ b/LayoutTests/resources/testharnessreport.js
@@ -47,6 +47,10 @@ function isCSSWGTest() {
return content && content.match(/\bdom\b/);
}
+function isJSTest() {
+ return !!document.querySelector('script[src*="/resources/testharness"]');
+}
+
/* Using a callback function, test results will be added to the page in a
* manner that allows dumpAsText to produce readable test results
*/
@@ -94,7 +98,7 @@ add_completion_callback(function (tests, harness_status) {
function done() {
if (self.testRunner) {
var logDiv = document.getElementById('log');
- if (isCSSWGTest() && logDiv) {
+ if ((isCSSWGTest() || isJSTest()) && logDiv) {
// Assume it's a CSSWG style test, and anything other than the log div isn't
// material to the testrunner output, so should be hidden from the text dump
for (var i = 0; i < document.body.children.length; i++) {

Powered by Google App Engine
This is Rietveld 408576698