Bug 597580 - Remove old display:none hack in resize handler [r=stechz]
authorMatt Brubeck <mbrubeck@mozilla.com>
Wed, 20 Oct 2010 14:00:39 -0700
changeset 66884 2b2b0a35d4e387b8717842027492d69e4b51c9fd
parent 66883 ce008a40c5dee604656226b7438582a54352b19d
child 66885 47075e5d26f31dbaf07878332386d4bb8b1eb8fc
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)
reviewersstechz
bugs597580
Bug 597580 - Remove old display:none hack in resize handler [r=stechz]
mobile/chrome/content/browser.js
--- a/mobile/chrome/content/browser.js
+++ b/mobile/chrome/content/browser.js
@@ -281,21 +281,16 @@ var Browser = {
         if (tab.browser.contentWindowWidth == oldContentWindowWidth)
           tab.restoreViewportPosition(oldWidth, w);
       }
 
       // XXX page scrollbox jumps to a strange value on resize. Scrolling it will
       // bound it to a sane place, but not where we were when the resize began :(
       Browser.hideTitlebar();
 
-      // XXX Hack. Browser also behaves badly on resize.
-      let browser = getBrowser();
-      browser.style.display = "none";
-      browser.style.display = "block";
-
       // We want to keep the current focused element into view if possible
       let currentElement = document.activeElement;
       let [scrollbox, scrollInterface] = ScrollUtils.getScrollboxFromElement(currentElement);
       if (currentElement && (currentElement.id != "inputhandler-overlay") && scrollbox && currentElement != scrollbox) {
         // retrieve the direct child of the scrollbox
         while (currentElement.parentNode != scrollbox)
           currentElement = currentElement.parentNode;