[ios] Pass more --jscomp_error switches to the closure Compiler.

ios/web/web_state/js/resources/message.js

 // Copyright 2014 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 // Scripts for the message handler.
 
 goog.provide('__crWeb.message');
 
 goog.require('__crWeb.common');
 
@@ skipping 17 matching lines @@
       // Since the array will be JSON serialized, protect against non-standard
       // custom versions of Array.prototype.toJSON.
       delete messageQueue_.queue.toJSON;
     }
   };
   messageQueue_.reset();
 
   /**
    * Invokes a command on the Objective-C side.
    * @param {Object} command The command in a JavaScript object.
-   * @private
+   * @public

[dpapad, 2016/10/26 17:05:24]

No need for @public. I don't even think such an annotation exists, but
regardless, no @private/@protected implies public.

[Eugene But (OOO till 7-30), 2016/10/26 17:54:45]

Done.

[Justin Donnelly, 2016/10/26 21:06:52]

If you prefer not to use it for stylistic reasons, that's fine. But @public
exists
(https://github.com/google/closure-compiler/wiki/Annotating-JavaScript-for-the...)
and seems to be in wide use
(https://cs.corp.google.com/search/?q=@public+file:%5C.js). One person's noise
is another person's explicitness and clarity. :)

[Eugene But (OOO till 7-30), 2016/10/26 21:16:23]

Thanks! I would prefer to use @public if it's a real thing. Demetrios, what do
you think?

[Eugene But (OOO till 7-30), 2016/10/27 23:23:21]

Added @public back.

    */
   __gCrWeb.message.invokeOnHost = function(command) {
     messageQueue_.queue.push(command);
     sendQueue_(messageQueue_);
   };
 
   /**
    * Returns the message queue as a string.
    * @return {string} The current message queue as a JSON string.
    */
@@ skipping 18 matching lines @@
     }
     // Some pages/plugins implement Object.prototype.toJSON, which can result
     // in serializing messageQueue_ to an invalid format.
     var originalObjectToJSON = Object.prototype.toJSON;
     if (originalObjectToJSON)
       delete Object.prototype.toJSON;
 
     queueObject.queue.forEach(function(command) {
         var stringifiedMessage = __gCrWeb.common.JSONStringify({
             "crwCommand": command,
-            "crwWindowId": __gCrWeb.windowId
+            "crwWindowId": __gCrWeb['windowId']
         });
         // A web page can override |window.webkit| with any value. Deleting the
         // object ensures that original and working implementation of
         // window.webkit is restored.
         var oldWebkit = window.webkit;
         delete window['webkit'];
         window.webkit.messageHandlers[queueObject.scheme].postMessage(
             stringifiedMessage);
         window.webkit = oldWebkit;
     });
     queueObject.reset();
 
     if (originalObjectToJSON) {
       // Restore Object.prototype.toJSON to prevent from breaking any
       // functionality on the page that depends on its custom implementation.
       Object.prototype.toJSON = originalObjectToJSON;
     }
   };
 }());