Add ChromeCodingConvention.java to Closure Compiler to preserve getInstance() type

third_party/closure_compiler/tests/getInstance_test.js

+var cr = {};
+var myModule = {};
+
+myModule.Mess = function() {};
+myModule.Mess.prototype = {
+  whatever: function() {}
+};

[Dan Beam, 2014/07/29 18:05:12]

^ unused

[Vitaly Pavlenko, 2014/07/29 18:53:46]

Acknowledged.

+
+/**
+ * @constructor
+ */
+myModule.myBaseClass = function() {};
+
+myModule.myBaseClass.prototype = {
+  /**
+   * @param {number} name
+   */
+  myMethod: function(name) {
+    alert(name);
+  }
+};
+
+/**
+ * @constructor
+ * @extends {myModule.myBaseClass}
+ */
+myModule.myClass = function() {};
+
+/**
+ * @param {!Function} ctor
+ */
+cr.addSingletonGetter = function(ctor) {

[Dan Beam, 2014/07/29 18:05:12]

can this file be:

var cr = {
  /** @param {!Function} ctor */
  addSingletonGetter: function(ctor) {
    ctor.getInstance = function() {
      return ctor.instance_ || (ctor.instance_ = new ctor());
    };
  }
};

/** @constructor */
function Class() {
  /** @param {number} num */
  this.needsNumber = function(num) {};
}

cr.addSingletonGetter(Class);
Class.getInstance().needsNumber("wrong type");

[Vitaly Pavlenko, 2014/07/29 18:53:46]

Done.

+  ctor.getInstance = function() {
+    return ctor.instance_ || (ctor.instance_ = new ctor());
+  };
+};
+
+cr.addSingletonGetter(myModule.myClass);
+
+/**
+ * @param {string} name
+ */
+myModule.myClass.myMethod = function(name) {
+  myModule.myClass.getInstance().whatever();
+  myModule.myClass.getInstance().myMethod(name);
+};
+
+myModule.myClass.myMethod('string');