DevTools: introduce WebInspector.Throttler

Source/devtools/front_end/sdk/OverridesSupport.js

Index: Source/devtools/front_end/sdk/OverridesSupport.js
diff --git a/Source/devtools/front_end/sdk/OverridesSupport.js b/Source/devtools/front_end/sdk/OverridesSupport.js
index 8d46a0cde1002f1fbfdd9dbd296d0cb6e27801e9..2c7f5523263958ea9487db507b89034c182090a6 100644
--- a/Source/devtools/front_end/sdk/OverridesSupport.js
+++ b/Source/devtools/front_end/sdk/OverridesSupport.js
@@ -42,6 +42,7 @@ WebInspector.OverridesSupport = function(responsiveDesignAvailable)
     this._userAgent = "";
     this._pageResizer = null;
     this._initialized = false;
+    this._deviceMetricsThrottler = new WebInspector.Throttler(0);

[vsevik, 2014/06/09 12:32:58]

why not 100?

[lushnikov, 2014/06/09 13:07:26]

Throttler is used here as a message serializer that sends requests one-by-one to
backend.

     WebInspector.targetManager.observeTargets(this);
     this._responsiveDesignAvailable = responsiveDesignAvailable;
 }
@@ -623,10 +624,9 @@ WebInspector.OverridesSupport.prototype = {
         var overrideDeviceResolution = this.settings.overrideDeviceResolution.get();
         var emulationEnabled = overrideDeviceResolution || this.settings.emulateViewport.get();
         if (responsiveDesignAvailableAndDisabled || !emulationEnabled) {
-            PageAgent.clearDeviceMetricsOverride(apiCallback.bind(this));
+            this._deviceMetricsThrottler.schedule(clearDeviceMetricsOverride.bind(this), true);
             if (this._pageResizer && !emulationEnabled)
                 this._pageResizer.update(0, 0, 0);
-            this.maybeHasActiveOverridesChanged();
             return;
         }
 
@@ -651,47 +651,42 @@ WebInspector.OverridesSupport.prototype = {
             }
         }
 
-        // Do not emulate resolution more often than 10Hz.
-        this._setDeviceMetricsTimers = (this._setDeviceMetricsTimers || 0) + 1;
-        if (overrideWidth || overrideHeight)
-            setTimeout(setDeviceMetricsOverride.bind(this), 100);
-        else
-            setDeviceMetricsOverride.call(this);

[vsevik, 2014/06/09 12:32:57]

This one is not called synchronously anymore, could this cause a problem?

[lushnikov, 2014/06/09 13:07:26]

That's intentional, should be fine here.

+        var immediately = !overrideWidth && !overrideHeight;
+        this._deviceMetricsThrottler.schedule(setDeviceMetricsOverride.bind(this), immediately);
 
         /**
+         * @param {!WebInspector.Throttler.FinishCallback} finishCallback
          * @this {WebInspector.OverridesSupport}
          */
-        function setDeviceMetricsOverride()
+        function setDeviceMetricsOverride(finishCallback)
         {
-            // Drop heavy intermediate commands.
-            this._setDeviceMetricsTimers--;
-            var isExpensive = overrideWidth || overrideHeight;
-            if (isExpensive && this._setDeviceMetricsTimers) {
-                var commandThreshold = 100;
-                var time = window.performance.now();
-                if (time - this._lastExpensivePageAgentCommandTime < commandThreshold)
-                    return;
-                this._lastExpensivePageAgentCommandTime = time;
-            }
-
             PageAgent.setDeviceMetricsOverride(
                 overrideWidth, overrideHeight, this.settings.deviceScaleFactor.get(),
                 this.settings.emulateViewport.get(), this._pageResizer ? false : this.settings.deviceFitWindow.get(),
                 this.settings.deviceTextAutosizing.get(), this._fontScaleFactor(overrideWidth || dipWidth, overrideHeight || dipHeight),
-                apiCallback.bind(this));
+                apiCallback.bind(this, finishCallback));
         }
 
-        this.maybeHasActiveOverridesChanged();
+        /**
+         * @param {!WebInspector.Throttler.FinishCallback} finishCallback
+         * @this {WebInspector.OverridesSupport}
+         */
+        function clearDeviceMetricsOverride(finishCallback)
+        {
+            PageAgent.clearDeviceMetricsOverride(apiCallback.bind(this, finishCallback));
+        }
 
         /**
+         * @param {!WebInspector.Throttler.FinishCallback} finishCallback
          * @param {?Protocol.Error} error
          * @this {WebInspector.OverridesSupport}
          */
-        function apiCallback(error)
+        function apiCallback(finishCallback, error)
         {
             if (error) {
                 this._updateDeviceMetricsWarningMessage(WebInspector.UIString("Screen emulation is not available on this page."));
                 this._deviceMetricsOverrideAppliedForTest();
+                finishCallback();
                 return;
             }
 
@@ -702,6 +697,8 @@ WebInspector.OverridesSupport.prototype = {
             this._overrideDeviceResolution = overrideDeviceResolution;
             this._emulateViewportEnabled = viewportEnabled;
             this._deviceMetricsOverrideAppliedForTest();
+            this.maybeHasActiveOverridesChanged();
+            finishCallback();
         }
     },