[DevTools] Move part of CommandLineAPI to native

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 589 matching lines @@
             }
 
             for (var i = 0; i < jsonMLObject.length; ++i)
                 this._substituteObjectTagsInCustomPreview(objectGroupName, jsonMLObject[i]);
         } finally {
             this._customPreviewRecursionDepth--;
         }
     },
 
     /**
-     * @return {!CommandLineAPI}
+     * @param {!Object} nativeCommandLineAPI
+     * @return {!Object}
      */
-    commandLineAPI: function()
+    installCommandLineAPI: function(nativeCommandLineAPI)
     {
-        return new CommandLineAPI(this._commandLineAPIImpl);
+        var commandLineAPIImpl = this._commandLineAPIImpl;
+        for (var member of CommandLineAPIImpl.members_)
+            nativeCommandLineAPI[member] = bind(commandLineAPIImpl[member], commandLineAPIImpl);
+
+        for (var i = 0; i < 5; ++i) {
+            var member = "$" + i;
+            nativeCommandLineAPI[member] = bind(commandLineAPIImpl._inspectedObject, commandLineAPIImpl, i);
+        }
+
+        nativeCommandLineAPI["$"].toString = function() { return "function $(selector, [startNode]) { [Command Line API] }"};

[dgozman, 2016/04/25 20:55:08]

Let's make it an explicit map.

[kozy, 2016/04/25 21:33:58]

Done.

+        nativeCommandLineAPI["$$"].toString = function() { return "function $$(selector, [startNode]) { [Command Line API] }"};
+        nativeCommandLineAPI["$x"].toString = function() { return "function $x(xpath, [startNode]) { [Command Line API] }"};
+        nativeCommandLineAPI["dir"].toString = function() { return "function dir(value) { [Command Line API] }"};
+        nativeCommandLineAPI["dirxml"].toString = function() { return "function dirxml(value) { [Command Line API] }"};
+        nativeCommandLineAPI["keys"].toString = function() { return "function keys(object) { [Command Line API] }"};
+        nativeCommandLineAPI["values"].toString = function() { return "function values(object) { [Command Line API] }"};
+        nativeCommandLineAPI["profile"].toString = function() { return "function profile(value) { [Command Line API] }"};
+        nativeCommandLineAPI["profileEnd"].toString = function() { return "function profileEnd(value) { [Command Line API] }"};
+        nativeCommandLineAPI["monitorEvents"].toString = function() { return "function monitorEvents(object, [types]) { [Command Line API] }"};
+        nativeCommandLineAPI["unmonitorEvents"].toString = function() { return "function unmonitorEvents(object, [types]) { [Command Line API] }"};
+        nativeCommandLineAPI["inspect"].toString = function() { return "function inspect(object) { [Command Line API] }"};
+        nativeCommandLineAPI["copy"].toString = function() { return "function copy(value) { [Command Line API] }"};
+        nativeCommandLineAPI["clear"].toString = function() { return "function clear() { [Command Line API] }"};
+        nativeCommandLineAPI["getEventListeners"].toString = function() { return "function getEventListeners(node) { [Command Line API] }"};
+        nativeCommandLineAPI["debug"].toString = function() { return "function debug(function) { [Command Line API] }"};
+        nativeCommandLineAPI["undebug"].toString = function() { return "function undebug(function) { [Command Line API] }"};
+        nativeCommandLineAPI["monitor"].toString = function() { return "function monitor(function) { [Command Line API] }"};
+        nativeCommandLineAPI["unmonitor"].toString = function() { return "function unmonitor(function) { [Command Line API] }"};
+        nativeCommandLineAPI["table"].toString = function() { return "function table(value) { [Command Line API] }"};
+
+        return nativeCommandLineAPI;
     },
 
     /**
      * @param {string} objectGroup
      * @return {!Object}
      */
     remoteObjectAPI: function(objectGroup)
     {
         /**
          * @suppressReceiverCheck
@@ skipping 504 matching lines @@
             return string.substr(0, leftHalf) + "\u2026" + string.substr(string.length - rightHalf, rightHalf);
         }
         return string.substr(0, maxLength) + "\u2026";
     },
 
     __proto__: null
 }
 
 /**
  * @constructor
- * @param {!CommandLineAPIImpl} commandLineAPIImpl
- */
-function CommandLineAPI(commandLineAPIImpl)
-{
-    /**
-     * @param {string} name The name of the method for which a toString method should be generated.
-     * @return {function():string}
-     */
-    function customToStringMethod(name)
-    {
-        return function()
-        {
-            var funcArgsSyntax = "";
-            try {
-                var funcSyntax = "" + commandLineAPIImpl[name];
-                funcSyntax = funcSyntax.replace(/\n/g, " ");
-                funcSyntax = funcSyntax.replace(/^function[^\(]*\(([^\)]*)\).*$/, "$1");
-                funcSyntax = funcSyntax.replace(/\s*,\s*/g, ", ");
-                funcSyntax = funcSyntax.replace(/\bopt_(\w+)\b/g, "[$1]");
-                funcArgsSyntax = funcSyntax.trim();
-            } catch (e) {
-            }
-            return "function " + name + "(" + funcArgsSyntax + ") { [Command Line API] }";
-        };
-    }
-
-    for (var i = 0; i < CommandLineAPI.members_.length; ++i) {
-        var member = CommandLineAPI.members_[i];
-        this[member] = bind(commandLineAPIImpl[member], commandLineAPIImpl);
-        this[member].toString = customToStringMethod(member);
-    }
-
-    for (var i = 0; i < 5; ++i) {
-        var member = "$" + i;
-        this[member] = bind(commandLineAPIImpl._inspectedObject, commandLineAPIImpl, i);
-    }
-    this.__proto__ = null;
-}
-
-// NOTE: Please keep the list of API methods below synchronized to that in WebInspector.RuntimeModel
-// and V8InjectedScriptHost!
-// NOTE: Argument names of these methods will be printed in the console, so use pretty names!
-/**
- * @type {!Array.<string>}
- * @const
- */
-CommandLineAPI.members_ = [
-    "$", "$$", "$x", "dir", "dirxml", "keys", "values", "profile", "profileEnd",
-    "monitorEvents", "unmonitorEvents", "inspect", "copy", "clear", "getEventListeners",
-    "debug", "undebug", "monitor", "unmonitor", "table"
-];
-
-/**
- * @constructor
  */
 function CommandLineAPIImpl()
 {
 }
 
+// NOTE: This list contains only not native Command Line API methods. For full list: V8InjectedScriptHost.

[dgozman, 2016/04/25 20:55:08]

V8InjectedScriptHost -> V8Console

[kozy, 2016/04/25 21:33:58]

Done.

+// NOTE: Argument names of these methods will be printed in the console, so use pretty names!
+/**
+ * @type {!Array.<string>}
+ * @const
+ */
+CommandLineAPIImpl.members_ = [

[dgozman, 2016/04/25 20:55:08]

nit: inline this, but keep the comment.

[kozy, 2016/04/25 21:33:58]

Done.

+    "$", "$$", "$x", "monitorEvents", "unmonitorEvents", "inspect", "copy", "getEventListeners"
+];
+
 CommandLineAPIImpl.prototype = {
     /**
      * @param {string} selector
      * @param {!Node=} opt_startNode
      * @return {*}
      */
     $: function (selector, opt_startNode)
     {
         if (this._canQuerySelectorOnNode(opt_startNode))
             return opt_startNode.querySelector(selector);
@@ skipping 41 matching lines @@
         default:
             var nodes = [];
             var node;
             while (node = result.iterateNext())
                 push(nodes, node);
             return nodes;
         }
     },
 
     /**
-     * @return {*}
-     */
-    dir: function(var_args)
-    {
-        return InjectedScriptHost.callFunction(inspectedGlobalObject.console.dir, inspectedGlobalObject.console, slice(arguments));
-    },
-
-    /**
-     * @return {*}
-     */
-    dirxml: function(var_args)
-    {
-        return InjectedScriptHost.callFunction(inspectedGlobalObject.console.dirxml, inspectedGlobalObject.console, slice(arguments));
-    },
-
-    /**
-     * @return {!Array.<string>}
-     */
-    keys: function(object)
-    {
-        return Object.keys(object);
-    },
-
-    /**
-     * @return {!Array.<*>}
-     */
-    values: function(object)
-    {
-        var result = [];
-        for (var key in object)
-            push(result, object[key]);
-        return result;
-    },
-
-    /**
-     * @return {*}
-     */
-    profile: function(opt_title)
-    {
-        return InjectedScriptHost.callFunction(inspectedGlobalObject.console.profile, inspectedGlobalObject.console, slice(arguments));
-    },
-
-    /**
-     * @return {*}
-     */
-    profileEnd: function(opt_title)
-    {
-        return InjectedScriptHost.callFunction(inspectedGlobalObject.console.profileEnd, inspectedGlobalObject.console, slice(arguments));
-    },
-
-    /**
      * @param {!Object} object
      * @param {!Array.<string>|string=} opt_types
      */
     monitorEvents: function(object, opt_types)
     {
         if (!object || !object.addEventListener || !object.removeEventListener)
             return;
         var types = this._normalizeEventTypes(opt_types);
         for (var i = 0; i < types.length; ++i) {
             object.removeEventListener(types[i], this._logEvent, false);
@@ skipping 36 matching lines @@
             } catch (e) {
                 string = toString(object);
             }
         }
 
         var hints = { copyToClipboard: true, __proto__: null };
         var remoteObject = injectedScript._wrapObject(string, "")
         InjectedScriptHost.inspect(remoteObject, hints);
     },
 
-    clear: function()
-    {
-        InjectedScriptHost.clearConsoleMessages();
-    },
-
     /**
      * @param {!Node} node
      * @return {!Object|undefined}
      */
     getEventListeners: function(node)
     {
         var result = nullifyObjectProto(InjectedScriptHost.getEventListeners(node));
         if (!result)
             return;
 
@@ skipping 28 matching lines @@
         for (var type in result) {
             var listeners = result[type];
             for (var i = 0, listener; listener = listeners[i]; ++i) {
                 listener["type"] = type;
                 listener["remove"] = removeFunc;
             }
         }
         return result;
     },
 
-    debug: function(fn)
-    {
-        InjectedScriptHost.debugFunction(fn);
-    },
-
-    undebug: function(fn)
-    {
-        InjectedScriptHost.undebugFunction(fn);
-    },
-
-    monitor: function(fn)
-    {
-        InjectedScriptHost.monitorFunction(fn);
-    },
-
-    unmonitor: function(fn)
-    {
-        InjectedScriptHost.unmonitorFunction(fn);
-    },
-
-    table: function(data, opt_columns)
-    {
-        InjectedScriptHost.callFunction(inspectedGlobalObject.console.table, inspectedGlobalObject.console, slice(arguments));
-    },
-
     /**
      * @param {number} num
      */
     _inspectedObject: function(num)
     {
         return InjectedScriptHost.inspectedObject(num);
     },
 
     /**
      * @param {!Array.<string>|string=} types
@@ skipping 29 matching lines @@
      */
     _logEvent: function(event)
     {
         inspectedGlobalObject.console.log(event.type, event);
     }
 }
 
 injectedScript._commandLineAPIImpl = new CommandLineAPIImpl();
 return injectedScript;
 })