Bug 991134 - Output an additional failure line for loadfailed: null failures to make bug 789751 and bug 720452 TBPL starable. r=dbaron, a=test-only
authorEd Morley <emorley@mozilla.com>
Thu, 17 Apr 2014 01:13:38 +0100
changeset 193083 241e852cea2a3f44b34c9c4ea12fbe8cefba2d73
parent 193082 4521025a674c64779a5b3434cf24221eb9798062
child 193084 7cd73a59acae03456c6735c36d61d0073f08555c
push id474
push userasasaki@mozilla.com
push dateMon, 02 Jun 2014 21:01:02 +0000
treeherdermozilla-release@967f4cf1b31c [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersdbaron, test-only
bugs991134, 789751, 720452
milestone30.0a2
Bug 991134 - Output an additional failure line for loadfailed: null failures to make bug 789751 and bug 720452 TBPL starable. r=dbaron, a=test-only
layout/tools/reftest/reftest.js
--- a/layout/tools/reftest/reftest.js
+++ b/layout/tools/reftest/reftest.js
@@ -1682,16 +1682,21 @@ function RecordResult(testRunTime, error
         default:
             throw "Unexpected state.";
     }
 }
 
 function LoadFailed(why)
 {
     ++gTestResults.FailedLoad;
+    // Once bug 896840 is fixed, this can go away, but for now it will give log
+    // output that is TBPL starable for bug 789751 and bug 720452.
+    if (!why) {
+        gDumpLog("REFTEST TEST-UNEXPECTED-FAIL | load failed with unknown reason\n");
+    }
     gDumpLog("REFTEST TEST-UNEXPECTED-FAIL | " +
          gURLs[0]["url" + gState].spec + " | load failed: " + why + "\n");
     FlushTestLog();
     FinishTestItem();
 }
 
 function RemoveExpectedCrashDumpFiles()
 {