Bug 468023 - Synchronize RefTest results to tinderbox waterfall from (new) log; (Bv2a) New summarized log format; r=dbaron

layout/tools/reftest/reftest.js

@@ -63,17 +63,17 @@ var gURIUseCounts;
 // Map from URI spec to the canvas rendered for that URI
 var gURICanvases;
 var gTestResults = {
-  // The errors...
+  // Successful...
+  Pass: 0,
+  LoadOnly: 0,
+  // Unexpected...
   Exception: 0,
   FailedLoad: 0,
   UnexpectedFail: 0,
   UnexpectedPass: 0,
-  // The successes...
-  Pass: 0,
+  // Known problems...
   KnownFail : 0,
   Random : 0,
-  LoadOnly: 0,
-  // The unknowns.
   Skip: 0,
 };
 var gTotalTests = 0;
@@ -423,9 +423,22 @@ function DoneTests()
          "ms (" + gSlowestTestURL + ")\n");
 
     dump("REFTEST INFO | Result summary:\n");
-    for (var result in gTestResults)
-      if (gTestResults[result] != 0)
-        dump("REFTEST INFO | " + result + ": " + gTestResults[result] + "\n");
+    var count = gTestResults.Pass + gTestResults.LoadOnly;
+    dump("REFTEST INFO | Successful: " + count + " (" +
+         gTestResults.Pass + " pass, " +
+         gTestResults.LoadOnly + " load only)\n");
+    count = gTestResults.Exception + gTestResults.FailedLoad +
+            gTestResults.UnexpectedFail + gTestResults.UnexpectedPass;
+    dump("REFTEST INFO | Unexpected: " + count + " (" +
+         gTestResults.UnexpectedFail + " unexpected fail, " +
+         gTestResults.UnexpectedPass + " unexpected pass, " +
+         gTestResults.FailedLoad + " failed load, " +
+         gTestResults.Exception + " exception)\n");
+    count = gTestResults.KnownFail + gTestResults.Random + gTestResults.Skip;
+    dump("REFTEST INFO | Known problems: " + count + " (" +
+         gTestResults.KnownFail + " known fail, " +
+         gTestResults.Random + " random, " +
+         gTestResults.Skip + " skipped)\n");
 
     dump("REFTEST INFO | Total canvas count = " + gRecycledCanvases.length + "\n");