MD Settings: ESLint --fix output, WIP.

chrome/browser/resources/settings/site_settings/cookie_tree_node.js

 // Copyright 2016 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.
 
 /**
  * @typedef {{title: string,
  *            id: string,
  *            data: CookieDetails}}
  */
 var CookieDataItem;
@@ skipping 56 matching lines @@
      * The data for this cookie node.
      * @type {CookieDetails}
      */
     this.data = data;
 
     /**
      * The child cookie nodes.
      * @private {!Array<!settings.CookieTreeNode>}
      */
     this.children_ = [];
-  };
+  }
 
   CookieTreeNode.prototype = {
     /**
      * Converts a list of cookies and add them as CookieTreeNode children to
      * the given parent node.
      * @param {!settings.CookieTreeNode} parentNode The parent node to add
      *     children to.
      * @param {!Array<!CookieDetails>} newNodes The list containing the data to
      *     add.
      */
@@ skipping 44 matching lines @@
      * Returns an array of cookies from the current node within the cookie tree.
      * @return {!Array<!CookieDataItem>} The Cookie List.
      */
     getCookieList: function() {
       var list = [];
       for (var i = 0; i < this.children_.length; i++) {
         var child = this.children_[i];
         for (var j = 0; j < child.children_.length; j++) {
           var cookie = child.children_[j];
           list.push({title: cookie.data.title,
-                     id: cookie.data.id,
-                     data: cookie.data});
+            id: cookie.data.id,
+            data: cookie.data});

[dschuyler, 2017/04/19 18:58:41]

four space indent

[Dan Beam, 2017/04/19 19:04:40]

well, actually, array and object literals that are multi-line are supposed to
have 2\s indent in JS (which differs from C++, for example).

this is a continuation, so it should either be "box indented" or 4\s (as Dave
said), but I think it'd be more common to see:

list.push({
  title: ...
  id: ...
  data: ...
});

or the box-indented version.

[dpapad, 2017/04/19 19:19:55]

Will tweak config again.

         }
       }
 
       return list;
     },
 
     /**
      * Get a summary list of all sites and their stored data.
      * @return {!Array<!CookieDataSummaryItem>} The summary list.
      */
@@ skipping 72 matching lines @@
           return this.children_[i];
       }
       return null;
     },
   };
 
   return {
     CookieTreeNode: CookieTreeNode,
   };
 });