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 686632 3c71880b30a78b95271fb20a7ed11296bf517378
parent 686631 675dc2d06f0ee7a7d4a0dfa36874a258215aba81
child 737410 19f7ee893c8bc6dfe87994d9e31cf981c6ee32f4
push id86227
push usermozilla@noorenberghe.ca
push dateThu, 26 Oct 2017 03:23:25 +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);