Add ability to toggle passive state on an individual event listener.

third_party/WebKit/Source/devtools/front_end/sdk/RuntimeModel.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 703 matching lines @@
  * @constructor
  * @extends {WebInspector.SDKObject}
  * @param {!WebInspector.Target} target
  * @param {string} type
  * @param {boolean} useCapture
  * @param {boolean} passive
  * @param {?WebInspector.RemoteObject} handler
  * @param {?WebInspector.RemoteObject} originalHandler
  * @param {!WebInspector.DebuggerModel.Location} location
  * @param {?WebInspector.RemoteObject} removeFunction
+ * @param {?WebInspector.RemoteObject} togglePassiveFunction
  * @param {string=} listenerType
  */
-WebInspector.EventListener = function(target, type, useCapture, passive, handler, originalHandler, location, removeFunction, listenerType)
+WebInspector.EventListener = function(target, type, useCapture, passive, handler, originalHandler, location, removeFunction, togglePassiveFunction, listenerType)
 {
     WebInspector.SDKObject.call(this, target);
     this._type = type;
     this._useCapture = useCapture;
     this._passive = passive;
     this._handler = handler;
     this._originalHandler = originalHandler || handler;
     this._location = location;
     var script = location.script();
     this._sourceURL = script ? script.contentURL() : "";
     this._removeFunction = removeFunction;
+    this._togglePassiveFunction = togglePassiveFunction;
     this._listenerType = listenerType || "normal";
 }
 
 WebInspector.EventListener.prototype = {
     /**
      * @return {string}
      */
     type: function()
     {
         return this._type;
@@ skipping 86 matching lines @@
              * @param {boolean} passive
              */
             function callCustomRemove(func, type, listener, useCapture, passive)
             {
                 func.call(null, type, listener, useCapture, passive);
             }
         }
     },
 
     /**
+     * @return {?WebInspector.RemoteObject}
+     */
+    togglePassiveFunction: function()

[caseq, 2016/06/01 17:48:11]

I don't think this is something we should expose -- the remote object itself is
not too useful for the caller, especially considering the amount of boilerplate
necessary around the call below. Let's rename to canTogglePassive() and it would
be a nice opportunity to encapsulate isScrollBlockingType() as well. Ditto for
removeFunction(), I guess.

[dtapuska, 2016/06/01 17:54:37]

About merging them... I wanted to separate the choice of toggling passive to the
UI. You can toggle any listener as
passive or not. But really we just restrict the UI so it doesn't confuse the
users because toggling a
non scroll blocking listener doesn't really have much perf benefit.

+    {
+        return this._togglePassiveFunction;
+    },
+
+    /**
+     * @return {!Promise<undefined>}
+     */
+    togglePassive: function()
+    {
+        if (!this._togglePassiveFunction)
+            return Promise.resolve();
+        return new Promise(promiseConstructor.bind(this));
+
+        /**
+         * @param {function()} success
+         * @this {WebInspector.EventListener}
+         */
+        function promiseConstructor(success)
+        {
+            this._togglePassiveFunction.callFunction(callCustomRemove, [
+                WebInspector.RemoteObject.toCallArgument(this._togglePassiveFunction),
+                WebInspector.RemoteObject.toCallArgument(this._type),
+                WebInspector.RemoteObject.toCallArgument(this._originalHandler),
+                WebInspector.RemoteObject.toCallArgument(this._useCapture),
+                WebInspector.RemoteObject.toCallArgument(this._passive),
+            ], success);
+
+            /**
+             * @param {function(string, function(), boolean, boolean)} func
+             * @param {string} type
+             * @param {function()} listener
+             * @param {boolean} useCapture
+             * @param {boolean} passive
+             */
+            function callCustomRemove(func, type, listener, useCapture, passive)

[caseq, 2016/06/01 17:48:11]

copy-paste hurts :-) Perhaps, extract something like
_dispatchEventListenerHelper()?

[dtapuska, 2016/06/02 16:02:11]

Done.

+            {
+                func.call(null, type, listener, useCapture, passive);
+            }
+        }
+    },
+
+    /**
      * @return {string}
      */
     listenerType: function()
     {
         return this._listenerType;
     },
 
     /**
      * @param {string} listenerType
      */
     setListenerType: function(listenerType)
     {
         this._listenerType = listenerType;
     },
 
+    /**
+     * @return {boolean}
+     */
+    isScrollBlockingType: function()
+    {
+        return this._type == "touchstart" ||

[caseq, 2016/06/01 17:48:11]

s/==/===/g

[dtapuska, 2016/06/02 16:02:11]

Done.

+               this._type == "touchmove" ||
+               this._type == "mousewheel" ||
+               this._type == "wheel";
+    },
+
     __proto__: WebInspector.SDKObject.prototype
 }