Run clang-format on ui/webui/resources

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 91 matching lines @@
    * 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.
    */
   function getGetter(name, kind) {
     switch (kind) {
       case PropertyKind.JS:
         var privateName = name + '_';
-        return function() {
-          return this[privateName];
-        };
+        return function() { return this[privateName]; };
       case PropertyKind.ATTR:
         var attributeName = getAttributeName(name);
-        return function() {
-          return this.getAttribute(attributeName);
-        };
+        return function() { return this.getAttribute(attributeName); };
       case PropertyKind.BOOL_ATTR:
         var attributeName = getAttributeName(name);
-        return function() {
-          return this.hasAttribute(attributeName);
-        };
+        return function() { return this.hasAttribute(attributeName); };
     }
 
     // TODO(dbeam): replace with assertNotReached() in assert.js when I can coax
     // the browser/unit tests to preprocess this file through grit.
     throw 'not reached';
   }
 
   /**
    * Helper function for defineProperty that returns the setter of the right
    * kind.
@@ skipping 78 matching lines @@
   }
 
   /**
    * Counter for use with createUid
    */
   var uidCounter = 1;
 
   /**
    * @return {number} A new unique ID.
    */
-  function createUid() {
-    return uidCounter++;
-  }
+  function createUid() { return uidCounter++; }
 
   /**
    * Returns a unique ID for the item. This mutates the item so it needs to be
    * an object
    * @param {!Object} item The item to get the unique ID for.
    * @return {number} The unique ID for the item.
    */
   function getUid(item) {
     if (item.hasOwnProperty('uid'))
       return item.uid;
@@ skipping 37 matching lines @@
    * @param {!Function} fun The function that will return an object containing
    *     the names and values of the new fields.
    */
   function define(name, fun) {
     var obj = exportPath(name);
     var exports = fun();
     for (var propertyName in exports) {
       // Maybe we should check the prototype chain here? The current usage
       // pattern is always using an object literal so we only care about own
       // properties.
-      var propertyDescriptor = Object.getOwnPropertyDescriptor(exports,
-                                                               propertyName);
+      var propertyDescriptor =
+          Object.getOwnPropertyDescriptor(exports, propertyName);
       if (propertyDescriptor)
         Object.defineProperty(obj, propertyName, propertyDescriptor);
     }
   }
 
   /**
    * Adds a {@code getInstance} static method that always return the same
    * 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>} methods List of public method names that have their
    *     underscored counterparts in constructor's prototype.
    * @param {string=} opt_target Selector for target node.
    */
   function makePublic(ctor, methods, opt_target) {
     methods.forEach(function(method) {
       ctor[method] = function() {
         var target = opt_target ? document.getElementById(opt_target) :
-                     ctor.getInstance();
+                                  ctor.getInstance();

[dschuyler, 2016/12/22 19:41:40]

In some cases the format seems to favor doing a simple 4 space indent, without
doing this 'align to something in the line above' thing. In the ternary operator
case it seems to have gone with the 'align to something in the line above', is
that intended/desired?

[Dan Beam, 2016/12/22 19:48:11]

this is configurable to some extent

         return target[method + '_'].apply(target, arguments);
       };
     });
   }
 
   /**
    * The mapping used by the sendWithPromise mechanism to tie the Promise
    * returned to callers with the corresponding WebUI response. The mapping is
    * from ID to the PromiseResolver helper; the ID is generated by
    * sendWithPromise and is unique across all invocations of said method.
@@ skipping 116 matching lines @@
     makePublic: makePublic,
     PropertyKind: PropertyKind,
 
     // C++ <-> JS communication related methods.
     addWebUIListener: addWebUIListener,
     removeWebUIListener: removeWebUIListener,
     sendWithPromise: sendWithPromise,
     webUIListenerCallback: webUIListenerCallback,
     webUIResponse: webUIResponse,
 
-    get doc() {
-      return document;
-    },
+    get doc() { return document; },
 
     /** Whether we are using a Mac or not. */
-    get isMac() {
-      return /Mac/.test(navigator.platform);
-    },
+    get isMac() { return /Mac/.test(navigator.platform); },
 
     /** Whether this is on the Windows platform or not. */
-    get isWindows() {
-      return /Win/.test(navigator.platform);
-    },
+    get isWindows() { return /Win/.test(navigator.platform); },
 
     /** Whether this is on chromeOS or not. */
-    get isChromeOS() {
-      return /CrOS/.test(navigator.userAgent);
-    },
+    get isChromeOS() { return /CrOS/.test(navigator.userAgent); },
 
     /** Whether this is on vanilla Linux (not chromeOS). */
-    get isLinux() {
-      return /Linux/.test(navigator.userAgent);
-    },
+    get isLinux() { return /Linux/.test(navigator.userAgent); },
 
     /** Whether this is on Android. */
-    get isAndroid() {
-      return /Android/.test(navigator.userAgent);
-    },
+    get isAndroid() { return /Android/.test(navigator.userAgent); },
 
     /** Whether this is on iOS. */
-    get isIOS() {
-      return /iPad|iPhone|iPod/.test(navigator.platform);
-    }
+    get isIOS() { return /iPad|iPhone|iPod/.test(navigator.platform); }
   };
 }();