DevTools: [JSDoc] Fix injected scripts JSDoc and related code

Source/core/inspector/InjectedScriptSource.js

Index: Source/core/inspector/InjectedScriptSource.js
diff --git a/Source/core/inspector/InjectedScriptSource.js b/Source/core/inspector/InjectedScriptSource.js
index b60be3e95288e365eddde8188f7fb2b7acba3747..dc883fc283c441f1e9a528516550cff374f7ab7d 100644
--- a/Source/core/inspector/InjectedScriptSource.js
+++ b/Source/core/inspector/InjectedScriptSource.js
@@ -107,6 +107,7 @@ function toStringDescription(obj)
  * @param {function(...)} func
  * @param {Object} thisObject

[aandrey, 2014/06/05 19:08:24]

why the check for nullability modificator does not work?
should be {?Object} or {!Object}, right?

[apavlov, 2014/06/05 19:26:25]

None of these fixed errors are currently checked for, too. I will add this check
with the following step (the nullability modifiers are checked for using Python
code, not Java).

  * @param {...} var_args
+ * @return {function(...)}
  */
 function bind(func, thisObject, var_args)
 {
@@ -410,7 +411,7 @@ InjectedScript.prototype = {
      * @param {string} objectId
      * @return {Array.<Object>|boolean}
      */
-    getInternalProperties: function(objectId, ownProperties)
+    getInternalProperties: function(objectId)
     {
         var parsedObjectId = this._parseObjectId(objectId);
         var object = this._objectForId(parsedObjectId);
@@ -571,6 +572,7 @@ InjectedScript.prototype = {
     /**
      * @param {string} objectId
      * @param {string} expression
+     * @param {string} args
      * @param {boolean} returnByValue
      * @return {!Object|string}
      */
@@ -639,6 +641,7 @@ InjectedScript.prototype = {
     /**
      * @param {Function} evalFunction
      * @param {Object} object
+     * @param {string} expression
      * @param {string} objectGroup
      * @param {boolean} isEvalOnCallFrame
      * @param {boolean} injectCommandLineAPI
@@ -1405,6 +1408,7 @@ CommandLineAPIImpl.prototype = {
     /**
      * @param {string} selector
      * @param {Node=} opt_startNode
+     * @return {*}
      */
     $: function (selector, opt_startNode)
     {
@@ -1417,6 +1421,7 @@ CommandLineAPIImpl.prototype = {
     /**
      * @param {string} selector
      * @param {Node=} opt_startNode
+     * @return {*}
      */
     $$: function (selector, opt_startNode)
     {
@@ -1437,6 +1442,7 @@ CommandLineAPIImpl.prototype = {
     /**
      * @param {string} xpath
      * @param {Node=} opt_startNode
+     * @return {*}
      */
     $x: function(xpath, opt_startNode)
     {
@@ -1458,21 +1464,33 @@ CommandLineAPIImpl.prototype = {
         }
     },
 
+    /**
+     * @return {*}
+     */
     dir: function(var_args)
     {
         return inspectedWindow.console.dir.apply(inspectedWindow.console, arguments)
     },
 
+    /**
+     * @return {*}
+     */
     dirxml: function(var_args)
     {
         return inspectedWindow.console.dirxml.apply(inspectedWindow.console, arguments)
     },
 
+    /**
+     * @return {!Array.<string>}
+     */
     keys: function(object)
     {
         return Object.keys(object);
     },
 
+    /**
+     * @return {!Array.<*>}
+     */
     values: function(object)
     {
         var result = [];
@@ -1481,11 +1499,17 @@ CommandLineAPIImpl.prototype = {
         return result;
     },
 
+    /**
+     * @return {*}
+     */
     profile: function(opt_title)
     {
         return inspectedWindow.console.profile.apply(inspectedWindow.console, arguments)
     },
 
+    /**
+     * @return {*}
+     */
     profileEnd: function(opt_title)
     {
         return inspectedWindow.console.profileEnd.apply(inspectedWindow.console, arguments)