Bug 705184 - Count unexpected reftest crash dump files as test failures. r=dbaron
authorCameron McCormack <cam@mcc.id.au>
Mon, 05 Dec 2011 14:06:42 +1100
changeset 105123 fdc76cb3b523191deb357accab5b24f2e22dad26
parent 105122 d76195e9e2b8c4ea553cd977dac86b73ac4d9108
child 105124 6dc02881f1b640a8f5a3342660ae5a0eea6a5704
push id55
push usershu@rfrn.org
push dateThu, 30 Aug 2012 01:33:09 +0000
reviewersdbaron
bugs705184
milestone17.0a1
Bug 705184 - Count unexpected reftest crash dump files as test failures. r=dbaron
layout/tools/reftest/reftest.js
--- a/layout/tools/reftest/reftest.js
+++ b/layout/tools/reftest/reftest.js
@@ -1549,16 +1549,17 @@ function FindUnexpectedCrashDumpFiles()
     }
 
     let foundCrashDumpFile = false;
     while (entries.hasMoreElements()) {
         let file = entries.getNext().QueryInterface(CI.nsIFile);
         let path = String(file.path);
         if (path.match(/\.(dmp|extra)$/) && !gUnexpectedCrashDumpFiles[path]) {
             if (!foundCrashDumpFile) {
+                ++gTestResults.UnexpectedFail;
                 foundCrashDumpFile = true;
                 gDumpLog("REFTEST TEST-UNEXPECTED-FAIL | " + gCurrentURL +
                          " | This test left crash dumps behind, but we weren't expecting it to!\n");
             }
             gDumpLog("REFTEST INFO | Found unexpected crash dump file" + path +
                      ".\n");
             gUnexpectedCrashDumpFiles[path] = true;
         }