Web Inspector: Should remove JSDoc comment blocks.
authorcommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 16 May 2014 03:32:19 +0000 (03:32 +0000)
committercommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 16 May 2014 03:32:19 +0000 (03:32 +0000)
https://bugs.webkit.org/show_bug.cgi?id=132984

Patch by Jono Wells <jonowells@apple.com> on 2014-05-15
Reviewed by Timothy Hatcher.

Remove JSDoc comment blocks. These are no longer necessary.

* UserInterface/Base/Main.js:
* UserInterface/Base/Object.js:
* UserInterface/Controllers/DOMTreeManager.js:
* UserInterface/Models/CSSCompletions.js:
* UserInterface/Models/DOMNode.js:
* UserInterface/Models/SourceMap.js:
* UserInterface/Protocol/InspectorBackend.js:
* UserInterface/Protocol/RemoteObject.js:
* UserInterface/Views/ApplicationCacheFrameContentView.js:
* UserInterface/Views/ConsoleMessage.js:
* UserInterface/Views/ConsoleMessageImpl.js:
* UserInterface/Views/ContextMenu.js:
* UserInterface/Views/DOMTreeElement.js:
* UserInterface/Views/DOMTreeOutline.js:
* UserInterface/Views/DOMTreeUpdater.js:
* UserInterface/Views/DataGrid.js:
* UserInterface/Views/DatabaseContentView.js:
* UserInterface/Views/EditingSupport.js:
* UserInterface/Views/ObjectPropertiesSection.js:
* UserInterface/Views/Popover.js:
* UserInterface/Views/TreeOutline.js:
JSDoc style comment blocks removed.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@168919 268f45cc-cd09-0410-ab3c-d52691b4dbfc

22 files changed:
Source/WebInspectorUI/ChangeLog
Source/WebInspectorUI/UserInterface/Base/Main.js
Source/WebInspectorUI/UserInterface/Base/Object.js
Source/WebInspectorUI/UserInterface/Controllers/DOMTreeManager.js
Source/WebInspectorUI/UserInterface/Models/CSSCompletions.js
Source/WebInspectorUI/UserInterface/Models/DOMNode.js
Source/WebInspectorUI/UserInterface/Models/SourceMap.js
Source/WebInspectorUI/UserInterface/Protocol/InspectorBackend.js
Source/WebInspectorUI/UserInterface/Protocol/RemoteObject.js
Source/WebInspectorUI/UserInterface/Views/ApplicationCacheFrameContentView.js
Source/WebInspectorUI/UserInterface/Views/ConsoleMessage.js
Source/WebInspectorUI/UserInterface/Views/ConsoleMessageImpl.js
Source/WebInspectorUI/UserInterface/Views/ContextMenu.js
Source/WebInspectorUI/UserInterface/Views/DOMTreeElement.js
Source/WebInspectorUI/UserInterface/Views/DOMTreeOutline.js
Source/WebInspectorUI/UserInterface/Views/DOMTreeUpdater.js
Source/WebInspectorUI/UserInterface/Views/DataGrid.js
Source/WebInspectorUI/UserInterface/Views/DatabaseContentView.js
Source/WebInspectorUI/UserInterface/Views/EditingSupport.js
Source/WebInspectorUI/UserInterface/Views/ObjectPropertiesSection.js
Source/WebInspectorUI/UserInterface/Views/Popover.js
Source/WebInspectorUI/UserInterface/Views/TreeOutline.js

index ff0ede8..f06f970 100644 (file)
@@ -1,3 +1,35 @@
+2014-05-15  Jono Wells  <jonowells@apple.com>
+
+        Web Inspector: Should remove JSDoc comment blocks.
+        https://bugs.webkit.org/show_bug.cgi?id=132984
+
+        Reviewed by Timothy Hatcher.
+
+        Remove JSDoc comment blocks. These are no longer necessary.
+
+        * UserInterface/Base/Main.js:
+        * UserInterface/Base/Object.js:
+        * UserInterface/Controllers/DOMTreeManager.js:
+        * UserInterface/Models/CSSCompletions.js:
+        * UserInterface/Models/DOMNode.js:
+        * UserInterface/Models/SourceMap.js:
+        * UserInterface/Protocol/InspectorBackend.js:
+        * UserInterface/Protocol/RemoteObject.js:
+        * UserInterface/Views/ApplicationCacheFrameContentView.js:
+        * UserInterface/Views/ConsoleMessage.js:
+        * UserInterface/Views/ConsoleMessageImpl.js:
+        * UserInterface/Views/ContextMenu.js:
+        * UserInterface/Views/DOMTreeElement.js:
+        * UserInterface/Views/DOMTreeOutline.js:
+        * UserInterface/Views/DOMTreeUpdater.js:
+        * UserInterface/Views/DataGrid.js:
+        * UserInterface/Views/DatabaseContentView.js:
+        * UserInterface/Views/EditingSupport.js:
+        * UserInterface/Views/ObjectPropertiesSection.js:
+        * UserInterface/Views/Popover.js:
+        * UserInterface/Views/TreeOutline.js:
+        JSDoc style comment blocks removed.
+
 2014-05-13  Radu Stavila  <stavila@adobe.com>
 
         Web Inspector: Restore regionLayoutUpdated event in iOS7 inspector and remove regionOversetChanged
index 231a118..a4a6cce 100644 (file)
@@ -1696,12 +1696,6 @@ WebInspector.redo = function()
     DOMAgent.redo();
 }
 
-/**
- * @param {Element} element
- * @param {Array.<Object>} resultRanges
- * @param {string} styleClass
- * @param {Array.<Object>=} changes
- */
 WebInspector.highlightRangesWithStyleClass = function(element, resultRanges, styleClass, changes)
 {
     changes = changes || [];
index 0321875..e582feb 100644 (file)
  * OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
  */
 
-/**
- * @constructor
- */
 WebInspector.Object = function()
 {
 }
 
-/**
- * @param {function} constructor
- */
 WebInspector.Object.addConstructorFunctions = function(subclassConstructor)
 {
     // Copies the relevant functions the subclass constructor.
@@ -46,11 +40,6 @@ WebInspector.Object.addConstructorFunctions = function(subclassConstructor)
     }
 }
 
-/**
- * @param {string} eventType
- * @param {function(WebInspector.Event)} listener
- * @param {Object=} thisObject
- */
 WebInspector.Object.addEventListener = function(eventType, listener, thisObject)
 {
     thisObject = thisObject || null;
@@ -79,11 +68,6 @@ WebInspector.Object.addEventListener = function(eventType, listener, thisObject)
     listeners.push({thisObject: thisObject, listener: listener});
 };
 
-/**
- * @param {string} eventType
- * @param {function(WebInspector.Event)} listener
- * @param {Object=} thisObject
- */
 WebInspector.Object.removeEventListener = function(eventType, listener, thisObject)
 {
     eventType = eventType || null;
@@ -122,10 +106,6 @@ WebInspector.Object.removeAllListeners = function()
     delete this._listeners;
 };
 
-/**
- * @param {string} eventType
- * @return {boolean}
- */
 WebInspector.Object.hasEventListeners = function(eventType)
 {
     if (!this._listeners || !this._listeners[eventType])
@@ -136,33 +116,14 @@ WebInspector.Object.hasEventListeners = function(eventType)
 WebInspector.Object.prototype = {
     constructor: WebInspector.Object,
 
-    /**
-     * @param {string} eventType
-     * @param {function(WebInspector.Event)} listener
-     * @param {Object=} thisObject
-     */
     addEventListener: WebInspector.Object.addEventListener,
 
-    /**
-     * @param {string} eventType
-     * @param {function(WebInspector.Event)} listener
-     * @param {Object=} thisObject
-     */
     removeEventListener: WebInspector.Object.removeEventListener,
 
     removeAllListeners: WebInspector.Object.removeAllListeners,
 
-    /**
-     * @param {string} eventType
-     * @return {boolean}
-     */
     hasEventListeners: WebInspector.Object.hasEventListeners,
 
-    /**
-     * @param {string} eventType
-     * @param {*=} eventData
-     * @return {boolean}
-     */
     dispatchEventToListeners: function(eventType, eventData)
     {
         var event = new WebInspector.Event(this, eventType, eventData);
@@ -204,12 +165,6 @@ WebInspector.Object.prototype = {
     }
 }
 
-/**
- * @constructor
- * @param {WebInspector.Object} target
- * @param {string} type
- * @param {*=} data
- */
 WebInspector.Event = function(target, type, data)
 {
     this.target = target;
index f4b8425..5d97537 100644 (file)
  * OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
  */
 
-/**
- * @extends {WebInspector.Object}
- * @constructor
- */
 WebInspector.DOMTreeManager = function() {
-    /** @type {Object|undefined} */
     this._idToDOMNode = {};
     this._document = null;
     this._attributeLoadNodeIds = {};
     this._flows = new Map;
     this._contentNodesToFlowsMap = new Map;
-}
+};
 
 WebInspector.Object.addConstructorFunctions(WebInspector.DOMTreeManager);
 
@@ -68,9 +63,6 @@ WebInspector.DOMTreeManager._flowPayloadHashKey = function(flowPayload)
 };
 
 WebInspector.DOMTreeManager.prototype = {
-    /**
-     * @param {function(WebInspector.DOMDocument)=} callback
-     */
     requestDocument: function(callback)
     {
         if (this._document) {
@@ -86,11 +78,6 @@ WebInspector.DOMTreeManager.prototype = {
 
         this._pendingDocumentRequestCallbacks = [callback];
 
-        /**
-         * @this {WebInspector.DOMTreeManager}
-         * @param {?Protocol.Error} error
-         * @param {DOMAgent.Node} root
-         */
         function onDocumentAvailable(error, root)
         {
             if (!error)
@@ -107,29 +94,17 @@ WebInspector.DOMTreeManager.prototype = {
         DOMAgent.getDocument(onDocumentAvailable.bind(this));
     },
 
-    /**
-     * @param {RuntimeAgent.RemoteObjectId} objectId
-     * @param {function()=} callback
-     */
     pushNodeToFrontend: function(objectId, callback)
     {
         this._dispatchWhenDocumentAvailable(DOMAgent.requestNode.bind(DOMAgent, objectId), callback);
     },
 
-    /**
-     * @param {string} path
-     * @param {function(?WebInspector.DOMNode)=} callback
-     */
     pushNodeByPathToFrontend: function(path, callback)
     {
-        var callbackCast = /** @type {function(*)} */ callback;
+        var callbackCast = callback;
         this._dispatchWhenDocumentAvailable(DOMAgent.pushNodeByPathToFrontend.bind(DOMAgent, path), callbackCast);
     },
 
-    /**
-     * @param {function(*)=} callback
-     * @return {function(?Protocol.Error,*=)|undefined}
-     */
     _wrapClientCallback: function(callback)
     {
         if (!callback)
@@ -141,13 +116,9 @@ WebInspector.DOMTreeManager.prototype = {
         };
     },
 
-    /**
-     * @param {function(function()=)} func
-     * @param {function(*)=} callback
-     */
     _dispatchWhenDocumentAvailable: function(func, callback)
     {
-        var callbackWrapper = /** @type {function(?Protocol.Error, *=)} */ this._wrapClientCallback(callback);
+        var callbackWrapper = this._wrapClientCallback(callback);
 
         function onDocumentAvailable()
         {
@@ -161,11 +132,6 @@ WebInspector.DOMTreeManager.prototype = {
         this.requestDocument(onDocumentAvailable.bind(this));
     },
 
-    /**
-     * @param {DOMAgent.NodeId} nodeId
-     * @param {string} name
-     * @param {string} value
-     */
     _attributeModified: function(nodeId, name, value)
     {
         var node = this._idToDOMNode[nodeId];
@@ -176,10 +142,6 @@ WebInspector.DOMTreeManager.prototype = {
         node.dispatchEventToListeners(WebInspector.DOMNode.Event.AttributeModified, {name: name});
     },
 
-    /**
-     * @param {DOMAgent.NodeId} nodeId
-     * @param {string} name
-     */
     _attributeRemoved: function(nodeId, name)
     {
         var node = this._idToDOMNode[nodeId];
@@ -190,9 +152,6 @@ WebInspector.DOMTreeManager.prototype = {
         node.dispatchEventToListeners(WebInspector.DOMNode.Event.AttributeRemoved, {name: name});
     },
 
-    /**
-     * @param {Array.<DOMAgent.NodeId>} nodeIds
-     */
     _inlineStyleInvalidated: function(nodeIds)
     {
         for (var i = 0; i < nodeIds.length; ++i)
@@ -204,12 +163,6 @@ WebInspector.DOMTreeManager.prototype = {
 
     _loadNodeAttributes: function()
     {
-        /**
-         * @this {WebInspector.DOMTreeManager}
-         * @param {DOMAgent.NodeId} nodeId
-         * @param {?Protocol.Error} error
-         * @param {Array.<string>} attributes
-         */
         function callback(nodeId, error, attributes)
         {
             if (error) {
@@ -233,10 +186,6 @@ WebInspector.DOMTreeManager.prototype = {
         this._attributeLoadNodeIds = {};
     },
 
-    /**
-     * @param {DOMAgent.NodeId} nodeId
-     * @param {string} newValue
-     */
     _characterDataModified: function(nodeId, newValue)
     {
         var node = this._idToDOMNode[nodeId];
@@ -244,10 +193,6 @@ WebInspector.DOMTreeManager.prototype = {
         this.dispatchEventToListeners(WebInspector.DOMTreeManager.Event.CharacterDataModified, {node: node});
     },
 
-    /**
-     * @param {DOMAgent.NodeId} nodeId
-     * @return {WebInspector.DOMNode|undefined}
-     */
     nodeForId: function(nodeId)
     {
         return this._idToDOMNode[nodeId];
@@ -258,9 +203,6 @@ WebInspector.DOMTreeManager.prototype = {
         this._setDocument(null);
     },
 
-    /**
-     * @param {DOMAgent.Node} payload
-     */
     _setDocument: function(payload)
     {
         this._idToDOMNode = {};
@@ -271,18 +213,11 @@ WebInspector.DOMTreeManager.prototype = {
         this.dispatchEventToListeners(WebInspector.DOMTreeManager.Event.DocumentUpdated, this._document);
     },
 
-    /**
-     * @param {DOMAgent.Node} payload
-     */
     _setDetachedRoot: function(payload)
     {
         new WebInspector.DOMNode(this, null, false, payload);
     },
 
-    /**
-     * @param {DOMAgent.NodeId} parentId
-     * @param {Array.<DOMAgent.Node>} payloads
-     */
     _setChildNodes: function(parentId, payloads)
     {
         if (!parentId && payloads.length) {
@@ -294,10 +229,6 @@ WebInspector.DOMTreeManager.prototype = {
         parent._setChildrenPayload(payloads);
     },
 
-    /**
-     * @param {DOMAgent.NodeId} nodeId
-     * @param {number} newValue
-     */
     _childNodeCountUpdated: function(nodeId, newValue)
     {
         var node = this._idToDOMNode[nodeId];
@@ -305,11 +236,6 @@ WebInspector.DOMTreeManager.prototype = {
         this.dispatchEventToListeners(WebInspector.DOMTreeManager.Event.ChildNodeCountUpdated, node);
     },
 
-    /**
-     * @param {DOMAgent.NodeId} parentId
-     * @param {DOMAgent.NodeId} prevId
-     * @param {DOMAgent.Node} payload
-     */
     _childNodeInserted: function(parentId, prevId, payload)
     {
         var parent = this._idToDOMNode[parentId];
@@ -319,10 +245,6 @@ WebInspector.DOMTreeManager.prototype = {
         this.dispatchEventToListeners(WebInspector.DOMTreeManager.Event.NodeInserted, {node: node, parent: parent});
     },
 
-    /**
-     * @param {DOMAgent.NodeId} parentId
-     * @param {DOMAgent.NodeId} nodeId
-     */
     _childNodeRemoved: function(parentId, nodeId)
     {
         var parent = this._idToDOMNode[parentId];
@@ -332,9 +254,6 @@ WebInspector.DOMTreeManager.prototype = {
         this.dispatchEventToListeners(WebInspector.DOMTreeManager.Event.NodeRemoved, {node:node, parent: parent});
     },
 
-    /**
-     * @param {DOMAgent.Node} node
-     */
     _unbind: function(node)
     {
         this._removeContentNodeFromFlowIfNeeded(node);
@@ -344,9 +263,6 @@ WebInspector.DOMTreeManager.prototype = {
             this._unbind(node.children[i]);
     },
 
-    /**
-     * @param {number} nodeId
-     */
     inspectElement: function(nodeId)
     {
         var node = this._idToDOMNode[nodeId];
@@ -373,19 +289,10 @@ WebInspector.DOMTreeManager.prototype = {
         remoteObject.pushNodeToFrontend(nodeAvailable.bind(this));
     },
 
-    /**
-     * @param {string} query
-     * @param {function(number)} searchCallback
-     */
     performSearch: function(query, searchCallback)
     {
         this.cancelSearch();
 
-        /**
-         * @param {?Protocol.Error} error
-         * @param {string} searchId
-         * @param {number} resultsCount
-         */
         function callback(error, searchId, resultsCount)
         {
             this._searchId = searchId;
@@ -394,17 +301,9 @@ WebInspector.DOMTreeManager.prototype = {
         DOMAgent.performSearch(query, callback.bind(this));
     },
 
-    /**
-     * @param {number} index
-     * @param {?function(DOMAgent.Node)} callback
-     */
     searchResult: function(index, callback)
     {
         if (this._searchId) {
-            /**
-             * @param {?Protocol.Error} error
-             * @param {Array.<number>} nodeIds
-             */
             function mycallback(error, nodeIds)
             {
                 if (error) {
@@ -430,32 +329,18 @@ WebInspector.DOMTreeManager.prototype = {
         }
     },
 
-    /**
-     * @param {DOMAgent.NodeId} nodeId
-     * @param {string} selectors
-     * @param {function(?DOMAgent.NodeId)=} callback
-     */
     querySelector: function(nodeId, selectors, callback)
     {
-        var callbackCast = /** @type {function(*)|undefined} */callback;
+        var callbackCast = callback;
         DOMAgent.querySelector(nodeId, selectors, this._wrapClientCallback(callbackCast));
     },
 
-    /**
-     * @param {DOMAgent.NodeId} nodeId
-     * @param {string} selectors
-     * @param {function(?Array.<DOMAgent.NodeId>)=} callback
-     */
     querySelectorAll: function(nodeId, selectors, callback)
     {
-        var callbackCast = /** @type {function(*)|undefined} */callback;
+        var callbackCast = callback;
         DOMAgent.querySelectorAll(nodeId, selectors, this._wrapClientCallback(callbackCast));
     },
 
-    /**
-     * @param {?number} nodeId
-     * @param {string=} mode
-     */
     highlightDOMNode: function(nodeId, mode)
     {
         if (this._hideDOMNodeHighlightTimeout) {
@@ -488,9 +373,6 @@ WebInspector.DOMTreeManager.prototype = {
         this.highlightDOMNode(0);
     },
 
-    /**
-     * @param {?DOMAgent.NodeId} nodeId
-     */
     highlightDOMNodeForTwoSeconds: function(nodeId)
     {
         this.highlightDOMNode(nodeId);
@@ -513,9 +395,6 @@ WebInspector.DOMTreeManager.prototype = {
         DOMAgent.setInspectModeEnabled(enabled, this._buildHighlightConfig(), callback.bind(this));
     },
 
-    /**
-     * @param {string=} mode
-     */
     _buildHighlightConfig: function(mode)
     {
         mode = mode || "all";
@@ -823,6 +702,6 @@ WebInspector.DOMTreeManager.prototype = {
             return result;
         }
     }
-}
+};
 
 WebInspector.DOMTreeManager.prototype.__proto__ = WebInspector.Object.prototype;
index aef5e50..59650b1 100644 (file)
@@ -31,9 +31,6 @@
  * OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
  */
 
-/**
- * @constructor
- */
 WebInspector.CSSCompletions = function(properties, acceptEmptyPrefix)
 {
     this._values = [];
@@ -75,10 +72,6 @@ WebInspector.CSSCompletions = function(properties, acceptEmptyPrefix)
     this._acceptEmptyPrefix = acceptEmptyPrefix;
 }
 
-
-/**
- * @type {WebInspector.CSSCompletions}
- */
 WebInspector.CSSCompletions.cssNameCompletions = null;
 
 WebInspector.CSSCompletions.requestCSSNameCompletions = function()
index 803fe86..644c210 100644 (file)
  * OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
  */
 
-/**
- * @constructor
- * @param {WebInspector.DOMAgent} domAgent
- * @param {?WebInspector.DOMNode} doc
- * @param {boolean} isInShadowTree
- * @param {DOMAgent.Node} payload
- */
 WebInspector.DOMNode = function(domAgent, doc, isInShadowTree, payload) {
     WebInspector.Object.call(this);
 
@@ -205,147 +198,88 @@ WebInspector.DOMNode.prototype = {
     {
         this._childNodeCount = count;
     },
-    
-    /**
-     * @return {string}
-     */
+
     computedRole: function()
     {
         return this._computedRole;
     },
 
-    /**
-     * @return {boolean}
-     */
     hasAttributes: function()
     {
         return this._attributes.length > 0;
     },
 
-    /**
-     * @return {boolean}
-     */
     hasChildNodes: function()
     {
         return this.childNodeCount > 0;
     },
 
-    /**
-     * @return {boolean}
-     */
     hasShadowRoots: function()
     {
         return !!this._shadowRoots.length;
     },
 
-    /**
-     * @return {boolean}
-     */
     isInShadowTree: function()
     {
         return this._isInShadowTree;
     },
 
-    /**
-     * @return {number}
-     */
     nodeType: function()
     {
         return this._nodeType;
     },
 
-    /**
-     * @return {string}
-     */
     nodeName: function()
     {
         return this._nodeName;
     },
 
-    /**
-     * @return {string}
-     */
     nodeNameInCorrectCase: function()
     {
         return this.isXMLNode() ? this.nodeName() : this.nodeName().toLowerCase();
     },
 
-    /**
-     * @param {string} name
-     * @param {function()=} callback
-     */
     setNodeName: function(name, callback)
     {
         DOMAgent.setNodeName(this.id, name, this._makeUndoableCallback(callback));
     },
 
-    /**
-     * @return {string}
-     */
     localName: function()
     {
         return this._localName;
     },
 
-    /**
-     * @return {string}
-     */
     nodeValue: function()
     {
         return this._nodeValue;
     },
 
-    /**
-     * @param {string} value
-     * @param {function(?Protocol.Error)=} callback
-     */
     setNodeValue: function(value, callback)
     {
         DOMAgent.setNodeValue(this.id, value, this._makeUndoableCallback(callback));
     },
 
-    /**
-     * @param {string} name
-     * @return {string}
-     */
     getAttribute: function(name)
     {
         var attr = this._attributesMap[name];
         return attr ? attr.value : undefined;
     },
 
-    /**
-     * @param {string} name
-     * @param {string} text
-     * @param {function()=} callback
-     */
     setAttribute: function(name, text, callback)
     {
         DOMAgent.setAttributesAsText(this.id, text, name, this._makeUndoableCallback(callback));
     },
 
-    /**
-     * @param {string} name
-     * @param {string} value
-     * @param {function()=} callback
-     */
     setAttributeValue: function(name, value, callback)
     {
         DOMAgent.setAttributeValue(this.id, name, value, this._makeUndoableCallback(callback));
     },
 
-    /**
-     * @return {Object}
-     */
     attributes: function()
     {
         return this._attributes;
     },
 
-    /**
-     * @param {string} name
-     * @param {function()=} callback
-     */
     removeAttribute: function(name, callback)
     {
         function mycallback(error, success)
@@ -365,9 +299,6 @@ WebInspector.DOMNode.prototype = {
         DOMAgent.removeAttribute(this.id, name, mycallback.bind(this));
     },
 
-    /**
-     * @param {function(Array.<WebInspector.DOMNode>)=} callback
-     */
     getChildNodes: function(callback)
     {
         if (this.children) {
@@ -376,10 +307,6 @@ WebInspector.DOMNode.prototype = {
             return;
         }
 
-        /**
-         * @this {WebInspector.DOMNode}
-         * @param {?Protocol.Error} error
-         */
         function mycallback(error) {
             if (!error && callback)
                 callback(this.children);
@@ -388,45 +315,27 @@ WebInspector.DOMNode.prototype = {
         DOMAgent.requestChildNodes(this.id, mycallback.bind(this));
     },
 
-     /**
-      * @param {number} depth
-      * @param {function(Array.<WebInspector.DOMNode>)=} callback
-      */
     getSubtree: function(depth, callback)
     {
-        /**
-         * @this {WebInspector.DOMNode}
-         * @param {?Protocol.Error} error
-         */
         function mycallback(error)
         {
             if (callback)
-                callback(error ? null : this.children);                
+                callback(error ? null : this.children);
         }
 
         DOMAgent.requestChildNodes(this.id, depth, mycallback.bind(this));
     },
 
-    /**
-     * @param {function(?Protocol.Error)=} callback
-     */
     getOuterHTML: function(callback)
     {
         DOMAgent.getOuterHTML(this.id, callback);
     },
 
-    /**
-     * @param {string} html
-     * @param {function(?Protocol.Error)=} callback
-     */
     setOuterHTML: function(html, callback)
     {
         DOMAgent.setOuterHTML(this.id, html, this._makeUndoableCallback(callback));
     },
 
-    /**
-     * @param {function(?Protocol.Error)=} callback
-     */
     removeNode: function(callback)
     {
         DOMAgent.removeNode(this.id, this._makeUndoableCallback(callback));
@@ -442,9 +351,6 @@ WebInspector.DOMNode.prototype = {
         DOMAgent.getOuterHTML(this.id, copy);
     },
 
-    /**
-     * @param {function(?Protocol.Error)=} callback
-     */
     eventListeners: function(callback)
     {
         DOMAgent.getEventListenersForNode(this.id, callback);
@@ -489,9 +395,6 @@ WebInspector.DOMNode.prototype = {
         DOMAgent.getAccessibilityPropertiesForNode(this.id, accessibilityPropertiesCallback.bind(this));
     },
 
-    /**
-     * @return {string}
-     */
     path: function()
     {
         var path = [];
@@ -504,10 +407,6 @@ WebInspector.DOMNode.prototype = {
         return path.join(",");
     },
 
-    /**
-     * @param {boolean} justSelector
-     * @return {string}
-     */
     appropriateSelectorFor: function(justSelector)
     {
         var lowerCaseName = this.localName() || this.nodeName().toLowerCase();
@@ -534,10 +433,6 @@ WebInspector.DOMNode.prototype = {
         return lowerCaseName;
     },
 
-    /**
-     * @param {WebInspector.DOMNode} node
-     * @return {boolean}
-     */
     isAncestor: function(node)
     {
         if (!node)
@@ -552,18 +447,11 @@ WebInspector.DOMNode.prototype = {
         return false;
     },
 
-    /**
-     * @param {WebInspector.DOMNode} descendant
-     * @return {boolean}
-     */
     isDescendant: function(descendant)
     {
         return descendant !== null && descendant.isAncestor(this);
     },
 
-    /**
-     * @param {Array.<string>} attrs
-     */
     _setAttributesPayload: function(attrs)
     {
         this._attributes = [];
@@ -572,11 +460,6 @@ WebInspector.DOMNode.prototype = {
             this._addAttribute(attrs[i], attrs[i + 1]);
     },
 
-    /**
-     * @param {WebInspector.DOMNode} prev
-     * @param {DOMAgent.Node} payload
-     * @return {WebInspector.DOMNode}
-     */
     _insertChild: function(prev, payload)
     {
         var node = new WebInspector.DOMNode(this._domAgent, this.ownerDocument, this._isInShadowTree, payload);
@@ -592,9 +475,6 @@ WebInspector.DOMNode.prototype = {
         return node;
     },
 
-    /**
-     * @param {WebInspector.DOMNode} node
-     */
     _removeChild: function(node)
     {
         this._children.splice(this._children.indexOf(node), 1);
@@ -602,9 +482,6 @@ WebInspector.DOMNode.prototype = {
         this._renumber();
     },
 
-    /**
-     * @param {Array.<DOMAgent.Node>} payloads
-     */
     _setChildrenPayload: function(payloads)
     {
         // We set children in the constructor.
@@ -637,10 +514,6 @@ WebInspector.DOMNode.prototype = {
         }
     },
 
-    /**
-     * @param {string} name
-     * @param {string} value
-     */
     _addAttribute: function(name, value)
     {
         var attr = {
@@ -652,10 +525,6 @@ WebInspector.DOMNode.prototype = {
         this._attributes.push(attr);
     },
 
-    /**
-     * @param {string} name
-     * @param {string} value
-     */
     _setAttribute: function(name, value)
     {
         var attr = this._attributesMap[name];
@@ -665,9 +534,6 @@ WebInspector.DOMNode.prototype = {
             this._addAttribute(name, value);
     },
 
-    /**
-     * @param {string} name
-     */
     _removeAttribute: function(name)
     {
         var attr = this._attributesMap[name];
@@ -677,19 +543,11 @@ WebInspector.DOMNode.prototype = {
         }
     },
 
-    /**
-     * @param {WebInspector.DOMNode} targetNode
-     * @param {?WebInspector.DOMNode} anchorNode
-     * @param {function(?Protocol.Error)=} callback
-     */
     moveTo: function(targetNode, anchorNode, callback)
     {
         DOMAgent.moveTo(this.id, targetNode.id, anchorNode ? anchorNode.id : undefined, this._makeUndoableCallback(callback));
     },
 
-    /**
-     * @return {boolean}
-     */
     isXMLNode: function()
     {
         return !!this.ownerDocument && !!this.ownerDocument.xmlVersion;
@@ -699,7 +557,7 @@ WebInspector.DOMNode.prototype = {
     {
         return this._enabledPseudoClasses;
     },
-    
+
     setPseudoClassEnabled: function(pseudoClass, enabled)
     {
         var pseudoClasses = this._enabledPseudoClasses;
index b910baa..293a040 100644 (file)
  * THE POSSIBILITY OF SUCH DAMAGE.
  */
 
-/**
- * Implements Source Map V3 model. See http://code.google.com/p/closure-compiler/wiki/SourceMaps
- * for format description.
- * @constructor
- * @param {string} sourceMappingURL
- * @param {SourceMapV3} payload
- * @param {WebInspector.Resource|WebInspector.Script} originalSourceCode
- */
 WebInspector.SourceMap = function(sourceMappingURL, payload, originalSourceCode)
 {
     if (!WebInspector.SourceMap.prototype._base64Map) {
@@ -101,26 +93,16 @@ WebInspector.SourceMap.prototype = {
         return this._sourceMapResources[url];
     },
 
-    /**
-     * @return {Array.<string>}
-     */
     sources: function()
     {
         return Object.keys(this._sources);
     },
 
-    /**
-     * @param {string} sourceURL
-     * @return {string|undefined}
-     */
     sourceContent: function(sourceURL)
     {
         return this._sourceContentByURL[sourceURL];
     },
 
-    /**
-     * @param {SourceMapV3} mappingPayload
-     */
     _parseMappingPayload: function(mappingPayload)
     {
         if (mappingPayload.sections)
@@ -129,9 +111,6 @@ WebInspector.SourceMap.prototype = {
             this._parseMap(mappingPayload, 0, 0);
     },
 
-    /**
-     * @param {Array.<SourceMapV3.Section>} sections
-     */
     _parseSections: function(sections)
     {
         for (var i = 0; i < sections.length; ++i) {
@@ -140,11 +119,6 @@ WebInspector.SourceMap.prototype = {
         }
     },
 
-    /**
-     * @param {number} lineNumber in compiled resource
-     * @param {number} columnNumber in compiled resource
-     * @return {?Array}
-     */
     findEntry: function(lineNumber, columnNumber)
     {
         var first = 0;
@@ -166,11 +140,6 @@ WebInspector.SourceMap.prototype = {
         return entry;
     },
 
-    /**
-     * @param {string} sourceURL of the originating resource
-     * @param {number} lineNumber in the originating resource
-     * @return {Array}
-     */
     findEntryReversed: function(sourceURL, lineNumber)
     {
         var mappings = this._reverseMappingsBySourceURL[sourceURL];
@@ -182,11 +151,6 @@ WebInspector.SourceMap.prototype = {
         return this._mappings[0];
     },
 
-    /**
-     * @param {SourceMapV3} map
-     * @param {number} lineNumber
-     * @param {number} columnNumber
-     */
     _parseMap: function(map, lineNumber, columnNumber)
     {
         var sourceIndex = 0;
@@ -261,19 +225,11 @@ WebInspector.SourceMap.prototype = {
         }
     },
 
-    /**
-     * @param {string} char
-     * @return {boolean}
-     */
     _isSeparator: function(char)
     {
         return char === "," || char === ";";
     },
 
-    /**
-     * @param {WebInspector.SourceMap.StringCharIterator} stringCharIterator
-     * @return {number}
-     */
     _decodeVLQ: function(stringCharIterator)
     {
         // Read unsigned value.
@@ -296,10 +252,6 @@ WebInspector.SourceMap.prototype = {
     _VLQ_CONTINUATION_MASK: 1 << 5
 }
 
-/**
- * @constructor
- * @param {string} string
- */
 WebInspector.SourceMap.StringCharIterator = function(string)
 {
     this._string = string;
@@ -307,25 +259,16 @@ WebInspector.SourceMap.StringCharIterator = function(string)
 }
 
 WebInspector.SourceMap.StringCharIterator.prototype = {
-    /**
-     * @return {string}
-     */
     next: function()
     {
         return this._string.charAt(this._position++);
     },
 
-    /**
-     * @return {string}
-     */
     peek: function()
     {
         return this._string.charAt(this._position);
     },
 
-    /**
-     * @return {boolean}
-     */
     hasNext: function()
     {
         return this._position < this._string.length;
index 8f192e3..64373aa 100644 (file)
@@ -30,9 +30,6 @@
  * OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
  */
 
-/**
- * @constructor
- */
 function InspectorBackendClass()
 {
     this._lastSequenceId = 1;
index 9a9a1d3..0229b12 100644 (file)
  * OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
  */
 
-/**
- * @constructor
- * @param {string|undefined} objectId
- * @param {string} type
- * @param {string|undefined} subtype
- * @param {*} value
- * @param {string=} description
- */
 WebInspector.RemoteObject = function(objectId, type, subtype, value, description)
 {
     this._type = type;
@@ -54,35 +46,18 @@ WebInspector.RemoteObject = function(objectId, type, subtype, value, description
     }
 }
 
-/**
- * @param {number|string|boolean} value
- * @return {WebInspector.RemoteObject}
- */
 WebInspector.RemoteObject.fromPrimitiveValue = function(value)
 {
     return new WebInspector.RemoteObject(undefined, typeof value, undefined, value);
 }
 
-/**
- * @param {Object} value
- * @return {WebInspector.RemoteObject}
- */
 WebInspector.RemoteObject.fromLocalObject = function(value)
 {
     return new WebInspector.LocalJSONObject(value);
 }
 
-/**
- * @param {WebInspector.DOMNode} node
- * @param {string} objectGroup
- * @param {function(?WebInspector.RemoteObject)} callback
- */
 WebInspector.RemoteObject.resolveNode = function(node, objectGroup, callback)
 {
-    /**
-     * @param {?Protocol.Error} error
-     * @param {RuntimeAgent.RemoteObject} object
-     */
     function mycallback(error, object)
     {
         if (!callback)
@@ -96,10 +71,6 @@ WebInspector.RemoteObject.resolveNode = function(node, objectGroup, callback)
     DOMAgent.resolveNode(node.id, objectGroup, mycallback);
 }
 
-/**
- * @param {RuntimeAgent.RemoteObject} payload
- * @return {WebInspector.RemoteObject}
- */
 WebInspector.RemoteObject.fromPayload = function(payload)
 {
     console.assert(typeof payload === "object", "Remote object payload should only be an object");
@@ -107,10 +78,6 @@ WebInspector.RemoteObject.fromPayload = function(payload)
     return new WebInspector.RemoteObject(payload.objectId, payload.type, payload.subtype, payload.value, payload.description);
 }
 
-/**
- * @param {WebInspector.RemoteObject} remoteObject
- * @return {string}
- */
 WebInspector.RemoteObject.type = function(remoteObject)
 {
     if (remoteObject === null)
@@ -124,56 +91,41 @@ WebInspector.RemoteObject.type = function(remoteObject)
 }
 
 WebInspector.RemoteObject.prototype = {
-    /** @return {RuntimeAgent.RemoteObjectId} */
     get objectId()
     {
         return this._objectId;
     },
 
-    /** @return {string} */
     get type()
     {
         return this._type;
     },
 
-    /** @return {string|undefined} */
     get subtype()
     {
         return this._subtype;
     },
 
-    /** @return {string|undefined} */
     get description()
     {
         return this._description;
     },
 
-    /** @return {boolean} */
     get hasChildren()
     {
         return this._hasChildren;
     },
 
-    /**
-     * @param {function(Array.<WebInspector.RemoteObjectProperty>)} callback
-     */
     getOwnProperties: function(callback)
     {
         this._getProperties(true, callback);
     },
 
-    /**
-     * @param {function(Array.<WebInspector.RemoteObjectProperty>)} callback
-     */
     getAllProperties: function(callback)
     {
         this._getProperties(false, callback);
     },
 
-    /**
-     * @param {boolean} ownProperties
-     * @param {function(Array.<RuntimeAgent.RemoteObject>)} callback
-     */
     _getProperties: function(ownProperties, callback)
     {
         if (!this._objectId) {
@@ -181,10 +133,6 @@ WebInspector.RemoteObject.prototype = {
             return;
         }
 
-        /**
-         * @param {?Protocol.Error} error
-         * @param {Array.<WebInspector.RemoteObjectProperty>} properties
-         */
         function remoteObjectBinder(error, properties)
         {
             if (error) {
@@ -207,11 +155,6 @@ WebInspector.RemoteObject.prototype = {
         RuntimeAgent.getProperties(this._objectId, ownProperties, remoteObjectBinder);
     },
 
-    /**
-     * @param {string} name
-     * @param {string} value
-     * @param {function(string=)} callback
-     */
     setPropertyValue: function(name, value, callback)
     {
         if (!this._objectId) {
@@ -221,11 +164,6 @@ WebInspector.RemoteObject.prototype = {
 
         RuntimeAgent.evaluate.invoke({expression:value, doNotPauseOnExceptionsAndMuteConsole:true}, evaluatedCallback.bind(this));
 
-        /**
-         * @param {?Protocol.Error} error
-         * @param {RuntimeAgent.RemoteObject} result
-         * @param {boolean=} wasThrown
-         */
         function evaluatedCallback(error, result, wasThrown)
         {
             if (error || wasThrown) {
@@ -244,11 +182,6 @@ WebInspector.RemoteObject.prototype = {
                 RuntimeAgent.releaseObject(result._objectId);
         }
 
-        /**
-         * @param {?Protocol.Error} error
-         * @param {RuntimeAgent.RemoteObject} result
-         * @param {boolean=} wasThrown
-         */
         function propertySetCallback(error, result, wasThrown)
         {
             if (error || wasThrown) {
@@ -259,9 +192,6 @@ WebInspector.RemoteObject.prototype = {
         }
     },
 
-    /**
-     * @param {function(DOMAgent.NodeId)} callback
-     */
     pushNodeToFrontend: function(callback)
     {
         if (this._objectId)
@@ -270,10 +200,6 @@ WebInspector.RemoteObject.prototype = {
             callback(0);
     },
 
-    /**
-     * @param {string} functionDeclaration
-     * @param {function(?WebInspector.RemoteObject)} callback
-     */
     callFunction: function(functionDeclaration, args, callback)
     {
         function mycallback(error, result, wasThrown)
@@ -284,10 +210,6 @@ WebInspector.RemoteObject.prototype = {
         RuntimeAgent.callFunctionOn(this._objectId, functionDeclaration.toString(), args, true, undefined, mycallback);
     },
 
-    /**
-     * @param {string} functionDeclaration
-     * @param {function(*)} callback
-     */
     callFunctionJSON: function(functionDeclaration, args, callback)
     {
         function mycallback(error, result, wasThrown)
@@ -303,9 +225,6 @@ WebInspector.RemoteObject.prototype = {
         RuntimeAgent.releaseObject(this._objectId);
     },
 
-    /**
-     * @return {number}
-     */
     arrayLength: function()
     {
         if (this.subtype !== "array")
@@ -318,12 +237,6 @@ WebInspector.RemoteObject.prototype = {
     }
 }
 
-/**
- * @constructor
- * @param {string} name
- * @param {WebInspector.RemoteObject} value 
- * @param {Object=} descriptor
- */
 WebInspector.RemoteObjectProperty = function(name, value, descriptor)
 {
     this.name = name;
@@ -334,11 +247,6 @@ WebInspector.RemoteObjectProperty = function(name, value, descriptor)
         this.wasThrown = true;
 }
 
-/**
- * @param {string} name
- * @param {string} value
- * @return {WebInspector.RemoteObjectProperty}
- */
 WebInspector.RemoteObjectProperty.fromPrimitiveValue = function(name, value)
 {
     return new WebInspector.RemoteObjectProperty(name, WebInspector.RemoteObject.fromPrimitiveValue(value));
@@ -350,20 +258,12 @@ WebInspector.RemoteObjectProperty.fromPrimitiveValue = function(name, value)
 // for traversing prototypes, extracting class names via constuctor, handling properties
 // or functions.
 
-/**
- * @constructor
- * @extends {WebInspector.RemoteObject}
- * @param {Object} value
- */
 WebInspector.LocalJSONObject = function(value)
 {
     this._value = value;
 }
 
 WebInspector.LocalJSONObject.prototype = {
-    /**
-     * @return {string}
-     */
     get description()
     {
         if (this._cachedDescription)
@@ -394,11 +294,6 @@ WebInspector.LocalJSONObject.prototype = {
         return this._cachedDescription;
     },
 
-    /**
-     * @param {string} prefix
-     * @param {string} suffix
-     * @return {string}
-     */
     _concatenate: function(prefix, suffix, formatProperty)
     {
         const previewChars = 100;
@@ -419,17 +314,11 @@ WebInspector.LocalJSONObject.prototype = {
         return buffer;
     },
 
-    /**
-     * @return {string}
-     */
     get type()
     {
         return typeof this._value;
     },
 
-    /**
-     * @return {string|undefined}
-     */
     get subtype()
     {
         if (this._value === null)
@@ -441,33 +330,21 @@ WebInspector.LocalJSONObject.prototype = {
         return undefined;
     },
 
-    /**
-     * @return {boolean}
-     */
     get hasChildren()
     {
         return typeof this._value === "object" && this._value !== null && !isEmptyObject(this._value);
     },
 
-    /**
-     * @param {function(Array.<WebInspector.RemoteObjectProperty>)} callback
-     */
     getOwnProperties: function(callback)
     {
         callback(this._children());
     },
 
-    /**
-     * @param {function(Array.<WebInspector.RemoteObjectProperty>)} callback
-     */
     getAllProperties: function(callback)
     {
         callback(this._children());
     },
 
-    /**
-     * @return {Array.<WebInspector.RemoteObjectProperty>}
-     */
     _children: function()
     {
         if (!this.hasChildren)
@@ -482,9 +359,6 @@ WebInspector.LocalJSONObject.prototype = {
         return this._cachedChildren;
     },
 
-    /**
-     * @return {boolean}
-     */
     isError: function()
     {
         return false;
index 6ff490d..8e1720f 100644 (file)
@@ -108,9 +108,6 @@ WebInspector.ApplicationCacheFrameContentView.prototype = {
         this.updateStatus(frameManifest.status);
     },
 
-    /**
-     * @param {number} status
-     */
     updateStatus: function(status)
     {
         var oldStatus = this._status;
@@ -127,9 +124,6 @@ WebInspector.ApplicationCacheFrameContentView.prototype = {
         WebInspector.applicationCacheManager.requestApplicationCache(this._frame, this._updateCallback.bind(this));
     },
 
-    /**
-     * @param {Object} applicationCache
-     */
     _updateCallback: function(applicationCache)
     {
         if (!applicationCache || !applicationCache.manifestURL) {
index 6c6d268..6646182 100644 (file)
@@ -43,9 +43,6 @@ WebInspector.ConsoleMessage = function(source, level, url, line, column, repeatC
 }
 
 WebInspector.ConsoleMessage.prototype = {
-    /**
-     * @return {boolean}
-     */
     isErrorOrWarning: function()
     {
         return (this.level === WebInspector.ConsoleMessage.MessageLevel.Warning || this.level === WebInspector.ConsoleMessage.MessageLevel.Error);
@@ -56,29 +53,12 @@ WebInspector.ConsoleMessage.prototype = {
         // Implemented by concrete instances
     },
 
-    /**
-     * @return {WebInspector.ConsoleMessage}
-     */
     clone: function()
     {
         // Implemented by concrete instances
     }
 }
 
-/**
- * @param {string} source
- * @param {string} level
- * @param {string} message
- * @param {string=} type
- * @param {string=} url
- * @param {number=} line
- * @param {number=} repeatCount
- * @param {Array.<RuntimeAgent.RemoteObject>=} parameters
- * @param {ConsoleAgent.StackTrace=} stackTrace
- * @param {WebInspector.Resource=} request
- *
- * @return {WebInspector.ConsoleMessage}
- */
 WebInspector.ConsoleMessage.create = function(source, level, message, type, url, line, column, repeatCount, parameters, stackTrace, request)
 {
     return new WebInspector.ConsoleMessageImpl(source, level, message, null, type, url, line, column, repeatCount, parameters, stackTrace, request);
index ad9c519..41322bf 100644 (file)
@@ -243,9 +243,6 @@ WebInspector.ConsoleMessageImpl.prototype = {
         return formattedResult;
     },
 
-    /**
-     * @param {boolean=} forceObjectFormat
-     */
     _formatParameter: function(output, forceObjectFormat)
     {
         var type;
@@ -661,9 +658,6 @@ WebInspector.ConsoleMessageImpl.prototype = {
         return this._stackTrace;
     },
 
-    /**
-     * @return {WebInspector.ConsoleMessage}
-     */
     clone: function()
     {
         return WebInspector.ConsoleMessage.create(this.source, this.level, this._messageText, this.type, this.url, this.line, this.column, this.repeatCount, this._parameters, this._stackTrace, this._request);
@@ -693,7 +687,7 @@ WebInspector.ConsoleMessageImpl.prototype = {
     toClipboardString: function(isPrefixOptional)
     {
         var isTrace = this._shouldDumpStackTrace();
-        
+
         var clipboardString = "";
         if (this._formattedMessage && !isTrace)
             clipboardString = this._formattedMessage.querySelector("span").innerText;
@@ -720,7 +714,7 @@ WebInspector.ConsoleMessageImpl.prototype = {
                 urlLine = " (" + components.join(", ") + ")";
             } else if (repeatString)
                 urlLine = " (" + repeatString + ")";
-        
+
             if (urlLine) {
                 var lines = clipboardString.split("\n");
                 lines[0] += urlLine;
index 16c862c..9319c1b 100644 (file)
  * OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
  */
 
-/**
- * @constructor
- * @param {WebInspector.ContextSubMenuItem} topLevelMenu
- * @param {string} type
- * @param {string=} label
- * @param {boolean=} disabled
- * @param {boolean=} checked
- */
 WebInspector.ContextMenuItem = function(topLevelMenu, type, label, disabled, checked)
 {
     this._type = type;
@@ -59,17 +51,11 @@ WebInspector.ContextMenuItem.prototype = {
         return this._type;
     },
 
-    /**
-     * @return {boolean}
-     */
     isEnabled: function()
     {
         return !this._disabled;
     },
 
-    /**
-     * @param {boolean} enabled
-     */
     setEnabled: function(enabled)
     {
         this._disabled = !enabled;
@@ -88,13 +74,6 @@ WebInspector.ContextMenuItem.prototype = {
     }
 }
 
-/**
- * @constructor
- * @extends {WebInspector.ContextMenuItem}
- * @param topLevelMenu
- * @param {string=} label
- * @param {boolean=} disabled
- */
 WebInspector.ContextSubMenuItem = function(topLevelMenu, label, disabled)
 {
     WebInspector.ContextMenuItem.call(this, topLevelMenu, "subMenu", label, disabled);
@@ -102,12 +81,6 @@ WebInspector.ContextSubMenuItem = function(topLevelMenu, label, disabled)
 }
 
 WebInspector.ContextSubMenuItem.prototype = {
-    /**
-     * @param {string} label
-     * @param {function} handler
-     * @param {boolean=} disabled
-     * @return {WebInspector.ContextMenuItem}
-     */
     appendItem: function(label, handler, disabled)
     {
         var item = new WebInspector.ContextMenuItem(this._contextMenu, "item", label, disabled);
@@ -116,11 +89,6 @@ WebInspector.ContextSubMenuItem.prototype = {
         return item;
     },
 
-    /**
-     * @param {string} label
-     * @param {boolean=} disabled
-     * @return {WebInspector.ContextMenuItem}
-     */
     appendSubMenuItem: function(label, disabled)
     {
         var item = new WebInspector.ContextSubMenuItem(this._contextMenu, label, disabled);
@@ -128,9 +96,6 @@ WebInspector.ContextSubMenuItem.prototype = {
         return item;
     },
 
-    /**
-     * @param {boolean=} disabled
-     */
     appendCheckboxItem: function(label, handler, checked, disabled)
     {
         var item = new WebInspector.ContextMenuItem(this._contextMenu, "checkbox", label, disabled, checked);
@@ -154,9 +119,6 @@ WebInspector.ContextSubMenuItem.prototype = {
         this._items.push(item);
     },
 
-    /**
-     * @return {boolean}
-     */
     isEmpty: function()
     {
         return !this._items.length;
@@ -173,10 +135,6 @@ WebInspector.ContextSubMenuItem.prototype = {
     __proto__: WebInspector.ContextMenuItem.prototype
 }
 
-/**
- * @constructor
- * @extends {WebInspector.ContextSubMenuItem}
- */
 WebInspector.ContextMenu = function(event) {
     WebInspector.ContextSubMenuItem.call(this, this, "");
     this._event = event;
@@ -245,9 +203,6 @@ WebInspector.ContextMenu.prototype = {
             this._handlers[id].call(this);
     },
 
-    /**
-     * @param {Object} target
-     */
     appendApplicableItems: function(target)
     {
         for (var i = 0; i < WebInspector.ContextMenu._providers.length; ++i) {
@@ -261,24 +216,14 @@ WebInspector.ContextMenu.prototype = {
     __proto__: WebInspector.ContextSubMenuItem.prototype
 }
 
-/**
- * @interface
- */
 WebInspector.ContextMenu.Provider = function()
 {
 }
 
 WebInspector.ContextMenu.Provider.prototype = {
-    /** 
-     * @param {WebInspector.ContextMenu} contextMenu
-     * @param {Object} target
-     */
     appendApplicableItems: function(event, contextMenu, target) { }
 }
 
-/**
- * @param {WebInspector.ContextMenu.Provider} provider
- */
 WebInspector.ContextMenu.registerProvider = function(provider)
 {
     WebInspector.ContextMenu._providers.push(provider);
index 274f90a..3c4716d 100644 (file)
  * THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
  */
 
-/**
- * @constructor
- * @extends {TreeElement}
- * @param {boolean=} elementCloseTag
- */
 WebInspector.DOMTreeElement = function(node, elementCloseTag)
 {
     this._elementCloseTag = elementCloseTag;
@@ -186,7 +181,7 @@ WebInspector.DOMTreeElement.prototype = {
 
     _createTooltipForNode: function()
     {
-        var node = /** @type {WebInspector.DOMNode} */ this.representedObject;
+        var node = this.representedObject;
         if (!node.nodeName() || node.nodeName().toLowerCase() !== "img")
             return;
 
@@ -277,9 +272,6 @@ WebInspector.DOMTreeElement.prototype = {
         this.representedObject.getSubtree(-1, callback.bind(this));
     },
 
-    /**
-     * @param {boolean=} fullRefresh
-     */
     updateChildren: function(fullRefresh)
     {
         if (this._elementCloseTag)
@@ -287,9 +279,6 @@ WebInspector.DOMTreeElement.prototype = {
         this.representedObject.getChildNodes(this._updateChildren.bind(this, fullRefresh));
     },
 
-    /**
-     * @param {boolean=} closingTag
-     */
     insertChildElement: function(child, index, closingTag)
     {
         var newElement = new WebInspector.DOMTreeElement(child, closingTag);
@@ -307,9 +296,6 @@ WebInspector.DOMTreeElement.prototype = {
             child.select();
     },
 
-    /**
-     * @param {boolean=} fullRefresh
-     */
     _updateChildren: function(fullRefresh)
     {
         if (this._updateChildrenInProgress || !this.treeOutline._visible)
@@ -982,9 +968,6 @@ WebInspector.DOMTreeElement.prototype = {
         return (tags.length === 1 ? null : tags[tags.length-1]);
     },
 
-    /**
-     * @param {boolean=} onlySearchQueryChanged
-     */
     updateTitle: function(onlySearchQueryChanged)
     {
         // If we are editing, return early to prevent canceling the edit.
@@ -1008,9 +991,6 @@ WebInspector.DOMTreeElement.prototype = {
         this._highlightSearchResults();
     },
 
-    /**
-     * @param {WebInspector.DOMNode=} node
-     */
     _buildAttributeDOM: function(parentElement, name, value, node)
     {
         var hasText = (value.length > 0);
@@ -1052,7 +1032,7 @@ WebInspector.DOMTreeElement.prototype = {
 
     _buildTagDOM: function(parentElement, tagName, isClosingTag, isDistinctTreeElement)
     {
-        var node = /** @type WebInspector.DOMNode */ this.representedObject;
+        var node = this.representedObject;
         var classes = [ "html-tag" ];
         if (isClosingTag && isDistinctTreeElement)
             classes.push("close");
index 06b8804..e2e9ad7 100644 (file)
  * THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
  */
 
-/**
- * @constructor
- * @extends {TreeOutline}
- * @param {boolean=} omitRootDOMNode
- * @param {boolean=} selectEnabled
- */
 WebInspector.DOMTreeOutline = function(omitRootDOMNode, selectEnabled, showInElementsPanelEnabled)
 {
     this.element = document.createElement("ol");
index 98f9b63..447e4a1 100644 (file)
@@ -28,9 +28,6 @@
  * THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
  */
 
-/**
- * @constructor
- */
 WebInspector.DOMTreeUpdater = function(treeOutline)
 {
     WebInspector.domTreeManager.addEventListener(WebInspector.DOMTreeManager.Event.NodeInserted, this._nodeInserted, this);
index 4845b4f..e633d1e 100644 (file)
@@ -1308,11 +1308,6 @@ WebInspector.DataGrid.ResizeMethod = {
 
 WebInspector.DataGrid.prototype.__proto__ = WebInspector.Object.prototype;
 
-/**
- * @constructor
- * @extends {WebInspector.Object}
- * @param {boolean=} hasChildren
- */
 WebInspector.DataGridNode = function(data, hasChildren)
 {
     this._expanded = false;
@@ -1480,7 +1475,7 @@ WebInspector.DataGridNode.prototype = {
     {
         if (typeof(this._leftPadding) === "number")
             return this._leftPadding;
-        
+
         this._leftPadding = this.depth * this.dataGrid.indentWidth;
         return this._leftPadding;
     },
@@ -1701,9 +1696,6 @@ WebInspector.DataGridNode.prototype = {
         this.dispatchEventToListeners("revealed");
     },
 
-    /**
-     * @param {boolean=} supressSelectedEvent
-     */
     select: function(supressSelectedEvent)
     {
         if (!this.dataGrid || !this.selectable || this.selected)
@@ -1728,9 +1720,6 @@ WebInspector.DataGridNode.prototype = {
         this.select();
     },
 
-    /**
-     * @param {boolean=} supressDeselectedEvent
-     */
     deselect: function(supressDeselectedEvent)
     {
         if (!this.dataGrid || this.dataGrid.selectedNode !== this || !this.selected)
index 597d614..07dbdb3 100644 (file)
@@ -141,10 +141,6 @@ WebInspector.DatabaseContentView.prototype = {
         this._appendErrorQueryResult(query, message);
     },
 
-    /**
-     * @param {string} query
-     * @param {WebInspector.View} view
-     */
     _appendViewQueryResult: function(query, view)
     {
         var resultElement = this._appendQueryResult(query);
@@ -156,10 +152,6 @@ WebInspector.DatabaseContentView.prototype = {
         this._promptElement.scrollIntoView(false);
     },
 
-    /**
-     * @param {string} query
-     * @param {string} errorText
-     */
     _appendErrorQueryResult: function(query, errorText)
     {
         var resultElement = this._appendQueryResult(query);
index 1192999..8e6e039 100644 (file)
@@ -74,40 +74,14 @@ WebInspector.isEventTargetAnEditableField = function(event)
     return false;
 }
 
-/**
- * @constructor
- * @param {function(Element,string,string,*,string)} commitHandler
- * @param {function(Element,*)} cancelHandler
- * @param {*=} context
- */
 WebInspector.EditingConfig = function(commitHandler, cancelHandler, context)
 {
     this.commitHandler = commitHandler;
     this.cancelHandler = cancelHandler;
     this.context = context;
-
-    /**
-     * Handles the "paste" event, return values are the same as those for customFinishHandler
-     * @type {function(Element)|undefined}
-     */
     this.pasteHandler;
-
-    /** 
-     * Whether the edited element is multiline
-     * @type {boolean|undefined}
-     */
     this.multiline;
-
-    /**
-     * Custom finish handler for the editing session (invoked on keydown)
-     * @type {function(Element,*)|undefined}
-     */
     this.customFinishHandler;
-
-    /**
-     * Whether or not spellcheck is enabled.
-     * @type {boolean}
-     */
     this.spellcheck = false;
 }
 
@@ -128,10 +102,6 @@ WebInspector.EditingConfig.prototype = {
     }
 }
 
-/** 
- * @param {Element} element
- * @param {WebInspector.EditingConfig=} config
- */
 WebInspector.startEditing = function(element, config)
 {
     if (!WebInspector.markBeingEdited(element, true))
@@ -168,7 +138,6 @@ WebInspector.startEditing = function(element, config)
             return element.textContent;
     }
 
-    /** @this {Element} */
     function cleanUpAfterEditing()
     {
         WebInspector.markBeingEdited(element, false);
@@ -195,7 +164,6 @@ WebInspector.startEditing = function(element, config)
         WebInspector.restoreFocusFromElement(element);
     }
 
-    /** @this {Element} */
     function editingCancelled()
     {
         if (this.tagName === "INPUT" && this.type === "text")
@@ -208,7 +176,6 @@ WebInspector.startEditing = function(element, config)
         cancelledCallback(this, context);
     }
 
-    /** @this {Element} */
     function editingCommitted()
     {
         cleanUpAfterEditing.call(this);
index 3233801..28ff3ef 100644 (file)
@@ -85,7 +85,7 @@ WebInspector.ObjectPropertiesSection.prototype = {
             this.propertiesTreeOutline.appendChild(infoElement);
         }
         this.propertiesForTest = properties;
-        
+
         this.dispatchEventToListeners(WebInspector.Section.Event.VisibleContentDidChange);
     }
 }
@@ -137,10 +137,6 @@ WebInspector.ObjectPropertiesSection.CompareProperties = function(propertyA, pro
     return diff;
 }
 
-/**
- * @constructor
- * @extends {TreeElement}
- */
 WebInspector.ObjectPropertyTreeElement = function(property)
 {
     this.property = property;
index 44e38ed..b51ef21 100644 (file)
@@ -41,7 +41,7 @@ WebInspector.Popover = function(delegate) {
     this._canvasId = "popover-" + (WebInspector.Popover.canvasId++);
     this._element.style.backgroundImage = "-webkit-canvas(" + this._canvasId + ")";
     this._element.addEventListener("transitionend", this, true);
-    
+
     this._container = this._element.appendChild(document.createElement("div"));
     this._container.className = "container";
 };
@@ -114,10 +114,6 @@ WebInspector.Popover.prototype = {
             previouslyFocusedElement.focus();
     },
 
-    /**
-     * @param {WebInspector.Rect} targetFrame
-     * @param {WebInspector.RectEdge}[] preferredEdges
-     */
     present: function(targetFrame, preferredEdges)
     {
         this._targetFrame = targetFrame;
@@ -131,7 +127,7 @@ WebInspector.Popover.prototype = {
 
         this._update();
     },
-    
+
     dismiss: function()
     {
         if (this._element.parentNode !== document.body)
@@ -421,7 +417,7 @@ WebInspector.Popover.prototype = {
 
         finalContext.drawImage(scratchCanvas, 0, 0, scaledWidth, scaledHeight);
     },
-    
+
     _bestMetricsForEdge: function(preferredSize, targetFrame, containerFrame, edge)
     {
         var x, y;
@@ -459,7 +455,7 @@ WebInspector.Popover.prototype = {
                 y = containerFrame.maxY() - height;
         } else {
             if (x < containerFrame.minX())
-                x = containerFrame.minX(); 
+                x = containerFrame.minX();
             if (x + width > containerFrame.maxX())
                 x = containerFrame.maxX() - width;
         }
@@ -538,7 +534,7 @@ WebInspector.Popover.prototype = {
         }
         ctx.closePath();
     }
-    
+
 };
 
 WebInspector.Popover.prototype.__proto__ = WebInspector.Object.prototype;
index cba78ee..fff1db7 100644 (file)
  * THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
  */
 
-/**
- * @constructor
- */
 function TreeOutline(listNode)
 {
     WebInspector.Object.call(this);
 
     this.element = listNode;
 
-    /**
-     * @type {Array.<TreeElement>}
-     */
     this.children = [];
     this.selectedTreeElement = null;
     this._childrenListNode = listNode;
@@ -489,9 +483,6 @@ TreeOutline.prototype.select = function()
     // this is the root, do nothing
 }
 
-/**
- * @param {boolean=} omitFocus
- */
 TreeOutline.prototype.revealAndSelect = function(omitFocus)
 {
     // this is the root, do nothing
@@ -499,11 +490,6 @@ TreeOutline.prototype.revealAndSelect = function(omitFocus)
 
 TreeOutline.prototype.__proto__ = WebInspector.Object.prototype;
 
-/**
- * @constructor
- * @param {Object=} representedObject
- * @param {boolean=} hasChildren
- */
 function TreeElement(title, representedObject, hasChildren)
 {
     WebInspector.Object.call(this);
@@ -937,10 +923,6 @@ TreeElement.prototype.selectOnMouseDown = function(event)
     this.select(false, true);
 }
 
-/**
- * @param {boolean=} omitFocus
- * @param {boolean=} selectedByUser
- */
 TreeElement.prototype.select = function(omitFocus, selectedByUser, suppressOnSelect, suppressOnDeselect)
 {
     if (!this.treeOutline || !this.selectable)
@@ -978,18 +960,12 @@ TreeElement.prototype.select = function(omitFocus, selectedByUser, suppressOnSel
     delete this.treeOutline.processingSelectionChange;
 }
 
-/**
- * @param {boolean=} omitFocus
- */
 TreeElement.prototype.revealAndSelect = function(omitFocus, selectedByUser, suppressOnSelect, suppressOnDeselect)
 {
     this.reveal();
     this.select(omitFocus, selectedByUser, suppressOnSelect, suppressOnDeselect);
 }
 
-/**
- * @param {boolean=} suppressOnDeselect
- */
 TreeElement.prototype.deselect = function(suppressOnDeselect)
 {
     if (!this.treeOutline || this.treeOutline.selectedTreeElement !== this || !this.selected)
@@ -1015,13 +991,6 @@ TreeElement.prototype.onpopulate = function()
     // Overriden by subclasses.
 }
 
-/**
- * @param {boolean} skipUnrevealed
- * @param {(TreeOutline|TreeElement)=} stayWithin
- * @param {boolean=} dontPopulate
- * @param {Object=} info
- * @return {TreeElement}
- */
 TreeElement.prototype.traverseNextTreeElement = function(skipUnrevealed, stayWithin, dontPopulate, info)
 {
     if (!dontPopulate && this.hasChildren)
@@ -1057,11 +1026,6 @@ TreeElement.prototype.traverseNextTreeElement = function(skipUnrevealed, stayWit
     return (skipUnrevealed ? (element.revealed() ? element.nextSibling : null) : element.nextSibling);
 }
 
-/**
- * @param {boolean} skipUnrevealed
- * @param {boolean=} dontPopulate
- * @return {TreeElement}
- */
 TreeElement.prototype.traversePreviousTreeElement = function(skipUnrevealed, dontPopulate)
 {
     var element = skipUnrevealed ? (this.revealed() ? this.previousSibling : null) : this.previousSibling;
@@ -1085,7 +1049,7 @@ TreeElement.prototype.traversePreviousTreeElement = function(skipUnrevealed, don
 
 TreeElement.prototype.isEventWithinDisclosureTriangle = function(event)
 {
-    // FIXME: We should not use getComputedStyle(). For that we need to get rid of using ::before for disclosure triangle. (http://webk.it/74446) 
+    // FIXME: We should not use getComputedStyle(). For that we need to get rid of using ::before for disclosure triangle. (http://webk.it/74446)
     var computedLeftPadding = window.getComputedStyle(this._listItemNode).getPropertyCSSValue("padding-left").getFloatValue(CSSPrimitiveValue.CSS_PX);
     var left = this._listItemNode.totalOffsetLeft + computedLeftPadding;
     return event.pageX >= left && event.pageX <= left + this.arrowToggleWidth && this.hasChildren;