Check capture type before it's potentially changed. (#3968)

mozilla-services · Jan 25, 2018 · b9ce266 · b9ce266
1 parent 43ad2df
commit b9ce266
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/addon/webextension/selector/uicontrol.js b/addon/webextension/selector/uicontrol.js
@@ -201,15 +201,15 @@ this.uicontrol = (function() {
     onDownloadPreview: () => {
       sendEvent(`download-${captureType.replace(/([a-z])([A-Z])/g, '$1-$2').toLowerCase()}`, "download-preview-button");
       // Downloaded shots don't have dimension limits
-      removeDimensionLimitsOnFullPageShot();
       let previewDataUrl = (captureType === "fullPageTruncated") ? null : dataUrl;
+      removeDimensionLimitsOnFullPageShot();
       shooter.downloadShot(selectedPos, previewDataUrl);
     },
     onCopyPreview: () => {
       sendEvent(`copy-${captureType.replace(/([a-z])([A-Z])/g, '$1-$2').toLowerCase()}`, "copy-preview-button");
       // Copied shots don't have dimension limits
-      removeDimensionLimitsOnFullPageShot();
       let previewDataUrl = (captureType === "fullPageTruncated") ? null : dataUrl;
+      removeDimensionLimitsOnFullPageShot();
       shooter.copyShot(selectedPos, previewDataUrl);
     }
   };