Reformat webkitpy.layout_tests w/ format-webkitpy.

Tools/Scripts/webkitpy/layout_tests/port/factory.py

 # Copyright (C) 2010 Google Inc. All rights reserved.
 #
 # Redistribution and use in source and binary forms, with or without
 # modification, are permitted provided that the following conditions are
 # met:
 #
 #     * Redistributions of source code must retain the above copyright
 # notice, this list of conditions and the following disclaimer.
 #     * Redistributions in binary form must reproduce the above
 # copyright notice, this list of conditions and the following disclaimer
@@ skipping 20 matching lines @@
 import fnmatch
 import optparse
 import re
 
 from webkitpy.layout_tests.port import builders
 
 
 def platform_options(use_globs=False):
     return [
         optparse.make_option('--platform', action='store',
-            help=('Glob-style list of platform/ports to use (e.g., "mac*")' if use_globs else 'Platform to use (e.g., "mac-lion")')),
+                             help=('Glob-style list of platform/ports to use (e.g., "mac*")' if use_globs else 'Platform to use (e.g., "mac-lion")')),
 
         # FIXME: Update run_webkit_tests.sh, any other callers to no longer pass --chromium, then remove this flag.
         optparse.make_option('--chromium', action='store_const', dest='platform',
-            const=('chromium*' if use_globs else 'chromium'),
-            help=('Alias for --platform=chromium*' if use_globs else 'Alias for --platform=chromium')),
+                             const=('chromium*' if use_globs else 'chromium'),
+                             help=('Alias for --platform=chromium*' if use_globs else 'Alias for --platform=chromium')),
 
         optparse.make_option('--android', action='store_const', dest='platform',
-            const=('android*' if use_globs else 'android'),
-            help=('Alias for --platform=android*' if use_globs else 'Alias for --platform=android')),
-        ]
+                             const=('android*' if use_globs else 'android'),
+                             help=('Alias for --platform=android*' if use_globs else 'Alias for --platform=android')),
+    ]
 
 
 def configuration_options():
     return [
-        optparse.make_option("-t", "--target", dest="configuration",
-                             help="specify the target configuration to use (Debug/Release)"),
-        optparse.make_option('--debug', action='store_const', const='Debug', dest="configuration",
-            help='Set the configuration to Debug'),
-        optparse.make_option('--release', action='store_const', const='Release', dest="configuration",
-            help='Set the configuration to Release'),
-        ]
-
+        optparse.make_option('-t', '--target', dest='configuration',
+                             help='specify the target configuration to use (Debug/Release)'),
+        optparse.make_option('--debug', action='store_const', const='Debug', dest='configuration',
+                             help='Set the configuration to Debug'),
+        optparse.make_option('--release', action='store_const', const='Release', dest='configuration',
+                             help='Set the configuration to Release'),
+    ]
 
 
 def _builder_options(builder_name):
-    configuration = "Debug" if re.search(r"[d|D](ebu|b)g", builder_name) else "Release"
-    is_webkit2 = builder_name.find("WK2") != -1
+    configuration = 'Debug' if re.search(r"[d|D](ebu|b)g", builder_name) else 'Release'
+    is_webkit2 = builder_name.find('WK2') != -1
     builder_name = builder_name
     return optparse.Values({'builder_name': builder_name, 'configuration': configuration})
 
 
 class PortFactory(object):
     PORT_CLASSES = (
         'android.AndroidPort',
         'linux.LinuxPort',
         'mac.MacPort',
         'win.WinPort',
@@ skipping 22 matching lines @@
 
         # FIXME(steveblock): There's no longer any need to pass '--platform
         # chromium' on the command line so we can remove this logic.
         if port_name == 'chromium':
             port_name = self._host.platform.os_name
 
         if 'browser_test' in port_name:
             module_name, class_name = port_name.rsplit('.', 1)
             module = __import__(module_name, globals(), locals(), [], -1)
             port_class_name = module.get_port_class_name(class_name)
-            if port_class_name != None:
+            if port_class_name is not None:
                 cls = module.__dict__[port_class_name]
                 port_name = cls.determine_full_port_name(self._host, options, class_name)
                 return cls(self._host, port_name, options=options, **kwargs)
         else:
             for port_class in self.PORT_CLASSES:
                 module_name, class_name = port_class.rsplit('.', 1)
                 module = __import__(module_name, globals(), locals(), [], -1)
                 cls = module.__dict__[class_name]
                 if port_name.startswith(cls.port_name):
                     port_name = cls.determine_full_port_name(self._host, options, port_name)
                     return cls(self._host, port_name, options=options, **kwargs)
         raise NotImplementedError('unsupported platform: "%s"' % port_name)
 
     def all_port_names(self, platform=None):
         """Return a list of all valid, fully-specified, "real" port names.
 
         This is the list of directories that are used as actual baseline_paths()
         by real ports. This does not include any "fake" names like "test"
         or "mock-mac", and it does not include any directories that are not.
 
         If platform is not specified, we will glob-match all ports"""
         platform = platform or '*'
         return fnmatch.filter(builders.all_port_names(), platform)
 
     def get_from_builder_name(self, builder_name):
         port_name = builders.port_name_for_builder_name(builder_name)
         assert port_name, "unrecognized builder name '%s'" % builder_name
         return self.get(port_name, _builder_options(builder_name))