DO NOT SUBMIT: what chrome/browser/resources/ could eventually look like with clang-format

chrome/browser/resources/sandbox_internals/sandbox_internals.js

Index: chrome/browser/resources/sandbox_internals/sandbox_internals.js
diff --git a/chrome/browser/resources/sandbox_internals/sandbox_internals.js b/chrome/browser/resources/sandbox_internals/sandbox_internals.js
index 02479b54f3b46ef3b523acc0ae8c09eb67cd9f42..81c4ad5cf7c5e5ed00a0f8aede380540f1508f35 100644
--- a/chrome/browser/resources/sandbox_internals/sandbox_internals.js
+++ b/chrome/browser/resources/sandbox_internals/sandbox_internals.js
@@ -3,137 +3,138 @@
 // found in the LICENSE file.
 
 (function() {
-  let GOOD = 'good';
-  let BAD = 'bad';
-  let INFO = 'info';
-
-  /**
-   * Adds a row to the sandbox status table.
-   * @param {string} name The name of the status item.
-   * @param {string} value The status of the item.
-   * @param {string?} cssClass A CSS class to apply to the row.
-   * @return {Element} The newly added TR.
-   */
-  function addStatusRow(name, value, cssClass) {
-    let row = cr.doc.createElement('tr');
-
-    let nameCol = row.appendChild(cr.doc.createElement('td'));
-    let valueCol = row.appendChild(cr.doc.createElement('td'));
-
-    nameCol.textContent = name;
-    valueCol.textContent = value;
-
-    if (cssClass != null) {
-      nameCol.classList.add(cssClass);
-      valueCol.classList.add(cssClass);
-    }
-
-    $('sandbox-status').appendChild(row);
-    return row;
-  }
-
-  /**
-   * Adds a status row that reports either Yes or No.
-   * @param {string} name The name of the status item.
-   * @param {boolean} result The status (good/bad) result.
-   * @return {Element} The newly added TR.
-   */
-  function addGoodBadRow(name, result) {
-    return addStatusRow(name, result ? 'Yes' : 'No', result ? GOOD : BAD);
+let GOOD = 'good';
+let BAD = 'bad';
+let INFO = 'info';
+
+/**
+ * Adds a row to the sandbox status table.
+ * @param {string} name The name of the status item.
+ * @param {string} value The status of the item.
+ * @param {string?} cssClass A CSS class to apply to the row.
+ * @return {Element} The newly added TR.
+ */
+function addStatusRow(name, value, cssClass) {
+  let row = cr.doc.createElement('tr');
+
+  let nameCol = row.appendChild(cr.doc.createElement('td'));
+  let valueCol = row.appendChild(cr.doc.createElement('td'));
+
+  nameCol.textContent = name;
+  valueCol.textContent = value;
+
+  if (cssClass != null) {
+    nameCol.classList.add(cssClass);
+    valueCol.classList.add(cssClass);
   }
 
-  /**
-   * Reports the overall sandbox status evaluation message.
-   * @param {boolean}
-   */
-  function setEvaluation(result) {
-    let message = result ? 'You are adequately sandboxed.'
-                         : 'You are NOT adequately sandboxed.';
-    $('evaluation').innerText = message;
-  }
-
-  /**
-   * Main page handler for Android.
-   */
-  function androidHandler() {
-    chrome.getAndroidSandboxStatus((status) => {
-      var isIsolated = false;
-      var isTsync = false;
-      var isChromeSeccomp = false;
-
-      addStatusRow('PID', status.pid, INFO);
-      addStatusRow('UID', status.uid, INFO);
-      isIsolated = status.secontext.indexOf(':isolated_app:') != -1;
-      addStatusRow('SELinux Context', status.secontext,
-          isIsolated ? GOOD : BAD);
-
-      let procStatus = status.procStatus.split('\n');
-      for (let line of procStatus) {
-        if (line.startsWith('Seccomp')) {
-          var value = line.split(':')[1].trim();
-          var cssClass = BAD;
-          if (value == '2') {
-            value = 'Yes - TSYNC (' + line + ')';
-            cssClass = GOOD;
-            isTsync = true;
-          } else if (value == '1') {
-            value = 'Yes (' + line + ')';
-          } else {
-            value = line;
-          }
-          addStatusRow('Seccomp-BPF Enabled (Kernel)', value, cssClass);
-          break;
+  $('sandbox-status').appendChild(row);
+  return row;
+}
+
+/**
+ * Adds a status row that reports either Yes or No.
+ * @param {string} name The name of the status item.
+ * @param {boolean} result The status (good/bad) result.
+ * @return {Element} The newly added TR.
+ */
+function addGoodBadRow(name, result) {
+  return addStatusRow(name, result ? 'Yes' : 'No', result ? GOOD : BAD);
+}
+
+/**
+ * Reports the overall sandbox status evaluation message.
+ * @param {boolean}
+ */
+function setEvaluation(result) {
+  let message = result ? 'You are adequately sandboxed.' :
+                         'You are NOT adequately sandboxed.';
+  $('evaluation').innerText = message;
+}
+
+/**
+ * Main page handler for Android.
+ */
+function androidHandler() {
+  chrome.getAndroidSandboxStatus((status) => {
+    var isIsolated = false;
+    var isTsync = false;
+    var isChromeSeccomp = false;
+
+    addStatusRow('PID', status.pid, INFO);
+    addStatusRow('UID', status.uid, INFO);
+    isIsolated = status.secontext.indexOf(':isolated_app:') != -1;
+    addStatusRow('SELinux Context', status.secontext, isIsolated ? GOOD : BAD);
+
+    let procStatus = status.procStatus.split('\n');
+    for (let line of procStatus) {
+      if (line.startsWith('Seccomp')) {
+        var value = line.split(':')[1].trim();
+        var cssClass = BAD;
+        if (value == '2') {
+          value = 'Yes - TSYNC (' + line + ')';
+          cssClass = GOOD;
+          isTsync = true;
+        } else if (value == '1') {
+          value = 'Yes (' + line + ')';
+        } else {
+          value = line;
         }
+        addStatusRow('Seccomp-BPF Enabled (Kernel)', value, cssClass);
+        break;
       }
+    }
 
-      var seccompStatus = 'Unknown';
-      switch (status.seccompStatus) {
-        case 0:
-          seccompStatus = 'Not Supported';
-          break;
-        case 1:
-          seccompStatus = 'Run-time Detection Failed';
-          break;
-        case 2:
-          seccompStatus = 'Disabled by Field Trial';
-          break;
-        case 3:
-          seccompStatus = 'Enabled by Field Trial (not started)';
-          break;
-        case 4:
-          seccompStatus = 'Sandbox Started';
-          isChromeSeccomp = true;
-          break;
-      }
-      addStatusRow('Seccomp-BPF Enabled (Chrome)', seccompStatus,
-          status.seccompStatus == 4 ? GOOD : BAD);
-
-      addStatusRow('Android Build ID', status.androidBuildId, INFO);
+    var seccompStatus = 'Unknown';
+    switch (status.seccompStatus) {
+      case 0:
+        seccompStatus = 'Not Supported';
+        break;
+      case 1:
+        seccompStatus = 'Run-time Detection Failed';
+        break;
+      case 2:
+        seccompStatus = 'Disabled by Field Trial';
+        break;
+      case 3:
+        seccompStatus = 'Enabled by Field Trial (not started)';
+        break;
+      case 4:
+        seccompStatus = 'Sandbox Started';
+        isChromeSeccomp = true;
+        break;
+    }
+    addStatusRow(
+        'Seccomp-BPF Enabled (Chrome)', seccompStatus,
+        status.seccompStatus == 4 ? GOOD : BAD);
 
-      setEvaluation(isIsolated && isTsync && isChromeSeccomp);
-    });
-  }
+    addStatusRow('Android Build ID', status.androidBuildId, INFO);
 
-  /**
-   * Main page handler for desktop Linux.
-   */
-  function linuxHandler() {
-    addGoodBadRow('SUID Sandbox', loadTimeData.getBoolean('suid'));
-    addGoodBadRow('Namespace Sandbox', loadTimeData.getBoolean('userNs'));
-    addGoodBadRow('PID namespaces', loadTimeData.getBoolean('pidNs'));
-    addGoodBadRow('Network namespaces', loadTimeData.getBoolean('netNs'));
-    addGoodBadRow('Seccomp-BPF sandbox', loadTimeData.getBoolean('seccompBpf'));
-    addGoodBadRow('Seccomp-BPF sandbox supports TSYNC',
-        loadTimeData.getBoolean('seccompTsync'));
-    addGoodBadRow('Yama LSM Enforcing', loadTimeData.getBoolean('yama'));
-    setEvaluation(loadTimeData.getBoolean('sandboxGood'));
-  }
-
-  document.addEventListener('DOMContentLoaded', () => {
-    if (cr.isAndroid) {
-      androidHandler();
-    } else {
-      linuxHandler();
-    }
+    setEvaluation(isIsolated && isTsync && isChromeSeccomp);
   });
+}
+
+/**
+ * Main page handler for desktop Linux.
+ */
+function linuxHandler() {
+  addGoodBadRow('SUID Sandbox', loadTimeData.getBoolean('suid'));
+  addGoodBadRow('Namespace Sandbox', loadTimeData.getBoolean('userNs'));
+  addGoodBadRow('PID namespaces', loadTimeData.getBoolean('pidNs'));
+  addGoodBadRow('Network namespaces', loadTimeData.getBoolean('netNs'));
+  addGoodBadRow('Seccomp-BPF sandbox', loadTimeData.getBoolean('seccompBpf'));
+  addGoodBadRow(
+      'Seccomp-BPF sandbox supports TSYNC',
+      loadTimeData.getBoolean('seccompTsync'));
+  addGoodBadRow('Yama LSM Enforcing', loadTimeData.getBoolean('yama'));
+  setEvaluation(loadTimeData.getBoolean('sandboxGood'));
+}
+
+document.addEventListener('DOMContentLoaded', () => {
+  if (cr.isAndroid) {
+    androidHandler();
+  } else {
+    linuxHandler();
+  }
+});
 })();