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

Unified Diff: third_party/WebKit/Tools/Scripts/webkitpy/performance_tests/perftest.py

Issue 1839193004: Run auto-formatter (autopep8) on webkitpy. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Rebased Created 4 years, 9 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/performance_tests/perftest.py
diff --git a/third_party/WebKit/Tools/Scripts/webkitpy/performance_tests/perftest.py b/third_party/WebKit/Tools/Scripts/webkitpy/performance_tests/perftest.py
index 9459f2971da040543fdb775db7d22ea6babb9a84..d1e43c5062472accf2bfc5295f1eb03806ec5703 100644
--- a/third_party/WebKit/Tools/Scripts/webkitpy/performance_tests/perftest.py
+++ b/third_party/WebKit/Tools/Scripts/webkitpy/performance_tests/perftest.py
@@ -49,6 +49,7 @@ _log = logging.getLogger(__name__)
class PerfTestMetric(object):
+
def __init__(self, metric, unit=None, iterations=None):
# FIXME: Fix runner.js to report correct metric names
self._iterations = iterations or []
@@ -133,7 +134,7 @@ class PerfTest(object):
if should_log:
legacy_chromium_bot_compatible_name = self.test_name_without_file_extension().replace('/', ': ')
self.log_statistics(legacy_chromium_bot_compatible_name + ': ' + metric.name(),
- metric.flattened_iteration_values(), metric.unit())
+ metric.flattened_iteration_values(), metric.unit())
return results
@@ -157,7 +158,7 @@ class PerfTest(object):
_log.info('RESULT %s= %s %s' % (test_name, mean, unit))
_log.info('median= %s %s, stdev= %s %s, min= %s %s, max= %s %s' %
- (median, unit, stdev, unit, sorted_values[0], unit, sorted_values[-1], unit))
+ (median, unit, stdev, unit, sorted_values[0], unit, sorted_values[-1], unit))
_description_regex = re.compile(r'^Description: (?P<description>.*)$', re.IGNORECASE)
_metrics_regex = re.compile(r'^(?P<metric>Time|Malloc|JS Heap):')
@@ -251,7 +252,8 @@ class PerfTest(object):
re.compile(re.escape("""frame "<!--framePath //<!--frame0-->/<!--frame0-->-->" - has 1 onunload handler(s)""")),
# Following is for html5.html
re.compile(re.escape("""Blocked access to external URL http://www.whatwg.org/specs/web-apps/current-work/""")),
- re.compile(r"CONSOLE MESSAGE: (line \d+: )?Blocked script execution in '[A-Za-z0-9\-\.:]+' because the document's frame is sandboxed and the 'allow-scripts' permission is not set."),
+ re.compile(
+ r"CONSOLE MESSAGE: (line \d+: )?Blocked script execution in '[A-Za-z0-9\-\.:]+' because the document's frame is sandboxed and the 'allow-scripts' permission is not set."),
re.compile(r"CONSOLE MESSAGE: (line \d+: )?Not allowed to load local resource"),
# Dromaeo reports values for subtests. Ignore them for now.
re.compile(r'(?P<name>.+): \[(?P<values>(\d+(.\d+)?,\s+)*\d+(.\d+)?)\]'),
@@ -259,12 +261,15 @@ class PerfTest(object):
def _filter_output(self, output):
if output.error:
- output.error = '\n'.join([line for line in re.split('\n', output.error) if not self._should_ignore_line(self._lines_to_ignore_in_stderr, line)])
+ output.error = '\n'.join([line for line in re.split('\n', output.error)
+ if not self._should_ignore_line(self._lines_to_ignore_in_stderr, line)])
if output.text:
- output.text = '\n'.join([line for line in re.split('\n', output.text) if not self._should_ignore_line(self._lines_to_ignore_in_parser_result, line)])
+ output.text = '\n'.join([line for line in re.split('\n', output.text)
+ if not self._should_ignore_line(self._lines_to_ignore_in_parser_result, line)])
class SingleProcessPerfTest(PerfTest):
+
def __init__(self, port, test_name, test_path, test_runner_count=1):
super(SingleProcessPerfTest, self).__init__(port, test_name, test_path, test_runner_count)

Powered by Google App Engine
This is Rietveld 408576698