Report w3c test result count for WPT

third_party/WebKit/LayoutTests/resources/testharnessreport.js

 /*
  * THIS FILE INTENTIONALLY LEFT BLANK
  *
  * More specifically, this file is intended for vendors to implement
  * code needed to integrate testharness.js tests with their own test systems.
  *
  * Typically such integration will attach callbacks when each test is
  * has run, using add_result_callback(callback(test)), or when the whole test file has
  * completed, using add_completion_callback(callback(tests, harness_status)).
  *
@@ skipping 167 matching lines @@
                                 " " + sanitize(prefix) + " " + (j - i) + " tests\n"
                             i = j - 1;
                             continue;
                         }
                     }
                 }
                 resultStr += convertResult(tests[i].status) + " " +
                     sanitize(tests[i].name) + " " +
                     sanitize(tests[i].message) + "\n";
             }
+        } else if (output_document.URL.indexOf("http://web-platform.test") >= 0){

[tkent, 2017/01/26 22:09:43]

I don't think this condition makes much sense.  The new stat line would be
helpful for all of testharness.js tests, not only external/wpt/.

[yoichio, 2017/01/27 07:21:56]

I would be helpful but it needs much rebasing -expected.txt.
I prefer step-by-step on each Layouttest directory.

+            let testResults = "";
+            let resultCounter = [0,0,0,0];

[tkent, 2017/01/26 22:09:43]

[0,0,0,0] -> [0, 0, 0, 0]

[yoichio, 2017/01/27 07:21:56]

Done.

+            for (var i = 0; i < tests.length; ++i) {
+                resultCounter[tests[i].status]++;
+                testResults += convertResult(tests[i].status) + " " +
+                    sanitize(tests[i].name) + " " +
+                    sanitize(tests[i].message) + "\n";
+            }
+            resultStr += `Found ${tests.length} tests.` +

[tkent, 2017/01/26 22:09:43]

I recommend to show the stats only if
 - non-0 FAIL/TIMEOUT/NOTRUN counts, and
  (If we do so, I think the change of testharness_results.py is unnecessary.)
 - tests.length is larger than a large number.
  For example, tests.length >= 50 ?
  If the number of tests are small, I don't think the stats line is very
helpful.
 - Or show the stats line only for specific URLs such as
web-platform.test/selection/.

The current format is something like:
  Found 46 tests. 36 PASS 10 FAIL 0 TIMEOUT 0 NOTRUN

I'd like:
  Found 46 tests; PASS:36 FAIL:10 TIMEOUT:0 NOTRUN:0
or
  Found 46 tests; 36 PASS, 10 FAIL, 0 TIMEOUT, 0 NOTRUN.

[yoichio, 2017/01/27 07:21:56]

Done.

+             ` ${resultCounter[0]} PASS` +
+             ` ${resultCounter[1]} FAIL` +
+             ` ${resultCounter[2]} TIMEOUT` +
+             ` ${resultCounter[3]} NOTRUN\n`;
+            resultStr += testResults;
         } 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";
             }
         }
 
+        

[tkent, 2017/01/26 22:09:43]

Do not add unnecessary blank lines.

[yoichio, 2017/01/27 07:21:56]

Done.

+
         resultStr += "Harness: the test ran to completion.\n";
 
         // Set results element's textContent to the results string.
         results.textContent = resultStr;
 
         function done() {
             // A temporary workaround since |window.self| property lookup starts
             // failing if the frame is detached. |output_document| may be an
             // ancestor of |self| so clearing |textContent| may detach |self|.
             // To get around this, cache window.self now and use the cached
@@ skipping 32 matching lines @@
             // another completion callback might generate more results.  So, we
             // don't dump the results immediately.
             setTimeout(done, 0);
         } else {
             // Parsing the test HTML isn't finished yet.
             window.addEventListener('load', done);
         }
     });
 
 })();