From b7c5c14e275a8a89b3ec16a3f51de146a93344a3 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Emilio=20Cobos=20=C3=81lvarez?= <emilio@crisal.io>
Date: Thu, 12 Jul 2018 09:06:28 +0000
Subject: [PATCH] Bug 1474877: Fix mach reftest --repeat N. r=jmaher

We need to call ReadTests because otherwise we wouldn't recompute `g.urls`.

Similarly, we don't want to throw when there are no remaining reftests, since we
handle that case properly.

Differential Revision: https://phabricator.services.mozilla.com/D2069

--HG--
extra : moz-landing-system : lando
---
 layout/tools/reftest/reftest.jsm | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/layout/tools/reftest/reftest.jsm b/layout/tools/reftest/reftest.jsm
index ce91f5e970c5..be420a631744 100644
--- a/layout/tools/reftest/reftest.jsm
+++ b/layout/tools/reftest/reftest.jsm
@@ -508,7 +508,7 @@ function StartTests()
         }
 
         g.totalTests = g.urls.length;
-        if (!g.totalTests && !g.verify)
+        if (!g.totalTests && !g.verify && !g.repeat)
             throw "No tests to run";
 
         g.uriCanvases = {};
@@ -626,7 +626,7 @@ function StartCurrentTest()
     } else if (g.urls.length == 0 && g.repeat > 0) {
         // Repeat
         g.repeat--;
-        StartTests();
+        ReadTests();
     } else {
         if (g.urls[0].chaosMode) {
             g.windowUtils.enterChaosMode();