Bug 1752734 - Remove scaling down of full page screenshots. r=sfoster
authorNiklas Baumgardner <nbaumgardner@mozilla.com>
Tue, 05 Apr 2022 07:10:24 +0000
changeset 613367 41e0f33c8e12416b6cdbbdaf8b2d075c7f84fd4c
parent 613366 c662b7c5ae2a81f36bcd07c709709d2577d9ebfb
child 613368 6e3c8547c307e4ec70341800f5917f4dc37c3edb
push id39523
push userctuns@mozilla.com
push dateTue, 05 Apr 2022 21:23:13 +0000
treeherdermozilla-central@7fac8607d414 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewerssfoster
bugs1752734
milestone101.0a1
first release with
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
last release without
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
Bug 1752734 - Remove scaling down of full page screenshots. r=sfoster Differential Revision: https://phabricator.services.mozilla.com/D142615
browser/extensions/screenshots/background/takeshot.js
--- a/browser/extensions/screenshots/background/takeshot.js
+++ b/browser/extensions/screenshots/background/takeshot.js
@@ -37,19 +37,16 @@ this.takeshot = (function() {
         x: 0,
         y: 0,
         width: pos.width,
         height: pos.height,
       };
       options.rect = rectangle;
       options.resetScrollPosition = true;
 
-      // To avoid creating extremely large images (which causes
-      // performance problems), we set the devicePixelRatio to 1.
-      devicePixelRatio = 1;
       options.scale = 1 / zoomFactor;
     } else if (screenshotType != "visible") {
       let rectangle = {
         x: pos.left,
         y: pos.top,
         width: pos.width,
         height: pos.height,
       };