commit 032591d6921120a1744e893e8712ee4bb84b8d62
author Mathias Bynens <mathias@chromium.org> Mon Oct 21 11:51:31 2019 +0200
committer Commit Bot <commit-bot@chromium.org> Mon Oct 21 13:02:51 2019 +0000
tree 57a762db9b66aa88bc9b63e3a3e2cc51cfcbabd3
parent 544479c4899d7f4a284c4c3ec1153a884fd4fddb

Fix formatting

Running `npm run format-py` on a fresh checkout results in some
changes. This patch commits those changes so that they do not
become part of unrelated CLs.

Change-Id: Ie8051c7907ac9624e9fe870e2137b9a9fe2769c3
Reviewed-on: https://chromium-review.googlesource.com/c/devtools/devtools-frontend/+/1871869
Auto-Submit: Mathias Bynens <mathias@chromium.org>
Reviewed-by: Yang Guo <yangguo@chromium.org>
Commit-Queue: Mathias Bynens <mathias@chromium.org>

PRESUBMIT.py

diff --git a/PRESUBMIT.py b/PRESUBMIT.py
index f674eff..4702f1e 100644
--- a/PRESUBMIT.py
+++ b/PRESUBMIT.py
@@ -59,8 +59,9 @@
     with open(eslint_ignore_path, 'r') as ignore_manifest:
         for line in ignore_manifest:
             ignore_files.append(line.strip())
-    formattable_files = [affected_file for affected_file in affected_files
-                         if all(ignore_file not in affected_file for ignore_file in ignore_files)]
+    formattable_files = [
+        affected_file for affected_file in affected_files if all(ignore_file not in affected_file for ignore_file in ignore_files)
+    ]
     if len(formattable_files) == 0:
         return []
 
@@ -77,7 +78,8 @@
 
     # Use eslint to autofix the braces.
     # Also fix semicolon to avoid confusing clang-format.
-    eslint_process = popen([devtools_paths.node_path(), devtools_paths.eslint_path(), '--config', '.eslintrc.js', '--fix'] + affected_files)
+    eslint_process = popen(
+        [devtools_paths.node_path(), devtools_paths.eslint_path(), '--config', '.eslintrc.js', '--fix'] + affected_files)
     eslint_process.communicate()
 
     # Need to run clang-format again to align the braces
@@ -173,17 +175,19 @@
                 results.append(output_api.PresubmitError(("%s:%d uses ::shadow selector") % (f.LocalPath(), line_number)))
     return results
 
+
 def _CommonChecks(input_api, output_api):
-  """Checks common to both upload and commit."""
-  results = []
-  results.extend(input_api.canned_checks.CheckOwnersFormat(input_api, output_api))
-  results.extend(input_api.canned_checks.CheckOwners(input_api, output_api))
-  results.extend(input_api.canned_checks.CheckChangeHasNoCrAndHasOnlyOneEol(input_api, output_api))
-  results.extend(input_api.canned_checks.CheckChangeHasNoStrayWhitespace(input_api, output_api))
-  results.extend(input_api.canned_checks.CheckGenderNeutral(input_api, output_api))
-  results.extend(_CheckDevtoolsLocalizableResources(input_api, output_api))
-  results.extend(_CheckDevtoolsLocalization(input_api, output_api))
-  return results
+    """Checks common to both upload and commit."""
+    results = []
+    results.extend(input_api.canned_checks.CheckOwnersFormat(input_api, output_api))
+    results.extend(input_api.canned_checks.CheckOwners(input_api, output_api))
+    results.extend(input_api.canned_checks.CheckChangeHasNoCrAndHasOnlyOneEol(input_api, output_api))
+    results.extend(input_api.canned_checks.CheckChangeHasNoStrayWhitespace(input_api, output_api))
+    results.extend(input_api.canned_checks.CheckGenderNeutral(input_api, output_api))
+    results.extend(_CheckDevtoolsLocalizableResources(input_api, output_api))
+    results.extend(_CheckDevtoolsLocalization(input_api, output_api))
+    return results
+
 
 def CheckChangeOnUpload(input_api, output_api):
     results = []
@@ -199,8 +203,7 @@
 def CheckChangeOnCommit(input_api, output_api):
     results = []
     results.extend(_CommonChecks(input_api, output_api))
-    results.extend(input_api.canned_checks.CheckChangeHasDescription(
-      input_api, output_api))
+    results.extend(input_api.canned_checks.CheckChangeHasDescription(input_api, output_api))
     return results