Add public API generation with cr.makePublic() and handle it in compiler pass

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 2a1573eb3f3b14f8b038effde6896c2768dd7047..b32ab84be1682986f37b0aae07a6f63d28e30468 100644
--- a/chrome/browser/resources/options/automatic_settings_reset_banner.js
+++ b/chrome/browser/resources/options/automatic_settings_reset_banner.js
@@ -49,16 +49,25 @@ cr.define('options', function() {
     },
   };
 
+  /**
+   * Fake declaration made for Closure Compiler. Actual method definition is
+   * in the base class.

[Dan Beam, 2014/09/10 19:26:27]

private methods should not be shared, make this protected

[Vitaly Pavlenko, 2014/09/10 20:25:21]

Should I make them protected in base class and here?

[Dan Beam, 2014/09/12 00:57:51]

yes

[Vitaly Pavlenko, 2014/09/12 18:38:07]

Done.

+   * @private
+   */
+  AutomaticSettingsResetBanner.prototype.show_;
+
+  /**
+   * Fake declaration made for Closure Compiler. Actual method definition is
+   * in the base class.
+   * @private
+   */
+  AutomaticSettingsResetBanner.prototype.dismiss_;
+
   // Forward public APIs to private implementations.
-  [
+  cr.makePublic(AutomaticSettingsResetBanner, [
     'show',
     'dismiss',
-  ].forEach(function(name) {
-    AutomaticSettingsResetBanner[name] = function() {
-      var instance = AutomaticSettingsResetBanner.getInstance();
-      return instance[name + '_'].apply(instance, arguments);
-    };
-  });
+  ]);
 
   // Export
   return {