Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(201)

Unified Diff: third_party/WebKit/Tools/Scripts/webkitpy/layout_tests/port/factory.py

Issue 2256793002: Make docstrings more consistent using format-webkitpy. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Make fix_docstrings match function stricter and fix style/main.py. Created 4 years, 3 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: third_party/WebKit/Tools/Scripts/webkitpy/layout_tests/port/factory.py
diff --git a/third_party/WebKit/Tools/Scripts/webkitpy/layout_tests/port/factory.py b/third_party/WebKit/Tools/Scripts/webkitpy/layout_tests/port/factory.py
index 4045d665e634fd28c36ddb0eeb255976e8ac1540..94d0f1e711f4f0741be3d377963eb40200cf010c 100644
--- a/third_party/WebKit/Tools/Scripts/webkitpy/layout_tests/port/factory.py
+++ b/third_party/WebKit/Tools/Scripts/webkitpy/layout_tests/port/factory.py
@@ -150,7 +150,8 @@ class PortFactory(object):
def get(self, port_name=None, options=None, **kwargs):
"""Returns an object implementing the Port interface. If
port_name is None, this routine attempts to guess at the most
- appropriate port on this platform."""
+ appropriate port on this platform.
+ """
port_name = port_name or self._default_port(options)
_check_configuration_and_target(self._host.filesystem, options)
@@ -185,7 +186,8 @@ class PortFactory(object):
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"""
+ If platform is not specified, we will glob-match all ports
+ """
platform = platform or '*'
return fnmatch.filter(self._host.builders.all_port_names(), platform)

Powered by Google App Engine
This is Rietveld 408576698