Bug 676321 - Do not reset popup position on content resize, panning, and zooming (r=mbrubeck)
authorLucas Rocha <lucasr@mozilla.com>
Fri, 19 Aug 2011 15:43:22 +0200
changeset 75562 79399ce1a1fbd1f10e22f9328d2af72b7cb0dcff
parent 75561 aa378dbf52e72b41e6c2e172ee966d715170a5a9
child 75563 33000157292b4cef2533a8769a49cd7d1e86d64d
push id21040
push userbmo@edmorley.co.uk
push dateSun, 21 Aug 2011 18:16:59 +0000
treeherdermozilla-central@482742e6fff7 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersmbrubeck
bugs676321
milestone9.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 676321 - Do not reset popup position on content resize, panning, and zooming (r=mbrubeck) We control the sidebar side effects by zeroing the maximum width of the popup every time the window size changes.
mobile/chrome/content/ContentPopupHelper.js
--- a/mobile/chrome/content/ContentPopupHelper.js
+++ b/mobile/chrome/content/ContentPopupHelper.js
@@ -205,27 +205,25 @@ var ContentPopupHelper = {
     switch(aEvent.type) {
       case "TabSelect":
       case "TabClose":
         this.popup = null;
         break;
 
       case "PanBegin":
       case "AnimatedZoomBegin":
-        popup.left = 0;
         popup.style.visibility = "hidden";
         break;
 
       case "PanFinished":
       case "AnimatedZoomEnd":
         this.anchorTo();
         break;
 
       case "MozBeforeResize":
-        popup.left = 0;
         popup.style.visibility = "hidden";
 
         // When screen orientation changes, we have to ensure that
         // the popup width doesn't overflow the content's visible
         // area.
         popup.firstChild.style.maxWidth = "0px";
         break;