MD Settings: ESLint --fix output, WIP.

chrome/browser/resources/settings/internet_page/internet_page.js

 // Copyright 2015 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-internet-page' is the settings page containing internet
  * settings.
  */
 Polymer({
@@ skipping 261 matching lines @@
    *     vpnProviders
    * @param {!chrome.management.ExtensionInfo} extension
    * @private
    */
   addVpnProvider_: function(vpnProviders, extension) {
     if (!extension.enabled ||
         extension.permissions.indexOf('vpnProvider') == -1) {
       return;
     }
     if (vpnProviders.find(function(provider) {
-          return provider.ExtensionID == extension.id;
-        })) {
+      return provider.ExtensionID == extension.id;
+    })) {

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

The body of the |if| now appears to be dangling.

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

yeah, i think the previous one was better.  i think this if () is weird to begin
with, though

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

This is also configurable. Specifically see http://eslint.org/docs/rules/indent,
FunctionDeclaration and FunctionExpression subpotions.

       return;
     }
     var newProvider = {
       ExtensionID: extension.id,
       ProviderName: extension.name,
     };
     vpnProviders.push(newProvider);
   },
 
   /**
@@ skipping 48 matching lines @@
   },
 
   /**
    * @param {!chrome.networkingPrivate.ThirdPartyVPNProperties} provider
    * @return {string}
    */
   getAddThirdParrtyVpnLabel_: function(provider) {
     return this.i18n('internetAddThirdPartyVPN', provider.ProviderName);
   }
 });