DevTools: Dump function location when printing it in console and support inspect() for functions.

Source/devtools/front_end/ConsoleView.js

Index: Source/devtools/front_end/ConsoleView.js
diff --git a/Source/devtools/front_end/ConsoleView.js b/Source/devtools/front_end/ConsoleView.js
index 84a3c8ac571df8d0cd3112cc63efe2bb8859c6c3..11bd653a710c03b91d64799fdc942ef170182794 100644
--- a/Source/devtools/front_end/ConsoleView.js
+++ b/Source/devtools/front_end/ConsoleView.js
@@ -551,6 +551,10 @@ WebInspector.ConsoleView.prototype = {
         }
     },
 
+    /**
+     * @param {string} expression
+     * @param {boolean} showResultOnly
+     */
     evaluateUsingTextPrompt: function(expression, showResultOnly)
     {
         this._appendCommand(expression, this.prompt.text, false, showResultOnly);
@@ -571,14 +575,63 @@ WebInspector.ConsoleView.prototype = {
         this._appendCommand(str, "", true, false);
     },
 
+    /**
+     * @param {WebInspector.RemoteObject} result
+     * @param {boolean} wasThrown
+     * @param {WebInspector.ConsoleCommand} originatingCommand
+     */
     _printResult: function(result, wasThrown, originatingCommand)
     {
         if (!result)
             return;
-        var message = new WebInspector.ConsoleCommandResult(result, wasThrown, originatingCommand, this._linkifier);
-        WebInspector.console.addMessage(message);
+
+        /**
+         * @param {string=} url
+         * @param {number=} lineNumber
+         * @param {number=} columnNumber
+         */
+        function addMessage(url, lineNumber, columnNumber)
+        {
+            var message = new WebInspector.ConsoleCommandResult(result, wasThrown, originatingCommand, this._linkifier, url, lineNumber, columnNumber);
+            WebInspector.console.addMessage(message);
+        }
+
+        if (result.type !== "function") {
+            addMessage.call(this);
+            return;
+        }
+
+        DebuggerAgent.getFunctionDetails(result.objectId, didGetDetails.bind(this));
+
+        /**
+         * @param {?Protocol.Error} error
+         * @param {DebuggerAgent.FunctionDetails} response
+         */
+        function didGetDetails(error, response)
+        {
+            if (error)
+                console.error(error);

[pfeldman, 2013/08/28 15:46:15]

Return early instead?

+
+            var url;
+            var lineNumber;
+            var columnNumber;
+            var rawLocation = response ? response.location : null;
+            var script = rawLocation ? WebInspector.debuggerModel.scriptForId(rawLocation.scriptId) : null;
+            if (script && script.sourceURL) {
+                url = script.sourceURL;
+                lineNumber = rawLocation.lineNumber + 1;
+                columnNumber = rawLocation.columnNumber + 1;

[pfeldman, 2013/08/28 15:46:15]

Lets remove the mess.

+            }
+            addMessage.call(this, url, lineNumber, columnNumber);
+        }
     },
 
+    /**
+     * @param {string} text
+     * @param {string} newPromptText
+     * @param {boolean} useCommandLineAPI
+     * @param {boolean} showResultOnly
+     */
     _appendCommand: function(text, newPromptText, useCommandLineAPI, showResultOnly)
     {
         if (!showResultOnly) {
@@ -587,7 +640,12 @@ WebInspector.ConsoleView.prototype = {
         }
         this.prompt.text = newPromptText;
 
-        function printResult(result, wasThrown)
+        /**
+         * @param {WebInspector.RemoteObject} result
+         * @param {boolean} wasThrown
+         * @param {RuntimeAgent.RemoteObject=} valueResult
+         */
+        function printResult(result, wasThrown, valueResult)
         {
             if (!result)
                 return;
@@ -596,7 +654,7 @@ WebInspector.ConsoleView.prototype = {
                 this.prompt.pushHistoryItem(text);
                 WebInspector.settings.consoleHistory.set(this.prompt.historyData.slice(-30));
             }
-            
+
             this._printResult(result, wasThrown, commandMessage);
         }
         WebInspector.runtimeModel.evaluate(text, "console", useCommandLineAPI, false, false, true, printResult.bind(this));
@@ -1016,6 +1074,9 @@ WebInspector.ConsoleCommand.prototype = {
         this._element.replaceChild(this._formattedCommand, highlightedMessage);
     },
 
+    /**
+     * @param {RegExp} regexObject
+     */
     highlightSearchResults: function(regexObject)
     {
         regexObject.lastIndex = 0;
@@ -1029,6 +1090,9 @@ WebInspector.ConsoleCommand.prototype = {
         this._element.scrollIntoViewIfNeeded();
     },
 
+    /**
+     * @param {RegExp} regexObject
+     */
     matchesRegex: function(regexObject)
     {
         regexObject.lastIndex = 0;
@@ -1061,16 +1125,19 @@ WebInspector.ConsoleCommand.prototype = {
 /**
  * @extends {WebInspector.ConsoleMessageImpl}
  * @constructor
- * @param {boolean} result
+ * @param {WebInspector.RemoteObject} result
  * @param {boolean} wasThrown
  * @param {WebInspector.ConsoleCommand} originatingCommand
  * @param {WebInspector.Linkifier} linkifier
+ * @param {string=} url
+ * @param {number=} lineNumber
+ * @param {number=} columnNumber
  */
-WebInspector.ConsoleCommandResult = function(result, wasThrown, originatingCommand, linkifier)
+WebInspector.ConsoleCommandResult = function(result, wasThrown, originatingCommand, linkifier, url, lineNumber, columnNumber)
 {
     var level = (wasThrown ? WebInspector.ConsoleMessage.MessageLevel.Error : WebInspector.ConsoleMessage.MessageLevel.Log);
     this.originatingCommand = originatingCommand;
-    WebInspector.ConsoleMessageImpl.call(this, WebInspector.ConsoleMessage.MessageSource.JS, level, "", linkifier, WebInspector.ConsoleMessage.MessageType.Result, undefined, undefined, undefined, undefined, [result]);
+    WebInspector.ConsoleMessageImpl.call(this, WebInspector.ConsoleMessage.MessageSource.JS, level, "", linkifier, WebInspector.ConsoleMessage.MessageType.Result, url, lineNumber, columnNumber, undefined, [result]);
 }
 
 WebInspector.ConsoleCommandResult.prototype = {