Backout bug 593310 - checkerboard background regresses panning [r=me]
authorMark Finkle <mfinkle@mozilla.com>
Sat, 13 Nov 2010 00:15:36 -0500
changeset 67016 f75096fc5adf4e6ae11c01fa681df79ca973e00f
parent 67015 7424154801813e6e16d8b2f7bf89bfd9d6c52e1a
child 67017 a83a8a6928f7cab63593c18eafe418c8dfdd3194
push id1
push userroot
push dateTue, 26 Apr 2011 22:38:44 +0000
treeherdermozilla-beta@bfdb6e623a36 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersme
bugs593310
Backout bug 593310 - checkerboard background regresses panning [r=me]
mobile/chrome/content/browser.js
mobile/themes/core/browser.css
--- a/mobile/chrome/content/browser.js
+++ b/mobile/chrome/content/browser.js
@@ -205,35 +205,26 @@ var Browser = {
 
     // Warning, total hack ahead. All of the real-browser related scrolling code
     // lies in a pretend scrollbox here. Let's not land this as-is. Maybe it's time
     // to redo all the dragging code.
     this.contentScrollbox = Elements.browsers;
     this.contentScrollboxScroller = {
       scrollBy: function(aDx, aDy) {
         getBrowser().scrollBy(aDx, aDy);
-
-        let scroll = getBrowser().getPosition();
-        this._updateBackground(scroll.x, scroll.y);
       },
 
       scrollTo: function(aX, aY) {
         getBrowser().scrollTo(aX, aY);
-        this._updateBackground(aX, aY);
       },
 
       getPosition: function(aScrollX, aScrollY) {
         let { x: x, y: y } = getBrowser().getPosition();
         aScrollX.value = x;
         aScrollY.value = y;
-      },
-
-      _updateBackground: function _updateBackground(aX, aY) {
-        let position = (-aX % 128) + "px " + (-aY % 128) + "px";
-        document.getElementById("browsers").style.backgroundPosition = position;
       }
     };
 
     /* horizontally scrolling box that holds the sidebars as well as the contentScrollbox */
     let controlsScrollbox = this.controlsScrollbox = document.getElementById("controls-scrollbox");
     this.controlsScrollboxScroller = controlsScrollbox.boxObject.QueryInterface(Ci.nsIScrollBoxObject);
     controlsScrollbox.customDragger = {
       isDraggable: function isDraggable(target, content) { return {}; },
--- a/mobile/themes/core/browser.css
+++ b/mobile/themes/core/browser.css
@@ -395,20 +395,16 @@ toolbarbutton.choice-history {
   list-style-image: url(chrome://browser/skin/images/history-48.png);
 }
 
 toolbarbutton.choice-remotetabs {
   list-style-image: url(chrome://browser/skin/images/remotetabs-48.png);
 }
 
 /* browsers ---------------------------------------------------------------- */
-#browsers {
-  background-image: url(chrome://browser/content/checkerboard.png);
-}
-
 .input-overlay:-moz-focusring {
   outline: 0 !important;
 }
 
 /*
  * These rules prevent browser to steal the focus while touching the screen
  * to allow panning/zooming of the browser view without closing the VKB
  */