Bug 1298909 - report slowest test in all cases, even with requestLongerTimeout() is called. r?jmaher draft
authorprakhar0409 <prakhar0409@gmail.com>
Fri, 16 Sep 2016 23:30:00 +0530
changeset 414574 f34e319dae49e7bc86574551629582b7de3ea76b
parent 413272 82d0a583a9a39bf0b0000bccbf6d5c9ec2596bcc
child 531476 9243e7242aaf7c284fe398cbc89004c23392849b
push id29709
push userbmo:prakhar0409@gmail.com
push dateFri, 16 Sep 2016 18:01:05 +0000
reviewersjmaher
bugs1298909
milestone51.0a1
Bug 1298909 - report slowest test in all cases, even with requestLongerTimeout() is called. r?jmaher HGcription>. r?jmaher branch 'default' MozReview-Commit-ID: C8oij8WCQtl
testing/mochitest/tests/SimpleTest/TestRunner.js
--- a/testing/mochitest/tests/SimpleTest/TestRunner.js
+++ b/testing/mochitest/tests/SimpleTest/TestRunner.js
@@ -558,17 +558,17 @@ TestRunner.testFinished = function(tests
 
         TestRunner.structuredLogger.testEnd(TestRunner.currentTestURL,
                                             "OK",
                                             undefined,
                                             "Finished in " + runtime + "ms",
                                             {runtime: runtime}
         );
 
-        if (TestRunner.slowestTestTime < runtime && TestRunner._timeoutFactor == 1) {
+        if (TestRunner.slowestTestTime < runtime && TestRunner._timeoutFactor >= 1) {
           TestRunner.slowestTestTime = runtime;
           TestRunner.slowestTestURL = TestRunner.currentTestURL;
         }
 
         TestRunner.updateUI(tests);
 
         // Don't show the interstitial if we just run one test with no repeats:
         if (TestRunner._urls.length == 1 && TestRunner.repeat <= 1) {