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

ui/webui/resources/js/cr.js

 // Copyright (c) 2012 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.
 
 /**
  * The global object.
  * @type {!Object}
  * @const
  */
 var global = this;
@@ skipping 276 matching lines @@
    * instance object.
    * @param {!Function} ctor The constructor for the class to add the static
    *     method to.
    */
   function addSingletonGetter(ctor) {
     ctor.getInstance = function() {
       return ctor.instance_ || (ctor.instance_ = new ctor());
     };
   }
 
+  /**
+   * Forwards public APIs to private implementations.
+   * @param {Function} ctor Constructor that have private implementations in its
+   *     prototype.
+   * @param {Array.<string>} publicAPI List of public method names that have
+   *     their underscored counterparts in constructor's prototype.
+   * @param {HTMLElement=} opt_target Target node (used in
+   *     oobe_screen_user_image.js)
+   */
+  function makePublic(ctor, publicAPI, opt_target) {
+    publicAPI.forEach(function(name) {
+      if (opt_target) {
+        ctor[name] = function(value) {
+          opt_target[name + '_'](value);
+        };
+      } else {
+        ctor[name] = function() {
+          var instance = ctor.getInstance();
+          return instance[name + '_'].apply(instance, arguments);
+        };
+      }
+    });
+  }
+
   return {
     addSingletonGetter: addSingletonGetter,
     createUid: createUid,
     define: define,
     defineProperty: defineProperty,
     dispatchPropertyChange: dispatchPropertyChange,
     dispatchSimpleEvent: dispatchSimpleEvent,
     exportPath: exportPath,
     getUid: getUid,
+    makePublic: makePublic,

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

expose might be a better name if it's apparent enough

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

So could you propose the best method name to use?

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

Tyler and I agreed that makePublic is fine

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

Acknowledged.

     PropertyKind: PropertyKind,
 
     get doc() {
       return document;
     },
 
     /** Whether we are using a Mac or not. */
     get isMac() {
       return /Mac/.test(navigator.platform);
     },
@@ skipping 13 matching lines @@
       return /Linux/.test(navigator.userAgent);
     },
 
     /** Whether this uses the views toolkit or not. */
     get isViews() {
       return typeof chrome.getVariableValue == 'function' &&
           /views/.test(chrome.getVariableValue('toolkit'));
     },
   };
 }();