Clang format JS: Disallow single line functions, conditionals, loops, and switch statements

ui/webui/resources/js/cr/ui/list_selection_model.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.
 
 cr.define('cr.ui', function() {
   /** @const */ var EventTarget = cr.EventTarget;
 
   /**
    * Creates a new selection model that is to be used with lists.
    *
@@ skipping 12 matching lines @@
     this.independentLeadItem_ = !cr.isMac && !cr.isChromeOS;
   }
 
   ListSelectionModel.prototype = {
     __proto__: EventTarget.prototype,
 
     /**
      * The number of items in the model.
      * @type {number}
      */
-    get length() { return this.length_; },
+    get length() {
+      return this.length_;
+    },
 
     /**
      * The selected indexes.
      * Setter also changes lead and anchor indexes if value list is nonempty.
      * @type {!Array}
      */
     get selectedIndexes() {
       return Object.keys(this.selectedIndexes_).map(Number);
     },
     set selectedIndexes(selectedIndexes) {
@@ skipping 211 matching lines @@
     },
 
     leadIndex_: -1,
     oldLeadIndex_: null,
 
     /**
      * The leadIndex is used with multiple selection and it is the index that
      * the user is moving using the arrow keys.
      * @type {number}
      */
-    get leadIndex() { return this.leadIndex_; },
+    get leadIndex() {
+      return this.leadIndex_;
+    },
     set leadIndex(leadIndex) {
       var oldValue = this.leadIndex_;
       var newValue = this.adjustIndex_(leadIndex);
       this.leadIndex_ = newValue;
       // Delays the call of dispatchPropertyChange if batch is running.
       if (!this.changeCount_ && newValue != oldValue)
         cr.dispatchPropertyChange(this, 'leadIndex', newValue, oldValue);
     },
 
     anchorIndex_: -1,
     oldAnchorIndex_: null,
 
     /**
      * The anchorIndex is used with multiple selection.
      * @type {number}
      */
-    get anchorIndex() { return this.anchorIndex_; },
+    get anchorIndex() {
+      return this.anchorIndex_;
+    },
     set anchorIndex(anchorIndex) {
       var oldValue = this.anchorIndex_;
       var newValue = this.adjustIndex_(anchorIndex);
       this.anchorIndex_ = newValue;
       // Delays the call of dispatchPropertyChange if batch is running.
       if (!this.changeCount_ && newValue != oldValue)
         cr.dispatchPropertyChange(this, 'anchorIndex', newValue, oldValue);
     },
 
     /**
@@ skipping 11 matching lines @@
         var index2 = this.getNearestSelectedIndex_(index);
         index = index2;
       }
       return index;
     },
 
     /**
      * Whether the selection model supports multiple selected items.
      * @type {boolean}
      */
-    get multiple() { return true; },
+    get multiple() {
+      return true;
+    },
 
     /**
      * Adjusts the selection after reordering of items in the table.
      * @param {!Array<number>} permutation The reordering permutation.
      */
     adjustToReordering: function(permutation) {
       this.beginChange();
       var oldLeadIndex = this.leadIndex;
       var oldAnchorIndex = this.anchorIndex;
       var oldSelectedItemsCount = this.selectedIndexes.length;
 
-      this.selectedIndexes =
-          this.selectedIndexes
-              .map(function(oldIndex) { return permutation[oldIndex]; })
-              .filter(function(index) { return index != -1; });
+      this.selectedIndexes = this.selectedIndexes
+                                 .map(function(oldIndex) {
+                                   return permutation[oldIndex];
+                                 })
+                                 .filter(function(index) {
+                                   return index != -1;
+                                 });

[dschuyler, 2016/12/22 22:34:27]

Could this have a cr after the assignment?
i.e.   s/ = / =\n/

I don't know if there's a way to get clformat to do that though.

 
       // Will be adjusted in endChange.
       if (oldLeadIndex != -1)
         this.leadIndex = permutation[oldLeadIndex];
       if (oldAnchorIndex != -1)
         this.anchorIndex = permutation[oldAnchorIndex];
 
       if (oldSelectedItemsCount && !this.selectedIndexes.length &&
           this.length_ && oldLeadIndex != -1) {
         // All selected items are deleted. We move selection to next item of
         // last selected item.
         this.selectedIndexes = [Math.min(oldLeadIndex, this.length_ - 1)];
       }
 
       this.endChange();
     },
 
     /**
      * Adjusts selection model length.
      * @param {number} length New selection model length.
      */
-    adjustLength: function(length) { this.length_ = length; }
+    adjustLength: function(length) {
+      this.length_ = length;
+    }
   };
 
   return {ListSelectionModel: ListSelectionModel};
 });