DevTools: check for canForceCompositingMode before trying to emulate device metrics.

Source/devtools/front_end/OverridesSupport.js

 /*
  * Copyright (C) 2012 Google Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions are
  * met:
  *
  *     * Redistributions of source code must retain the above copyright
  * notice, this list of conditions and the following disclaimer.
  *     * Redistributions in binary form must reproduce the above
@@ skipping 11 matching lines @@
  * OWNER OR CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL,
  * SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT
  * LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE,
  * DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY
  * THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT
  * (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE
  * OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
  */
 
 /**
+ * @extends {WebInspector.Object}

[pfeldman, 2013/10/31 17:45:31]

We define @extends below @constructor

[dgozman, 2013/10/31 17:57:48]

Done.

  * @constructor
  */
 WebInspector.OverridesSupport = function()
 {
-    this._updateAllOverrides();
+    this._canForceCompositingMode = true;
+    WebInspector.resourceTreeModel.addEventListener(WebInspector.ResourceTreeModel.EventTypes.MainFrameNavigated, this.updateCanForceCompositingMode.bind(this), this);
 
     WebInspector.settings.overrideUserAgent.addChangeListener(this._userAgentChanged, this);
     WebInspector.settings.userAgent.addChangeListener(this._userAgentChanged, this);
 
     WebInspector.settings.overrideDeviceMetrics.addChangeListener(this._deviceMetricsChanged, this);
     WebInspector.settings.deviceMetrics.addChangeListener(this._deviceMetricsChanged, this);
     WebInspector.settings.deviceFitWindow.addChangeListener(this._deviceMetricsChanged, this);
 
     WebInspector.settings.overrideGeolocation.addChangeListener(this._geolocationPositionChanged, this);
     WebInspector.settings.geolocationOverride.addChangeListener(this._geolocationPositionChanged, this);
 
     WebInspector.settings.overrideDeviceOrientation.addChangeListener(this._deviceOrientationChanged, this);
     WebInspector.settings.deviceOrientationOverride.addChangeListener(this._deviceOrientationChanged, this);
 
     WebInspector.settings.emulateTouchEvents.addChangeListener(this._emulateTouchEventsChanged, this);
 
     WebInspector.settings.overrideCSSMedia.addChangeListener(this._cssMediaChanged, this);
     WebInspector.settings.emulatedCSSMedia.addChangeListener(this._cssMediaChanged, this);
 }
 
+WebInspector.OverridesSupport.Events = {
+    CanForceCompositingModeChanged: "CanForceCompositingModeChanged",
+}
+
 /**
  * @constructor
  * @param {number} width
  * @param {number} height
  * @param {number} deviceScaleFactor
  * @param {number} fontScaleFactor
  * @param {boolean} textAutosizing
  */
 WebInspector.OverridesSupport.DeviceMetrics = function(width, height, deviceScaleFactor, fontScaleFactor, textAutosizing)
 {
@@ skipping 322 matching lines @@
 WebInspector.OverridesSupport.DeviceOrientation.clearDeviceOrientationOverride = function()
 {
     PageAgent.clearDeviceOrientationOverride();
 }
 
 WebInspector.OverridesSupport.prototype = {
     /**
      * @param {string} deviceMetrics
      * @param {string} userAgent
      */
     emulateDevice: function(deviceMetrics, userAgent)

[pfeldman, 2013/10/31 17:45:31]

Lets disable the whole button instead.

[dgozman, 2013/10/31 17:57:48]

Done.

     {
         WebInspector.settings.deviceMetrics.set(deviceMetrics);
         WebInspector.settings.userAgent.set(userAgent);
-        WebInspector.settings.overrideDeviceMetrics.set(true);
+        if (this._canForceCompositingMode)
+            WebInspector.settings.overrideDeviceMetrics.set(true);
         WebInspector.settings.overrideUserAgent.set(true);
         WebInspector.settings.emulateTouchEvents.set(true);
     },
 
     reset: function()
     {
         WebInspector.settings.overrideDeviceMetrics.set(false);
         WebInspector.settings.overrideUserAgent.set(false);
         WebInspector.settings.emulateTouchEvents.set(false);
         WebInspector.settings.overrideDeviceOrientation.set(false);
         WebInspector.settings.overrideGeolocation.set(false);
         WebInspector.settings.overrideCSSMedia.set(false);
     },
 
-    _updateAllOverrides: function()
+    applyInitialOverrides: function()
     {
         this._userAgentChanged();
-        this._deviceMetricsChanged();
+        if (WebInspector.settings.overrideDeviceMetrics.get())
+            this.updateCanForceCompositingMode(this._deviceMetricsChanged.bind(this));
+        else
+            this._deviceMetricsChanged();
         this._deviceOrientationChanged();
         this._geolocationPositionChanged();
         this._emulateTouchEventsChanged();
         this._cssMediaChanged();
     },
 
     _userAgentChanged: function()
     {
         NetworkAgent.setUserAgentOverride(WebInspector.settings.overrideUserAgent.get() ? WebInspector.settings.userAgent.get() : "");
     },
@@ skipping 49 matching lines @@
 
     _revealOverridesTabIfNeeded: function()
     {
         if (WebInspector.settings.overrideUserAgent.get() || WebInspector.settings.overrideDeviceMetrics.get() ||
                 WebInspector.settings.overrideGeolocation.get() || WebInspector.settings.overrideDeviceOrientation.get() ||
                 WebInspector.settings.emulateTouchEvents.get() || WebInspector.settings.overrideCSSMedia.get()) {
             if (!WebInspector.settings.showEmulationViewInDrawer.get())
                 WebInspector.settings.showEmulationViewInDrawer.set(true);
             WebInspector.inspectorView.showViewInDrawer("emulation");
         }
-    }
+    },
+
+    updateCanForceCompositingMode: function(callback)
+    {
+        function apiCallback(error, result)
+        {
+            if (!error) {
+                if (!result && WebInspector.settings.overrideDeviceMetrics.get())
+                    WebInspector.settings.overrideDeviceMetrics.set(false);
+
+                if (this._canForceCompositingMode !== result) {
+                    this._canForceCompositingMode = result;
+                    this.dispatchEventToListeners(WebInspector.OverridesSupport.Events.CanForceCompositingModeChanged);
+                }
+            }
+            if (callback)
+                callback();
+        }
+        PageAgent.canForceCompositingMode(apiCallback.bind(this));
+    },
+
+    canForceCompositingMode: function()
+    {
+        return this._canForceCompositingMode;
+    },
+
+    __proto__: WebInspector.Object.prototype
 }
 
 
 /**
  * @type {WebInspector.OverridesSupport}
  */
 WebInspector.overridesSupport;