Bug 1411825 - Capture browser-screenshots on try without MOZSCREENSHOTS_SETS. r=bgrins draft
authorMatthew Noorenberghe <mozilla@noorenberghe.ca>
Wed, 25 Oct 2017 19:55:45 -0700
changeset 686638 4421d3085270375a912e2611377ba2299c41d132
parent 686637 6725b54ac153c9cb488e21d09211acc514b9314d
child 737415 a9e1a26ff7a3c915600ac7d755050bf5ecdd051f
push id86232
push usermozilla@noorenberghe.ca
push dateThu, 26 Oct 2017 04:38:34 +0000
reviewersbgrins
bugs1411825
milestone58.0a1
Bug 1411825 - Capture browser-screenshots on try without MOZSCREENSHOTS_SETS. r=bgrins MozReview-Commit-ID: 5xI6hTVFSXh
browser/tools/mozscreenshots/head.js
--- a/browser/tools/mozscreenshots/head.js
+++ b/browser/tools/mozscreenshots/head.js
@@ -44,19 +44,12 @@ function shouldCapture() {
     return false;
   }
 
   if (AppConstants.MOZ_UPDATE_CHANNEL == "nightly") {
     ok(true, "Screenshots aren't captured on Nightlies");
     return false;
   }
 
-  // Don't run pre-defined sets (e.g. primaryUI) on try, require MOZSCREENSHOTS_SETS.
-  // The job is only scheduled on specific repos:
-  // https://dxr.mozilla.org/build-central/search?q=MOCHITEST_BC_SCREENSHOTS
-  let capture = !AppConstants.SOURCE_REVISION_URL.includes("/try/rev/");
-  if (!capture) {
-    ok(true, "Capturing is disabled for this REPO. You may need to use MOZSCREENSHOTS_SETS");
-  }
-  return capture;
+  return true;
 }
 
 add_task(setup);