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

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

Issue 1510833002: [Telemetry] Googler Mobile First benchmark (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 5 years 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/perf/benchmarks/memory_infra.py
diff --git a/tools/perf/benchmarks/memory_infra.py b/tools/perf/benchmarks/memory_infra.py
index 318eaa9cc08b09bec477b4dd46706fbdf3bcc15b..f3e4e21e12d04853e8f73e1845aef65387ac6db2 100644
--- a/tools/perf/benchmarks/memory_infra.py
+++ b/tools/perf/benchmarks/memory_infra.py
@@ -80,6 +80,25 @@ class MemoryHealthPlan(_MemoryInfra):
# TODO(bashi): Workaround for http://crbug.com/532075
# @benchmark.Enabled('android') shouldn't be needed.
@benchmark.Enabled('android')
+class GooglerMobileFirst(_MemoryInfra):
+ """Timeline based benchmark for the Googler Mobile First use case."""
+
+ page_set = page_sets.GooglerMobileFirstStorySet
+
+ @classmethod
+ def Name(cls):
+ return 'memory.googler_mobile_first'
+
+ @classmethod
+ def ShouldDisable(cls, possible_browser):
+ # Benchmark requires Gmail app only available on Nexus devices. (maybe?)
+ # See http://crbug.com/546842
+ return 'nexus' not in possible_browser.platform.GetDeviceTypeName().lower()
perezju 2015/12/08 11:47:08 I'm thinking instead maybe we could also land this
+
+
+# TODO(bashi): Workaround for http://crbug.com/532075
+# @benchmark.Enabled('android') shouldn't be needed.
+@benchmark.Enabled('android')
class RendererMemoryBlinkMemoryMobile(_MemoryInfra):
"""Timeline based benchmark for measuring memory consumption on mobile
sites on which blink's memory consumption is relatively high."""

Powered by Google App Engine
This is Rietveld 408576698