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 304502 1362d2b7cdbd79f920d6ef660a29da572e2a577d
parent 304501 d3061ac6aff99727ad5c4b98548c5b05d649ac3e
child 304503 dd5df13fccbdbca6a45ad11120a9485dc3272512
push id1001
push userraliiev@mozilla.com
push dateMon, 18 Jan 2016 19:06:03 +0000
treeherdermozilla-release@8b89261f3ac4 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
bugs1192683
milestone44.0a1
backs out4169913a206bbde202bd75c8729879885ad66b1d
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
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