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

Unified Diff: tools/perf/benchmarks/smoothness.py

Issue 2304703003: perf: Don't record first_gesture_scroll_update_latency in smoothness.tough_ad_cases (Closed)
Patch Set: 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
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: tools/perf/benchmarks/smoothness.py
diff --git a/tools/perf/benchmarks/smoothness.py b/tools/perf/benchmarks/smoothness.py
index c650764b7f4c7a69fbfc034b3178d9f7ab096b2d..168e862c2123fea125e0bdc077cde4d73fe3b28a 100644
--- a/tools/perf/benchmarks/smoothness.py
+++ b/tools/perf/benchmarks/smoothness.py
@@ -459,6 +459,12 @@ class SmoothnessToughAdCases(_Smoothness):
def ShouldDisable(cls, possible_browser):
return cls.IsSvelte(possible_browser) # http://crbug.com/555089
+ @classmethod
+ def ValueCanBeAddedPredicate(cls, value, is_first_result):
+ del is_first_result # unused
+ # These pages don't scroll so it's not necessary to measure input latency.
+ return value.name != 'first_gesture_scroll_update_latency'
+
# http://crbug.com/522619 (mac/win)
@benchmark.Disabled('win', 'mac')
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698