[DevTools] Pass bindRemoteObject function to Custom Formatter directly

third_party/WebKit/Source/platform/v8_inspector/InjectedScriptSource.js

 /*
  * Copyright (C) 2007 Apple Inc.  All rights reserved.
  * Copyright (C) 2013 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:
  *
  * 1.  Redistributions of source code must retain the above copyright
  *     notice, this list of conditions and the following disclaimer.
@@ skipping 50 matching lines @@
  */
 function slice(array, index)
 {
     var result = [];
     for (var i = index || 0, j = 0; i < array.length; ++i, ++j)
         result[j] = array[i];
     return result;
 }
 
 /**
- * @param {!Array.<T>} array1
- * @param {!Array.<T>} array2
- * @return {!Array.<T>}
- * @template T
- */
-function concat(array1, array2)
-{
-    var result = [];
-    for (var i = 0; i < array1.length; ++i)
-        push(result, array1[i]);
-    for (var i = 0; i < array2.length; ++i)
-        push(result, array2[i]);
-    return result;
-}
-
-/**
  * @param {*} obj
  * @return {string}
  * @suppress {uselessCode}
  */
 function toString(obj)
 {
     // We don't use String(obj) because String16 could be overridden.
     // Also the ("" + obj) expression may throw.
     try {
         return "" + obj;
     } catch (e) {
         var name = InjectedScriptHost.internalConstructorName(obj) || InjectedScriptHost.subtype(obj) || (typeof obj);
         return "#<" + name + ">";
     }
 }
 
 /**
  * @param {*} obj
  * @return {string}
  */
 function toStringDescription(obj)
 {
     if (typeof obj === "number" && obj === 0 && 1 / obj < 0)
         return "-0"; // Negative zero.
     return toString(obj);
 }
 
 /**
- * Please use this bind, not the one from Function.prototype
- * @param {function(...)} func
- * @param {?Object} thisObject
- * @param {...} var_args
- * @return {function(...)}
- */
-function bind(func, thisObject, var_args)
-{
-    var args = slice(arguments, 2);
-
-    /**
-     * @param {...} var_args
-     */
-    function bound(var_args)
-    {
-        return InjectedScriptHost.suppressWarningsAndCallFunction(func, thisObject, concat(args, slice(arguments)));
-    }
-    bound.toString = function()
-    {
-        return "bound: " + toString(func);
-    };
-    return bound;
-}
-
-/**
  * @param {T} obj
  * @return {T}
  * @template T
  */
 function nullifyObjectProto(obj)
 {
     if (obj && typeof obj === "object")
         obj.__proto__ = null;
     return obj;
 }
@@ skipping 45 matching lines @@
  * @param {*} obj
  * @return {boolean}
  */
 function isSymbol(obj)
 {
     var type = typeof obj;
     return (type === "symbol");
 }
 
 /**
- * @param {string} str
- * @param {string} searchElement
- * @param {number=} fromIndex
- * @return {number}
- */
-function indexOf(str, searchElement, fromIndex)
-{
-    var len = str.length;
-    var n = fromIndex || 0;
-    var k = max(n >= 0 ? n : len + n, 0);
-
-    while (k < len) {
-        if (str[k] === searchElement)
-            return k;
-        ++k;
-    }
-    return -1;
-}
-
-/**
  * DOM Attributes which have observable side effect on getter, in the form of
  *   {interfaceName1: {attributeName1: true,
  *                     attributeName2: true,
  *                     ...},
  *    interfaceName2: {...},
  *    ...}
  * @type {!Object<string, !Object<string, boolean>>}
  * @const
  */
 var domAttributesWithObservableSideEffectOnGet = nullifyObjectProto({});
@@ skipping 392 matching lines @@
             ["monitorEvents",   "function monitorEvents(object, [types]) { [Command Line API] }"],
             ["unmonitorEvents", "function unmonitorEvents(object, [types]) { [Command Line API] }"],
             ["getEventListeners", "function getEventListeners(node) { [Command Line API] }"]
         ]);
         for (let entry of functionToStringMap)
             nativeCommandLineAPI[entry[0]].toString = (() => entry[1]);
         return nativeCommandLineAPI;
     },
 
     /**
-     * @param {string} objectGroup
-     * @return {!Object}
-     */
-    remoteObjectAPI: function(objectGroup)
-    {
-        /**
-         * @suppressReceiverCheck
-         * @param {*} object
-         * @param {boolean=} forceValueType
-         * @param {boolean=} generatePreview
-         * @param {?Array.<string>=} columnNames
-         * @param {boolean=} isTable
-         * @param {*=} customObjectConfig
-         * @return {!RuntimeAgent.RemoteObject}
-         * @this {InjectedScript}
-         */
-        function wrap(object, forceValueType, generatePreview, columnNames, isTable, customObjectConfig)
-        {
-            return this._wrapObject(object, objectGroup, forceValueType, generatePreview, columnNames, isTable, false, customObjectConfig);
-        }
-        return { bindRemoteObject: bind(wrap, this), __proto__: null};
-    },
-
-    /**
      * @param {*} object
      * @return {boolean}
      */
     _isDefined: function(object)
     {
         return !!object || this._isHTMLAllCollection(object);
     },
 
     /**
      * @param {*} object
@@ skipping 197 matching lines @@
     _customPreview: function(object, objectGroupName, customObjectConfig)
     {
         /**
          * @param {!Error} error
          */
         function logError(error)
         {
             Promise.resolve().then(inspectedGlobalObject.console.error.bind(inspectedGlobalObject.console, "Custom Formatter Failed: " + error.message));
         }
 
+        /**
+         * @suppressReceiverCheck
+         * @param {*} object
+         * @param {*=} customObjectConfig
+         * @return {*}
+         */
+        function wrap(object, customObjectConfig)
+        {
+            return InjectedScriptHost.suppressWarningsAndCallFunction(injectedScript._wrapObject, injectedScript, [ object, objectGroupName, false, false, null, false, false, customObjectConfig ]);
+        }
+
         try {
             var formatters = inspectedGlobalObject["devtoolsFormatters"];
             if (!formatters || !isArrayLike(formatters))
                 return null;
 
             for (var i = 0; i < formatters.length; ++i) {
                 try {
                     var formatted = formatters[i].header(object, customObjectConfig);
                     if (!formatted)
                         continue;
 
                     var hasBody = formatters[i].hasBody(object, customObjectConfig);
                     injectedScript._substituteObjectTagsInCustomPreview(objectGroupName, formatted);
                     var formatterObjectId = injectedScript._bind(formatters[i], objectGroupName);
-                    var result = {header: JSON.stringify(formatted), hasBody: !!hasBody, formatterObjectId: formatterObjectId};
+                    var bindRemoteObjectFunctionId = injectedScript._bind(wrap, objectGroupName);

[dgozman, 2016/05/17 22:05:38]

Let's store this on InjectedScript.

[kozy, 2016/05/17 22:35:26]

We can't do it.
I need to bind it to objectGroupName of current object.

+                    var result = {header: JSON.stringify(formatted), hasBody: !!hasBody, formatterObjectId: formatterObjectId, bindRemoteObjectFunctionId: bindRemoteObjectFunctionId};
                     if (customObjectConfig)
                         result["configObjectId"] = injectedScript._bind(customObjectConfig, objectGroupName);
                     return result;
                 } catch (e) {
                     logError(e);
                 }
             }
         } catch (e) {
             logError(e);
         }
@@ skipping 427 matching lines @@
 /**
  * @param {!Event} event
  */
 CommandLineAPIImpl._logEvent = function(event)
 {
     inspectedGlobalObject.console.log(event.type, event);
 }
 
 return injectedScript;
 })