Reformat python code in third_party/blink/tools/blinkpy/formatter for PEP8

- added .style.yapf file
- reformat *.py files

Bug: 1051750
Change-Id: I902200e328a800ec23a4c34ce7896b5d25122b19
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2132791Reviewed-by: Kent Tamura <tkent@chromium.org>
Commit-Queue: Kent Tamura <tkent@chromium.org>
Cr-Commit-Position: refs/heads/master@{#758451}

third_party/blink/tools/blinkpy/formatter/.style.yapf

+[style]
+based_on_style = pep8

third_party/blink/tools/blinkpy/formatter/__main__.py

@@ -6,6 +6,5 @@ import sys
 
 from blinkpy.formatter.main import main
 
-
 if __name__ == '__main__':
     sys.exit(main())

third_party/blink/tools/blinkpy/formatter/fix_docstrings.py

 # 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.
-
 """A 2to3 fixer that reformats docstrings.
 
 This should transform docstrings to be closer to the conventions in pep-0257;
@@ -33,9 +32,8 @@ class FixDocstrings(BaseFix):
         """
         # Pylint incorrectly warns that there's no member simple_stmt on python_symbols
         # because the attribute is set dynamically.  pylint: disable=no-member
-        return (node.value.startswith('"""') and
-                node.prev_sibling is None and
-                node.parent.type == python_symbols.simple_stmt)
+        return (node.value.startswith('"""') and node.prev_sibling is None
+                and node.parent.type == python_symbols.simple_stmt)
 
     def transform(self, node, results):
         # First, strip whitespace at the beginning and end.

third_party/blink/tools/blinkpy/formatter/fix_double_quote_strings.py

 # Copyright 2014 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.
-
 """A 2to3 fixer that converts all string literals to use double quotes.
 
 Strings that contain double quotes will not be modified. Prefixed string

third_party/blink/tools/blinkpy/formatter/fix_single_quote_strings.py

 # Copyright 2014 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.
-
 """A 2to3 fixer that converts all string literals to use single quotes.
 
 Strings that contain single quotes will not be modified. Prefixed string
@@ -19,7 +18,8 @@ class FixSingleQuoteStrings(BaseFix):
     _accept_type = token.STRING
 
     def match(self, node):
-        res = node.value.startswith('"') and not node.value.startswith('"""') and "'" not in node.value[1:-1]
+        res = (node.value.startswith('"') and not node.value.startswith('"""')
+               and "'" not in node.value[1:-1])
         return res
 
     def transform(self, node, results):

third_party/blink/tools/blinkpy/formatter/main.py

@@ -15,20 +15,51 @@ from blinkpy.third_party import autopep8
 
 def parse_args(args=None):
     parser = argparse.ArgumentParser()
-    parser.add_argument('--chromium', action='store_const', dest='style', const='chromium', default='blink',
-                        help="Format according to Chromium's Python coding styles instead of Blink's.")
-    parser.add_argument('--no-backups', action='store_false', default=True, dest='backup',
-                        help='Do not back up files before overwriting them.')
-    parser.add_argument('-j', '--jobs', metavar='n', type=int, default=0,
-                        help='Number of parallel jobs; match CPU count if less than 1.')
-    parser.add_argument('files', nargs='*', default=['-'],
-                        help="files to format or '-' for standard in")
-    parser.add_argument('--double-quote-strings', action='store_const', dest='quoting', const='double', default='single',
-                        help='Rewrite string literals to use double quotes instead of single quotes.')
-    parser.add_argument('--no-autopep8', action='store_true',
-                        help='Skip the autopep8 code-formatting step.')
-    parser.add_argument('--leave-strings-alone', action='store_true',
-                        help='Do not reformat string literals to use a consistent quote style.')
+    parser.add_argument(
+        '--chromium',
+        action='store_const',
+        dest='style',
+        const='chromium',
+        default='blink',
+        help=
+        "Format according to Chromium's Python coding styles instead of Blink's."
+    )
+    parser.add_argument(
+        '--no-backups',
+        action='store_false',
+        default=True,
+        dest='backup',
+        help='Do not back up files before overwriting them.')
+    parser.add_argument(
+        '-j',
+        '--jobs',
+        metavar='n',
+        type=int,
+        default=0,
+        help='Number of parallel jobs; match CPU count if less than 1.')
+    parser.add_argument(
+        'files',
+        nargs='*',
+        default=['-'],
+        help="files to format or '-' for standard in")
+    parser.add_argument(
+        '--double-quote-strings',
+        action='store_const',
+        dest='quoting',
+        const='double',
+        default='single',
+        help=
+        'Rewrite string literals to use double quotes instead of single quotes.'
+    )
+    parser.add_argument(
+        '--no-autopep8',
+        action='store_true',
+        help='Skip the autopep8 code-formatting step.')
+    parser.add_argument(
+        '--leave-strings-alone',
+        action='store_true',
+        help='Do not reformat string literals to use a consistent quote style.'
+    )
     return parser.parse_args(args=args)
 
 
@@ -46,13 +77,17 @@ def main(host=None, args=None):
 
     if options.files == ['-']:
         host = host or SystemHost()
-        host.print_(reformat_source(host.stdin.read(), autopep8_options, fixers, '<stdin>'), end='')
+        host.print_(
+            reformat_source(host.stdin.read(), autopep8_options, fixers,
+                            '<stdin>'),
+            end='')
         return
 
     # We create the arglist before checking if we need to create a Host, because a
     # real host is non-picklable and can't be passed to host.executive.map().
 
-    arglist = [(host, name, autopep8_options, fixers, options.backup) for name in options.files]
+    arglist = [(host, name, autopep8_options, fixers, options.backup)
+               for name in options.files]
     host = host or SystemHost()
 
     host.executive.map(_reformat_thunk, arglist, processes=options.jobs)
@@ -61,18 +96,24 @@ def main(host=None, args=None):
 def _autopep8_options_for_style(style):
     return {
         None: [],
-        'blink': autopep8.parse_args([
+        'blink':
+        autopep8.parse_args([
             '--aggressive',
-            '--max-line-length', '132',
+            '--max-line-length',
+            '132',
             '--ignore=E309',
-            '--indent-size', '4',
+            '--indent-size',
+            '4',
             '',
         ]),
-        'chromium': autopep8.parse_args([
+        'chromium':
+        autopep8.parse_args([
             '--aggressive',
-            '--max-line-length', '80',
+            '--max-line-length',
+            '80',
             '--ignore=E309',
-            '--indent-size', '2',
+            '--indent-size',
+            '2',
             '',
         ]),
     }.get(style)
@@ -107,8 +148,8 @@ def reformat_source(source, autopep8_options, fixers, name):
         tmp_str = autopep8.fix_code(tmp_str, autopep8_options)
 
     if fixers:
-        tool = lib2to3.refactor.RefactoringTool(fixer_names=fixers,
-                                                explicit=fixers)
+        tool = lib2to3.refactor.RefactoringTool(
+            fixer_names=fixers, explicit=fixers)
         tmp_str = unicode(tool.refactor_string(tmp_str, name=name))
 
     return tmp_str

third_party/blink/tools/blinkpy/formatter/main_unittest.py

@@ -8,7 +8,6 @@ import unittest
 from blinkpy.common.system.system_host_mock import MockSystemHost
 from blinkpy.formatter.main import main
 
-
 ACTUAL_INPUT = '''
 def foo():
     """triple-quoted docstring"""
@@ -20,7 +19,6 @@ def foo():
         pass
 '''
 
-
 EXPECTED_BLINK_OUTPUT = '''
 def foo():
     """triple-quoted docstring"""
@@ -35,7 +33,6 @@ def foo():
         pass
 '''
 
-
 EXPECTED_CHROMIUM_OUTPUT = '''
 def foo():
   """triple-quoted docstring"""
@@ -69,32 +66,33 @@ class TestMain(unittest.TestCase):
 
     def test_files_blink(self):
         host = MockSystemHost()
-        host.filesystem.files = {
-            'test.py': ACTUAL_INPUT}
+        host.filesystem.files = {'test.py': ACTUAL_INPUT}
         main(host, ['test.py'])
         self.assertEqual(host.filesystem.files, {
             'test.py': EXPECTED_BLINK_OUTPUT,
-            'test.py.bak': ACTUAL_INPUT})
+            'test.py.bak': ACTUAL_INPUT
+        })
 
     def test_files_blink_no_backup(self):
         host = MockSystemHost()
-        host.filesystem.files = {
-            'test.py': ACTUAL_INPUT}
+        host.filesystem.files = {'test.py': ACTUAL_INPUT}
         main(host, ['--no-backups', 'test.py'])
-        self.assertEqual(host.filesystem.files, {
-            'test.py': EXPECTED_BLINK_OUTPUT})
+        self.assertEqual(host.filesystem.files,
+                         {'test.py': EXPECTED_BLINK_OUTPUT})
 
     def test_stdin_blink(self):
         host = MockSystemHost()
         host.stdin = StringIO.StringIO(ACTUAL_INPUT)
         main(host, ['-'])
-        self.assertMultiLineEqual(host.stdout.getvalue(), EXPECTED_BLINK_OUTPUT)
+        self.assertMultiLineEqual(host.stdout.getvalue(),
+                                  EXPECTED_BLINK_OUTPUT)
 
     def test_stdin_chromium(self):
         host = MockSystemHost()
         host.stdin = StringIO.StringIO(ACTUAL_INPUT)
         main(host, ['--chromium', '-'])
-        self.assertMultiLineEqual(host.stdout.getvalue(), EXPECTED_CHROMIUM_OUTPUT)
+        self.assertMultiLineEqual(host.stdout.getvalue(),
+                                  EXPECTED_CHROMIUM_OUTPUT)
 
     def test_stdin_no_changes(self):
         host = MockSystemHost()
@@ -106,7 +104,8 @@ class TestMain(unittest.TestCase):
         host = MockSystemHost()
         host.stdin = StringIO.StringIO(ACTUAL_INPUT)
         main(host, ['--no-autopep8', '--double-quote-strings', '-'])
-        self.assertMultiLineEqual(host.stdout.getvalue(), EXPECTED_ONLY_DOUBLE_QUOTED_OUTPUT)
+        self.assertMultiLineEqual(host.stdout.getvalue(),
+                                  EXPECTED_ONLY_DOUBLE_QUOTED_OUTPUT)
 
     def test_format_docstrings(self):
         host = MockSystemHost()
@@ -123,7 +122,8 @@ def f():
     return x
 ''')
         main(host, ['-'])
-        self.assertMultiLineEqual(host.stdout.getvalue(), '''
+        self.assertMultiLineEqual(
+            host.stdout.getvalue(), '''
 def f():
     """triple-quoted docstring
     with multiple lines
@@ -144,7 +144,8 @@ def f():
      """
 ''')
         main(host, ['-'])
-        self.assertMultiLineEqual(host.stdout.getvalue(), '''
+        self.assertMultiLineEqual(
+            host.stdout.getvalue(), '''
 def f():
     """This is a docstring
        With extra indentation on this line.