Handle property definition by {cr|Object}.defineProperty() 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 57 matching lines @@
 
   /**
    * The kind of property to define in {@code defineProperty}.
    * @enum {string}
    * @const
    */
   var PropertyKind = {
     /**
      * Plain old JS property where the backing data is stored as a "private"
      * field on the object.
+     * Use only for properties of any type.

[Tyler Breisacher (Chromium), 2014/08/13 20:35:43]

s/only //

[Vitaly Pavlenko, 2014/08/13 21:12:04]

Done.

      */
     JS: 'js',
 
     /**
      * The property backing data is stored as an attribute on an element.
+     * Use only for properties of type {string}.
      */
     ATTR: 'attr',
 
     /**
      * The property backing data is stored as an attribute on an element. If the
      * element has the attribute then the value is true.
+     * Use only for properties of type {boolean}.
      */
     BOOL_ATTR: 'boolAttr'
   };
 
   /**
    * Helper function for defineProperty that returns the getter to use for the
    * property.
    * @param {string} name The name of the property.
    * @param {PropertyKind} kind The kind of the property.
    * @return {function():*} The getter for the property.
@@ skipping 226 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'));
     },
   };
 }();