Compile chrome://settings, part 9: yet another final battle

chrome/browser/resources/options/automatic_settings_reset_banner.js

Index: chrome/browser/resources/options/automatic_settings_reset_banner.js
diff --git a/chrome/browser/resources/options/automatic_settings_reset_banner.js b/chrome/browser/resources/options/automatic_settings_reset_banner.js
index b16f6cbe0694c5bedf05740a86af564ba33e771e..4e5ebdbe92e97c8b3c9ff5d08fb9f345bb71e352 100644
--- a/chrome/browser/resources/options/automatic_settings_reset_banner.js
+++ b/chrome/browser/resources/options/automatic_settings_reset_banner.js
@@ -23,6 +23,8 @@ cr.define('options', function() {
 
     /**
      * Initializes the banner's event handlers.
+     * @suppress {checkTypes}
+     * TODO(vitalyp): remove the suppression.

[Dan Beam, 2014/10/01 23:25:45]

why?

[Vitaly Pavlenko, 2014/10/01 23:38:17]

Method AutomaticSettingsResetBanner.dismiss() is declared with pattern "Forward
public APIs to protected implementations":

https://code.google.com/p/chromium/codesearch#chromium/src/chrome/browser/res...

So it requires either to handle it in the compiler pass (slightly different as
"forward API to private implementations" - here method names don't have trailing
underscore) or to write some for-compilation-only in-place fake declarations. Or
not-so-fake declarations.

Do you want me to use one of this ways?

      */
     initialize: function() {
       this.showMetricName = 'AutomaticSettingsReset_WebUIBanner_BannerShown';