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

Unified Diff: tools/telemetry/telemetry/web_perf/metrics/rendering_stats.py

Issue 739623005: [Telemetry] Resolve errors due to new pylint checker. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: ready to land Created 6 years, 1 month 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: tools/telemetry/telemetry/web_perf/metrics/rendering_stats.py
diff --git a/tools/telemetry/telemetry/web_perf/metrics/rendering_stats.py b/tools/telemetry/telemetry/web_perf/metrics/rendering_stats.py
index 6925b77fb0057bba56ee5a79034245b5c728d7ad..2dd9819b70c8a3daecb3c0318e4b5f46974ef066 100644
--- a/tools/telemetry/telemetry/web_perf/metrics/rendering_stats.py
+++ b/tools/telemetry/telemetry/web_perf/metrics/rendering_stats.py
@@ -25,7 +25,7 @@ END_COMP_NAME = 'INPUT_EVENT_LATENCY_TERMINATED_FRAME_SWAP_COMPONENT'
# Name for a main thread scroll update latency event.
SCROLL_UPDATE_EVENT_NAME = 'InputLatency:ScrollUpdate'
# Name for a gesture scroll update latency event.
-GESTURE_SCROLL_UPDATE_EVENT_NAME = 'InputLatency:GestureScrollUpdate'
+GESTURE_SCROLL_UPDATE_EVENT_NAME = 'InputLatency:GestureScrollUpdate'
def GetInputLatencyEvents(process, timeline_range):
@@ -82,7 +82,7 @@ def ComputeInputEventLatencies(input_events):
elif BEGIN_SCROLL_UPDATE_COMP_NAME in data:
start_time = data[BEGIN_SCROLL_UPDATE_COMP_NAME]['time']
else:
- raise ValueError, 'LatencyInfo has no begin component'
+ raise ValueError('LatencyInfo has no begin component')
latency = (end_time - start_time) / 1000.0
input_event_latencies.append((start_time, event.name, latency))
@@ -126,7 +126,7 @@ class RenderingStats(object):
All *_time values are measured in milliseconds.
"""
- assert(len(timeline_ranges) > 0)
+ assert len(timeline_ranges) > 0
# Find the top level process with rendering stats (browser or renderer).
if HasRenderingStats(browser_process):
timestamp_process = browser_process

Powered by Google App Engine
This is Rietveld 408576698