Bug 505700 - Don't allow moving the window with the toolbar while in fullscreen mode. r=enndeakin
authorMarkus Stange <mstange@themasta.com>
Wed, 22 Jul 2009 14:18:08 +0200
changeset 30558 e1ae3719e72a759065b6a9ab8623a7d8c4056ef4
parent 30557 ab16012f68af5e2b6f6fd6a25172ccfbd4a39d04
child 30559 35ecfeedac76e6d0b3236249fd53129a5c9f2251
push id8135
push usermstange@themasta.com
push dateWed, 22 Jul 2009 12:19:07 +0000
treeherdermozilla-central@e1ae3719e72a [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersenndeakin
bugs505700
milestone1.9.2a1pre
Bug 505700 - Don't allow moving the window with the toolbar while in fullscreen mode. r=enndeakin
toolkit/content/widgets/toolbar.xml
--- a/toolkit/content/widgets/toolbar.xml
+++ b/toolkit/content/widgets/toolbar.xml
@@ -425,25 +425,26 @@
           this._setInactiveAsync();
       ]]></handler>
     </handlers>
   </binding>
 
   <binding id="toolbar-drag"
            extends="chrome://global/content/bindings/toolbar.xml#toolbar">
     <implementation>
-      <constructor>
+      <constructor><![CDATA[
         try {
           Components.utils.import("resource://gre/modules/WindowDraggingUtils.jsm");
           let draggableThis = new WindowDraggingElement(this, window);
           draggableThis.mouseDownCheck = function(e) {
-            return !this.parentNode.customizing; // Don't move while customizing.
+            // Don't move while customizing or while in full screen mode.
+            return !this.parentNode.customizing && !window.fullScreen;
           }
         } catch (e) {}
-      </constructor>
+      ]]></constructor>
     </implementation>
   </binding>
 
   <binding id="menubar" extends="chrome://global/content/bindings/toolbar.xml#toolbar-base" display="xul:menubar">
     <implementation implements="nsIAccessibleProvider">
       <property name="accessibleType" readonly="true">
         <getter>
           <![CDATA[