Backed out changeset 4169913a206b (bug 1192683) for being the likely cause of OSX 10.10 mochitest-2 leaks.
authorRyan VanderMeulen <ryanvm@gmail.com>
Fri, 23 Oct 2015 17:43:17 -0400
changeset 269200 1362d2b7cdbd79f920d6ef660a29da572e2a577d
parent 269199 d3061ac6aff99727ad5c4b98548c5b05d649ac3e
child 269201 dd5df13fccbdbca6a45ad11120a9485dc3272512
push id15855
push userryanvm@gmail.com
push dateFri, 23 Oct 2015 21:43:27 +0000
treeherderfx-team@1362d2b7cdbd [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
bugs1192683
milestone44.0a1
backs out4169913a206bbde202bd75c8729879885ad66b1d
Backed out changeset 4169913a206b (bug 1192683) for being the likely cause of OSX 10.10 mochitest-2 leaks.
browser/base/content/browser-fullScreen.js
--- a/browser/base/content/browser-fullScreen.js
+++ b/browser/base/content/browser-fullScreen.js
@@ -302,33 +302,28 @@ var FullScreen = {
     // Popups should only veto chrome collapsing if they were opened when the chrome was not collapsed.
     // Otherwise, they would not affect chrome and the user would expect the chrome to go away.
     // e.g. we wouldn't want the autoscroll icon firing this event, so when the user
     // toggles chrome when moving mouse to the top, it doesn't go away again.
     if (aEvent.type == "popupshown" && !FullScreen._isChromeCollapsed &&
         aEvent.target.localName != "tooltip" && aEvent.target.localName != "window")
       FullScreen._isPopupOpen = true;
     else if (aEvent.type == "popuphidden" && aEvent.target.localName != "tooltip" &&
-             aEvent.target.localName != "window") {
+             aEvent.target.localName != "window")
       FullScreen._isPopupOpen = false;
-      // Try again to hide toolbar when we close the popup.
-      FullScreen.hideNavToolbox(true);
-    }
   },
 
   // Autohide helpers for the context menu item
   getAutohide: function(aItem)
   {
     aItem.setAttribute("checked", gPrefService.getBoolPref("browser.fullscreen.autohide"));
   },
   setAutohide: function()
   {
     gPrefService.setBoolPref("browser.fullscreen.autohide", !gPrefService.getBoolPref("browser.fullscreen.autohide"));
-    // Try again to hide toolbar when we change the pref.
-    FullScreen.hideNavToolbox(true);
   },
 
   _WarningBox: {
     _element: null,
     _origin: null,
 
     /**
      * Timeout object for managing timeout request. If it is started when