Bug 1034619 - Add a comment to the screenshot gcli command. r=pbrosset
authorSébastien Blin <amarok@enconn.fr>
Thu, 03 Jul 2014 17:54:29 +0200
changeset 192604 1c3ec706f97bc8b5d13f5cab79aa7323729e84e1
parent 192603 3271ce25db004f93597060325e34a2f7001fc682
child 192605 b7bab8b058cb4c3ace56b9b06a231e8845680326
push id27091
push userkwierso@gmail.com
push dateMon, 07 Jul 2014 21:11:34 +0000
treeherdermozilla-central@f14b47c39013 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewerspbrosset
bugs1034619
milestone33.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 1034619 - Add a comment to the screenshot gcli command. r=pbrosset
toolkit/devtools/gcli/commands/screenshot.js
--- a/toolkit/devtools/gcli/commands/screenshot.js
+++ b/toolkit/devtools/gcli/commands/screenshot.js
@@ -119,16 +119,18 @@ exports.items = [
             let lh = new LayoutHelpers(window);
             let rect = lh.getRect(node, window);
             top = rect.top;
             left = rect.left;
             width = rect.width;
             height = rect.height;
           }
         } else {
+          // Bug 961832: GCLI screenshot shows fixed position element in wrong position if
+          // we don't scroll to top
           window.scrollTo(0,0);
           width = window.innerWidth + window.scrollMaxX;
           height = window.innerHeight + window.scrollMaxY;
         }
         canvas.width = width;
         canvas.height = height;
 
         let ctx = canvas.getContext("2d");