DevTools: Add context menu option for objects to save to temp variable.

Source/devtools/front_end/SourcesPanel.js

 /*
  * Copyright (C) 2008 Apple Inc. All Rights Reserved.
  * Copyright (C) 2011 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.
  * 2. Redistributions in binary form must reproduce the above copyright
@@ skipping 1334 matching lines @@
         WebInspector.RevisionHistoryView.showHistory(uiSourceCode);
     },
 
     /**
      * @param {!WebInspector.ContextMenu} contextMenu
      * @param {!Object} target
      */
     appendApplicableItems: function(event, contextMenu, target)
     {
         this._appendUISourceCodeItems(contextMenu, target);
-        this._appendFunctionItems(contextMenu, target);
+        this._appendRemoteObjectItems(contextMenu, target);
     },
 
     _suggestReload: function()
     {
         if (window.confirm(WebInspector.UIString("It is recommended to restart inspector after making these changes. Would you like to restart it?")))
             WebInspector.reload();
     },
 
     /**
      * @param {!WebInspector.UISourceCode} uiSourceCode
@@ skipping 82 matching lines @@
 
         var uiSourceCode = /** @type {!WebInspector.UISourceCode} */ (target);
         contextMenu.appendItem(WebInspector.UIString(WebInspector.useLowerCaseMenuTitles() ? "Local modifications\u2026" : "Local Modifications\u2026"), this._showLocalHistory.bind(this, uiSourceCode));
         this._appendUISourceCodeMappingItems(contextMenu, uiSourceCode);
     },
 
     /**
      * @param {!WebInspector.ContextMenu} contextMenu
      * @param {!Object} target
      */
-    _appendFunctionItems: function(contextMenu, target)
+    _appendRemoteObjectItems: function(contextMenu, target)
     {
         if (!(target instanceof WebInspector.RemoteObject))
             return;
         var remoteObject = /** @type {!WebInspector.RemoteObject} */ (target);
-        if (remoteObject.type !== "function")
-            return;
+        contextMenu.appendItem(WebInspector.UIString(WebInspector.useLowerCaseMenuTitles() ? "Save to temp variable" : "Save to Temp Variable"), this._saveToTempVariable.bind(this, remoteObject));
+        if (remoteObject.type === "function")
+            contextMenu.appendItem(WebInspector.UIString(WebInspector.useLowerCaseMenuTitles() ? "Show function definition" : "Show Function Definition"), this._showFunctionDefinition.bind(this, remoteObject));
+    },
 
+    /**
+     * @param {!WebInspector.RemoteObject} remoteObject
+     */
+    _saveToTempVariable: function(remoteObject)
+    {
+        /**
+         * @param {?WebInspector.RemoteObject} result
+         * @param {?string=} error
+         * @this {WebInspector.SourcesPanel}
+         */
+        function failedToSave(result, error)
+        {
+            --this._pendingSavesToTemp;
+            if (result) {
+                error = error || result.description;
+                result.release();
+            }
+            var message = "Failed to save to temp variable" + (error ? ": " + error : "");

[pfeldman, 2014/03/17 15:36:43]

UIString()

+            WebInspector.console.showErrorMessage(message)
+        }
+
+        /**
+         * @param {!WebInspector.RemoteObject} global
+         * @param {string} name
+         * @param {?Protocol.Error} error
+         * @this {WebInspector.SourcesPanel}
+         */
+        function didSave(global, name, error)
+        {
+            global.release();
+            if (error) {
+                failedToSave.call(this, null, error);
+            } else {
+                --this._pendingSavesToTemp;
+                WebInspector.console.evaluate(name);
+            }
+        }
+
+        /**
+         * @param {!WebInspector.RemoteObject} global
+         * @param {?WebInspector.RemoteObject} result
+         * @param {boolean=} wasThrown
+         * @this {WebInspector.SourcesPanel}
+         */
+        function didGetTempVarName(global, result, wasThrown)
+        {
+            if (wasThrown || !result || result.type !== "string") {
+                global.release();
+                failedToSave.call(this, result);
+                return;
+            }
+
+            var name = /** @type {string} */ (result.value);
+            global.setObjectPropertyValue(name, remoteObject, didSave.bind(this, global, name));
+        }
+
+        /**
+         * @param {?WebInspector.RemoteObject} global
+         * @param {boolean=} wasThrown
+         * @this {WebInspector.SourcesPanel}
+         */
+        function didGetGlobalObject(global, wasThrown)
+        {
+            /** @this {Window} */
+            function tempVarName(skip)
+            {
+                var prefix = "temp";
+                var index = 1;
+                while (skip-- >= 0) {
+                    while ((prefix + index) in this)
+                        ++index;
+                }
+                return prefix + index;
+            }
+
+            var skip = this._pendingSavesToTemp || 0;
+            this._pendingSavesToTemp = skip + 1;
+
+            if (wasThrown || !global) {
+                failedToSave.call(this, global);
+                return;
+            }
+            global.callFunction(tempVarName, [{ value: skip }], didGetTempVarName.bind(this, global));
+        }
+
+        WebInspector.runtimeModel.evaluate("window", "", false, true, false, false, didGetGlobalObject.bind(this));

[pfeldman, 2014/03/17 15:36:43]

Please implement it top-bottom for better readability.

+    },
+
+    /**
+     * @param {!WebInspector.RemoteObject} remoteObject
+     */
+    _showFunctionDefinition: function(remoteObject)
+    {
         /**
          * @param {?Protocol.Error} error
          * @param {!DebuggerAgent.FunctionDetails} response
          * @this {WebInspector.SourcesPanel}
          */
-        function didGetDetails(error, response)
+        function didGetFunctionDetails(error, response)
         {
             if (error) {
                 console.error(error);
                 return;
             }
 
             var uiLocation = WebInspector.debuggerModel.rawLocationToUILocation(response.location);
             if (!uiLocation)
                 return;
 
             this.showUILocation(uiLocation, true);
         }
-
-        /**
-         * @this {WebInspector.SourcesPanel}
-         */
-        function revealFunction()
-        {
-            DebuggerAgent.getFunctionDetails(remoteObject.objectId, didGetDetails.bind(this));
-        }
-
-        contextMenu.appendItem(WebInspector.UIString(WebInspector.useLowerCaseMenuTitles() ? "Show function definition" : "Show Function Definition"), revealFunction.bind(this));
+        DebuggerAgent.getFunctionDetails(remoteObject.objectId, didGetFunctionDetails.bind(this));
     },
 
     showGoToSourceDialog: function()
     {
         var uiSourceCodes = this._editorContainer.historyUISourceCodes();
         /** @type {!Map.<!WebInspector.UISourceCode, number>} */
         var defaultScores = new Map();
         for (var i = 1; i < uiSourceCodes.length; ++i) // Skip current element
             defaultScores.put(uiSourceCodes[i], uiSourceCodes.length - i);
         WebInspector.OpenResourceDialog.show(this, this.editorView.mainElement(), undefined, defaultScores);
@@ skipping 286 matching lines @@
 {
 }
 
 WebInspector.SourcesPanel.EditorAction.prototype = {
     /**
      * @param {!WebInspector.SourcesPanel} panel
      * @return {!Element}
      */
     button: function(panel) { }
 }