Backed out changeset 21d6238c711f (bug 1079222) because it breaks the same thing, just after restarting, which is no better...
authorGijs Kruitbosch <gijskruitbosch@gmail.com>
Mon, 27 Oct 2014 22:19:22 +0000
changeset 212521 37d5c621ad523ca6d18087f7d2826adc93a26230
parent 212520 21d6238c711f8dc533eef72b01d1d78aa385fe47
child 212522 908b844c13c874cc44f3a915656245edb451d5d6
push id27717
push userkwierso@gmail.com
push dateTue, 28 Oct 2014 00:49:29 +0000
treeherdermozilla-central@a255a234946e [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
bugs1079222
milestone36.0a1
backs out21d6238c711f8dc533eef72b01d1d78aa385fe47
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 21d6238c711f (bug 1079222) because it breaks the same thing, just after restarting, which is no better...
browser/base/content/sanitize.js
--- a/browser/base/content/sanitize.js
+++ b/browser/base/content/sanitize.js
@@ -519,42 +519,29 @@ Sanitizer.prototype = {
 
         // First create a new window. We do this first so that on non-mac, we don't
         // accidentally close the app by closing all the windows.
         let handler = Cc["@mozilla.org/browser/clh;1"].getService(Ci.nsIBrowserHandler);
         let defaultArgs = handler.defaultArgs;
         let features = "chrome,all,dialog=no," + this.privateStateForNewWindow;
         let newWindow = existingWindow.openDialog("chrome://browser/content/", "_blank",
                                                   features, defaultArgs);
-#ifdef XP_MACOSX
-        function onFullScreen(e) {
-          newWindow.removeEventListener("fullscreen", onFullScreen);
-          let sizemode = newWindow.document.documentElement.getAttribute("sizemode");
-          if (!newWindow.fullScreen && sizemode == "fullscreen") {
-            e.preventDefault();
-            e.stopPropagation();
-            return false;
-          }
-        }
-        newWindow.addEventListener("fullscreen", onFullScreen);
-#endif
 
         // Window creation and destruction is asynchronous. We need to wait
         // until all existing windows are fully closed, and the new window is
         // fully open, before continuing. Otherwise the rest of the sanitizer
         // could run too early (and miss new cookies being set when a page
         // closes) and/or run too late (and not have a fully-formed window yet
         // in existence). See bug 1088137.
         let newWindowOpened = false;
         function onWindowOpened(subject, topic, data) {
           if (subject != newWindow)
             return;
 
           Services.obs.removeObserver(onWindowOpened, "browser-delayed-startup-finished");
-          newWindow.removeEventListener("fullscreen", onFullScreen);
           newWindowOpened = true;
           // If we're the last thing to happen, invoke callback.
           if (numWindowsClosing == 0)
             aCallback();
         }
 
         let numWindowsClosing = windowList.length;
         function onWindowClosed() {