MD Settings: ESLint --fix output, WIP.

chrome/browser/resources/settings/printing_page/cups_printers.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.
 
 /**
  * @fileoverview 'settings-cups-printers' is a component for showing CUPS
  * Printer settings subpage (chrome://md-settings/cupsPrinters). It is used to
  * set up legacy & non-CloudPrint printers on ChromeOS by leveraging CUPS (the
  * unix printing system) and the many open source drivers built for CUPS.
  */
@@ skipping 43 matching lines @@
     chrome.networkingPrivate.onNetworksChanged.removeListener(
         this.networksChangedListener_);
   },
 
   /**
    * Callback function when networks change.
    * @private
    */
   refreshNetworks_: function() {
     chrome.networkingPrivate.getNetworks(
-        {'networkType': chrome.networkingPrivate.NetworkType.ALL,
-         'configured': true},
+      {'networkType': chrome.networkingPrivate.NetworkType.ALL,

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

This should be a four space indent I believe.

[Dan Beam, 2017/04/19 19:07:03]

yeah, i think this goes against our style guide

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

This is configurable, see http://eslint.org/docs/rules/object-curly-newline.
I'll play with the options and retry --fix.

+        'configured': true},
         this.onNetworksReceived_.bind(this));
   },
 
   /**
    * Callback function when configured networks are received.
    * @param {!Array<!chrome.networkingPrivate.NetworkStateProperties>} states
    *     A list of network state information for each network.
    * @private
    */
   onNetworksReceived_: function(states) {
@@ skipping 41 matching lines @@
   onAddPrinterTap_: function() {
     this.$.addPrinterDialog.open();
     this.$.addPrinterErrorMessage.hidden = true;
   },
 
   /** @private */
   onAddPrinterDialogClose_: function() {
     this.$$('#addPrinter').focus();
   },
 });