Run tools/clang-format-js on some of chrome/browser/resources/

chrome/browser/resources/inspect/inspect.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.
 
 var MIN_VERSION_TAB_CLOSE = 25;
 var MIN_VERSION_TARGET_ID = 26;
 var MIN_VERSION_NEW_TAB = 29;
 var MIN_VERSION_TAB_ACTIVATE = 30;
 var WEBRTC_SERIAL = 'WEBRTC';
 
 var queryParamsObject = {};
 var browserInspector;
 var browserInspectorTitle;
 
 (function() {
-var queryParams = window.location.search;
-if (!queryParams)
+  var queryParams = window.location.search;
+  if (!queryParams)
     return;
-var params = queryParams.substring(1).split('&');
-for (var i = 0; i < params.length; ++i) {
+  var params = queryParams.substring(1).split('&');
+  for (var i = 0; i < params.length; ++i) {
     var pair = params[i].split('=');
     queryParamsObject[pair[0]] = pair[1];
-}
+  }
 
-if ('trace' in queryParamsObject || 'tracing' in queryParamsObject) {
-  browserInspector = 'chrome://tracing';
-  browserInspectorTitle = 'trace';
-} else {
-  browserInspector = queryParamsObject['browser-inspector'];
-  browserInspectorTitle = 'inspect';
-}
+  if ('trace' in queryParamsObject || 'tracing' in queryParamsObject) {
+    browserInspector = 'chrome://tracing';
+    browserInspectorTitle = 'trace';
+  } else {
+    browserInspector = queryParamsObject['browser-inspector'];
+    browserInspectorTitle = 'inspect';
+  }
 })();
 
 function sendCommand(command, args) {
   chrome.send(command, Array.prototype.slice.call(arguments, 1));
 }
 
 function sendTargetCommand(command, target) {
   sendCommand(command, target.source, target.id);
 }
 
 function removeChildren(element_id) {
   var element = $(element_id);
   element.textContent = '';
 }
 
 function removeAdditionalChildren(element_id) {
   var element = $(element_id);
   var elements = element.querySelectorAll('.row.additional');
   for (var i = 0; i != elements.length; i++)
     element.removeChild(elements[i]);
 }
 
 function removeChildrenExceptAdditional(element_id) {
   var element = $(element_id);
   var elements = element.querySelectorAll('.row:not(.additional)');
   for (var i = 0; i != elements.length; i++)
-      element.removeChild(elements[i]);
+    element.removeChild(elements[i]);
 }
 
 function onload() {
   var tabContents = document.querySelectorAll('#content > div');
   for (var i = 0; i != tabContents.length; i++) {
     var tabContent = tabContents[i];
     var tabName = tabContent.querySelector('.content-header').textContent;
 
     var tabHeader = document.createElement('div');
     tabHeader.className = 'tab-header';
@@ skipping 87 matching lines @@
   var json = JSON.stringify(data);
   if (element.cachedJSON == json)
     return true;
   element.cachedJSON = json;
   return false;
 }
 
 function updateBrowserVisibility(browserSection) {
   var icon = browserSection.querySelector('.used-for-port-forwarding');
   browserSection.hidden = !browserSection.querySelector('.open') &&
-                          !browserSection.querySelector('.row') &&
-                          !browserInspector &&
-                          (!icon || icon.hidden);
+      !browserSection.querySelector('.row') && !browserInspector &&
+      (!icon || icon.hidden);
 }
 
 function updateUsernameVisibility(deviceSection) {
   var users = new Set();
   var browsers = deviceSection.querySelectorAll('.browser');
 
   Array.prototype.forEach.call(browsers, function(browserSection) {
     if (!browserSection.hidden) {
       var browserUser = browserSection.querySelector('.browser-user');
       if (browserUser)
@@ skipping 21 matching lines @@
   function browserCompare(a, b) {
     if (a.adbBrowserName != b.adbBrowserName)
       return a.adbBrowserName < b.adbBrowserName;
     if (a.adbBrowserVersion != b.adbBrowserVersion)
       return a.adbBrowserVersion < b.adbBrowserVersion;
     return a.id < b.id;
   }
 
   function insertBrowser(browserList, browser) {
     for (var sibling = browserList.firstElementChild; sibling;
-        sibling = sibling.nextElementSibling) {
+         sibling = sibling.nextElementSibling) {

[dschuyler, 2016/12/28 23:59:12]

Should this be indent +4? (It looks like it's open brace aligned, which is
indent +5).

       if (browserCompare(browser, sibling)) {
         browserList.insertBefore(browser, sibling);
         return;
       }
     }
     browserList.appendChild(browser);
   }
 
   var deviceList = $('devices-list');
   if (alreadyDisplayed(deviceList, devices))
     return;
 
   function removeObsolete(validIds, section) {
     if (validIds.indexOf(section.id) < 0)
       section.remove();
   }
 
-  var newDeviceIds = devices.map(function(d) { return d.id });
+  var newDeviceIds = devices.map(function(d) {
+    return d.id
+  });
   Array.prototype.forEach.call(
       deviceList.querySelectorAll('.device'),
       removeObsolete.bind(null, newDeviceIds));
 
   $('devices-help').hidden = !!devices.length;
 
   for (var d = 0; d < devices.length; d++) {
     var device = devices[d];
 
     var deviceSection = $(device.id);
@@ skipping 31 matching lines @@
       var authenticating = document.createElement('div');
       authenticating.className = 'device-auth';
       deviceSection.appendChild(authenticating);
     }
 
     if (alreadyDisplayed(deviceSection, device))
       continue;
 
     deviceSection.querySelector('.device-name').textContent = device.adbModel;
     deviceSection.querySelector('.device-auth').textContent =
-        device.adbConnected ? '' : 'Pending authentication: please accept ' +
-          'debugging session on the device.';
+        device.adbConnected ? '' :
+                              'Pending authentication: please accept ' +

[dschuyler, 2016/12/28 23:59:12]

It looks like the current settings are not consistently breaking after ternary
operators as well (referring to a previous comment in this CL about breaking
before ternary operators). So in this case it almost looks like the ?: has
higher precedence because of the formatting (when the addition actually has
higher precedence).

Is there a way (I'm not sure, maybe AlignOperands: true might work?) to break
after ternary operators to get something like this:

device.adbConnected ? 
    '' :
    'Pending authentication: please accept ' + 
        'debugging session on the device.';

or somehow keep the association with the addition on the deeper indent, like
this (which is okay-ish):


device.adbConnected ? '' :
                      'Pending authentication: please accept ' + 
                          'debugging session on the device.';

+            'debugging session on the device.';
 
     var browserList = deviceSection.querySelector('.browsers');
-    var newBrowserIds =
-        device.browsers.map(function(b) { return b.id });
+    var newBrowserIds = device.browsers.map(function(b) {
+      return b.id
+    });
     Array.prototype.forEach.call(
         browserList.querySelectorAll('.browser'),
         removeObsolete.bind(null, newBrowserIds));
 
     for (var b = 0; b < device.browsers.length; b++) {
       var browser = device.browsers[b];
       var majorChromeVersion = browser.adbBrowserChromeVersion;
       var pageList;
       var browserSection = $(browser.id);
       if (browserSection) {
@@ skipping 55 matching lines @@
             'forwarding. Closing it will drop current connections.';
         browserHeader.appendChild(portForwardingInfo);
 
         if (browserInspector) {
           var link = document.createElement('span');
           link.classList.add('action');
           link.setAttribute('tabindex', 1);
           link.textContent = browserInspectorTitle;
           browserHeader.appendChild(link);
           link.addEventListener(
-              'click',
-              sendCommand.bind(null, 'inspect-browser', browser.source,
-                  browser.id, browserInspector), false);
+              'click', sendCommand.bind(
+                           null, 'inspect-browser', browser.source, browser.id,
+                           browserInspector),
+              false);
         }
 
         pageList = document.createElement('div');
         pageList.className = 'list pages';
         browserSection.appendChild(pageList);
       }
 
       if (!alreadyDisplayed(browserSection, browser)) {
         pageList.textContent = '';
         for (var p = 0; p < browser.pages.length; p++) {
           var page = browser.pages[p];
           // Attached targets have no unique id until Chrome 26. For such
           // targets it is impossible to activate existing DevTools window.
-          page.hasNoUniqueId = page.attached &&
-              majorChromeVersion && majorChromeVersion < MIN_VERSION_TARGET_ID;
+          page.hasNoUniqueId = page.attached && majorChromeVersion &&
+              majorChromeVersion < MIN_VERSION_TARGET_ID;
           var row = addTargetToList(page, pageList, ['name', 'url']);
           if (page['description'])
             addWebViewDetails(row, page);
           else
             addFavicon(row, page);
           if (majorChromeVersion >= MIN_VERSION_TAB_ACTIVATE) {
-            addActionLink(row, 'focus tab',
+            addActionLink(
+                row, 'focus tab',
                 sendTargetCommand.bind(null, 'activate', page), false);
           }
           if (majorChromeVersion) {
-            addActionLink(row, 'reload',
-                sendTargetCommand.bind(null, 'reload', page), page.attached);
+            addActionLink(
+                row, 'reload', sendTargetCommand.bind(null, 'reload', page),
+                page.attached);
           }
           if (majorChromeVersion >= MIN_VERSION_TAB_CLOSE) {
-            addActionLink(row, 'close',
-                sendTargetCommand.bind(null, 'close', page), false);
+            addActionLink(
+                row, 'close', sendTargetCommand.bind(null, 'close', page),
+                false);
           }
         }
       }
       updateBrowserVisibility(browserSection);
     }
     updateUsernameVisibility(deviceSection);
   }
 }
 
 function addToPagesList(data) {
@@ skipping 21 matching lines @@
   Array.prototype.forEach.call(guests, function(guest) {
     var guestRow = addTargetToList(guest, row, ['name', 'url']);
     guestRow.classList.add('guest');
     addFavicon(guestRow, guest);
   });
 }
 
 function addToWorkersList(data) {
   var row =
       addTargetToList(data, $('workers-list'), ['name', 'description', 'url']);
-  addActionLink(row, 'terminate',
-      sendTargetCommand.bind(null, 'close', data), false);
+  addActionLink(
+      row, 'terminate', sendTargetCommand.bind(null, 'close', data), false);
 }
 
 function addToServiceWorkersList(data) {
-    var row = addTargetToList(
-        data, $('service-workers-list'), ['name', 'description', 'url']);
-    addActionLink(row, 'terminate',
-        sendTargetCommand.bind(null, 'close', data), false);
+  var row = addTargetToList(
+      data, $('service-workers-list'), ['name', 'description', 'url']);
+  addActionLink(
+      row, 'terminate', sendTargetCommand.bind(null, 'close', data), false);
 }
 
 function addToOthersList(data) {
   addTargetToList(data, $('others-list'), ['url']);
 }
 
 function addAdditionalTargetsToOthersList(data) {
   addTargetToList(data, $('others-list'), ['name', 'url']);
 }
 
@@ skipping 29 matching lines @@
   } catch (e) {
     return;
   }
   addWebViewDescription(row, webview);
   if (data.adbScreenWidth && data.adbScreenHeight)
     addWebViewThumbnail(
         row, webview, data.adbScreenWidth, data.adbScreenHeight);
 }
 
 function addWebViewDescription(row, webview) {
-  var viewStatus = { visibility: '', position: '', size: '' };
+  var viewStatus = {visibility: '', position: '', size: ''};
   if (!webview.empty) {
     if (webview.attached && !webview.visible)
       viewStatus.visibility = 'hidden';
     else if (!webview.attached)
       viewStatus.visibility = 'detached';
     viewStatus.size = 'size ' + webview.width + ' \u00d7 ' + webview.height;
   } else {
     viewStatus.visibility = 'empty';
   }
   if (webview.attached) {
-      viewStatus.position =
+    viewStatus.position =
         'at (' + webview.screenX + ', ' + webview.screenY + ')';
   }
 
   var subRow = document.createElement('div');
   subRow.className = 'subrow webview';
   if (webview.empty || !webview.attached || !webview.visible)
     subRow.className += ' invisible-view';
   if (viewStatus.visibility)
     subRow.appendChild(formatValue(viewStatus, 'visibility'));
   if (viewStatus.position)
@@ skipping 69 matching lines @@
   subrowBox.appendChild(subrow);
 
   for (var j = 0; j < properties.length; j++)
     subrow.appendChild(formatValue(data, properties[j]));
 
   var actionBox = document.createElement('div');
   actionBox.className = 'actions';
   subrowBox.appendChild(actionBox);
 
   if (data.isAdditional) {
-    addActionLink(row, 'inspect',
-        sendCommand.bind(null, 'inspect-additional', data.url), false);
+    addActionLink(
+        row, 'inspect', sendCommand.bind(null, 'inspect-additional', data.url),
+        false);
     row.classList.add('additional');
   } else if (!data.hasCustomInspectAction) {
-    addActionLink(row, 'inspect', sendTargetCommand.bind(null, 'inspect', data),
+    addActionLink(
+        row, 'inspect', sendTargetCommand.bind(null, 'inspect', data),
         data.hasNoUniqueId || data.adbAttachedForeign);
   }
 
   list.appendChild(row);
   return row;
 }
 
 function addActionLink(row, text, handler, opt_disabled) {
   var link = document.createElement('span');
   link.classList.add('action');
   link.setAttribute('tabindex', 1);
   if (opt_disabled)
     link.classList.add('disabled');
   else
     link.classList.remove('disabled');
 
   link.textContent = text;
   link.addEventListener('click', handler, true);
   function handleKey(e) {
     if (e.key == 'Enter' || e.key == ' ') {
       e.preventDefault();
       handler();
     }
   }
   link.addEventListener('keydown', handleKey, true);
   row.querySelector('.actions').appendChild(link);
 }
 
 function initSettings() {
-  checkboxSendsCommand('discover-usb-devices-enable',
-                       'set-discover-usb-devices-enabled');
+  checkboxSendsCommand(
+      'discover-usb-devices-enable', 'set-discover-usb-devices-enabled');
   checkboxSendsCommand('port-forwarding-enable', 'set-port-forwarding-enabled');
-  checkboxSendsCommand('discover-tcp-devices-enable',
-                       'set-discover-tcp-targets-enabled');
+  checkboxSendsCommand(
+      'discover-tcp-devices-enable', 'set-discover-tcp-targets-enabled');
 
-  $('port-forwarding-config-open').addEventListener(
-      'click', openPortForwardingConfig);
-  $('tcp-discovery-config-open').addEventListener(
-      'click', openTargetsConfig);
+  $('port-forwarding-config-open')
+      .addEventListener('click', openPortForwardingConfig);
+  $('tcp-discovery-config-open').addEventListener('click', openTargetsConfig);
   $('config-dialog-close').addEventListener('click', function() {
     $('config-dialog').commit(true);
   });
 }
 
 function checkboxHandler(command, event) {
   sendCommand(command, event.target.checked);
 }
 
 function checkboxSendsCommand(id, command) {
@@ skipping 55 matching lines @@
   for (var key in data)
     list.createRow(key, data[key]);
   list.createRow(null, null);
 
   dialog.showModal();
   var defaultFocus = dialog.querySelector('.fresh .preselected');
   if (defaultFocus)
     defaultFocus.focus();
   else
     doneButton.focus();
-
 }
 
 function openPortForwardingConfig() {
   function createPortForwardingConfigLine(port, location) {
     var line = document.createElement('div');
     line.className = 'port-forwarding-pair config-list-row';
 
-    var portInput = createConfigField(port, 'port preselected',
-                                      'Port', validatePort);
+    var portInput =
+        createConfigField(port, 'port preselected', 'Port', validatePort);
     line.appendChild(portInput);
 
     var locationInput = createConfigField(
         location, 'location', 'IP address and port', validateLocation);
     locationInput.classList.add('primary');
     line.appendChild(locationInput);
     return line;
   }
 
   function commitPortForwardingConfig() {
     var config = {};
     filterList(['.port', '.location'], function(port, location) {
       config[port] = location;
     });
     sendCommand('set-port-forwarding-config', config);
   }
 
-  openConfigDialog('port-forwarding',
-             commitPortForwardingConfig,
-             createPortForwardingConfigLine,
-             window.portForwardingConfig);
+  openConfigDialog(
+      'port-forwarding', commitPortForwardingConfig,
+      createPortForwardingConfigLine, window.portForwardingConfig);
 }
 
 function openTargetsConfig() {
   function createTargetDiscoveryConfigLine(index, targetDiscovery) {
     var line = document.createElement('div');
     line.className = 'target-discovery-line config-list-row';
 
     var locationInput = createConfigField(
         targetDiscovery, 'location preselected', 'IP address and port',
         validateLocation);
     locationInput.classList.add('primary');
     line.appendChild(locationInput);
     return line;
   }
 
   function commitTargetDiscoveryConfig() {
     var entries = [];
     filterList(['.location'], function(location) {
       entries.push(location);
     });
     sendCommand('set-tcp-discovery-config', entries);
   }
 
-  openConfigDialog('target-discovery',
-             commitTargetDiscoveryConfig,
-             createTargetDiscoveryConfigLine,
-             window.targetDiscoveryConfig);
+  openConfigDialog(
+      'target-discovery', commitTargetDiscoveryConfig,
+      createTargetDiscoveryConfigLine, window.targetDiscoveryConfig);
 }
 
 function filterList(fieldSelectors, callback) {
   var lines = $('config-dialog').querySelectorAll('.config-list-row');
   for (var i = 0; i != lines.length; i++) {
     var line = lines[i];
     var values = [];
     for (var selector of fieldSelectors) {
       var input = line.querySelector(selector);
-      var value = input.classList.contains('invalid') ?
-                  input.lastValidValue :
-                  input.value;
+      var value = input.classList.contains('invalid') ? input.lastValidValue :
+                                                        input.value;
       if (!value)
         break;
       values.push(value);
     }
     if (values.length == fieldSelectors.length)
       callback.apply(null, values);
   }
 }
 
 function updateCheckbox(id, enabled) {
@@ skipping 22 matching lines @@
 }
 
 function updateTCPDiscoveryConfig(config) {
   window.targetDiscoveryConfig = config;
   $('tcp-discovery-config-open').disabled = !config;
 }
 
 function appendRow(list, lineFactory, key, value) {
   var line = lineFactory(key, value);
   line.lastElementChild.addEventListener('keydown', function(e) {
-    if (e.key == 'Tab' &&
-        !hasKeyModifiers(e) &&
-        line.classList.contains('fresh') &&
-        !line.classList.contains('empty')) {
+    if (e.key == 'Tab' && !hasKeyModifiers(e) &&
+        line.classList.contains('fresh') && !line.classList.contains('empty')) {
       // Tabbing forward on the fresh line, try create a new empty one.
       if (commitFreshLineIfValid(true))
         e.preventDefault();
     }
   });
 
   var lineDelete = document.createElement('div');
   lineDelete.className = 'close-button';
   lineDelete.addEventListener('click', function() {
     var newSelection = line.nextElementSibling || line.previousElementSibling;
     selectLine(newSelection, true);
     line.parentNode.removeChild(line);
     $('config-dialog').commit(false);
   });
   line.appendChild(lineDelete);
 
-  line.addEventListener(
-      'click', selectLine.bind(null, line, true));
-  line.addEventListener(
-      'focus', selectLine.bind(null, line, true));
+  line.addEventListener('click', selectLine.bind(null, line, true));
+  line.addEventListener('focus', selectLine.bind(null, line, true));
   checkEmptyLine(line);
 
   if (!key && !value)
     line.classList.add('fresh');
 
   return list.appendChild(line);
 }
 
 function validatePort(input) {
   var match = input.value.match(/^(\d+)$/);
@@ skipping 159 matching lines @@
     Array.prototype.forEach.call(
         deviceSection.querySelectorAll('.browser'), clearBrowserPorts);
   }
 
   Array.prototype.forEach.call(
       document.querySelectorAll('.device'), clearPorts);
 }
 
 document.addEventListener('DOMContentLoaded', onload);
 window.addEventListener('hashchange', onHashChange);