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

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

Issue 737403002: Revert of telemetry: Remove command line args from page test (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: 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/perf/benchmarks/page_cycler.py
diff --git a/tools/perf/benchmarks/page_cycler.py b/tools/perf/benchmarks/page_cycler.py
index c10b811ed87c78e2f434d278b8dbb8bddfe84f26..ad855ddf5cbea53a1cc69c79b9ce798d725ebc46 100644
--- a/tools/perf/benchmarks/page_cycler.py
+++ b/tools/perf/benchmarks/page_cycler.py
@@ -7,69 +7,61 @@
from telemetry import benchmark
-class _PageCycler(benchmark.Benchmark):
+class PageCyclerDhtml(benchmark.Benchmark):
+ test = page_cycler.PageCycler
+ page_set = page_sets.DhtmlPageSet
options = {'pageset_repeat': 10}
- @classmethod
- def AddBenchmarkCommandLineArgs(cls, parser):
- parser.add_option('--v8-object-stats',
- action='store_true',
- help='Enable detailed V8 object statistics.')
- parser.add_option('--report-speed-index',
- action='store_true',
- help='Enable the speed index metric.')
-
- parser.add_option('--cold-load-percent', type='int', default=50,
- help='%d of page visits for which a cold load is forced')
-
- def CreatePageTest(self, options):
- return page_cycler.PageCycler(
- page_repeat = options.page_repeat,
- pageset_repeat = options.pageset_repeat,
- cold_load_percent = options.cold_load_percent,
- record_v8_object_stats = options.v8_object_stats,
- report_speed_index = options.report_speed_index)
-
-
-class PageCyclerDhtml(_PageCycler):
- page_set = page_sets.DhtmlPageSet
-
-
-class PageCyclerIntlArFaHe(_PageCycler):
+class PageCyclerIntlArFaHe(benchmark.Benchmark):
+ test = page_cycler.PageCycler
page_set = page_sets.IntlArFaHePageSet
+ options = {'pageset_repeat': 10}
@benchmark.Disabled('win') # crbug.com/388337
-class PageCyclerIntlEsFrPtBr(_PageCycler):
+class PageCyclerIntlEsFrPtBr(benchmark.Benchmark):
+ test = page_cycler.PageCycler
page_set = page_sets.IntlEsFrPtBrPageSet
+ options = {'pageset_repeat': 10}
-class PageCyclerIntlHiRu(_PageCycler):
+class PageCyclerIntlHiRu(benchmark.Benchmark):
+ test = page_cycler.PageCycler
page_set = page_sets.IntlHiRuPageSet
+ options = {'pageset_repeat': 10}
@benchmark.Disabled('android', 'win') # crbug.com/379564, crbug.com/330909
-class PageCyclerIntlJaZh(_PageCycler):
+class PageCyclerIntlJaZh(benchmark.Benchmark):
+ test = page_cycler.PageCycler
page_set = page_sets.IntlJaZhPageSet
+ options = {'pageset_repeat': 10}
-class PageCyclerIntlKoThVi(_PageCycler):
+class PageCyclerIntlKoThVi(benchmark.Benchmark):
+ test = page_cycler.PageCycler
page_set = page_sets.IntlKoThViPageSet
+ options = {'pageset_repeat': 10}
-class PageCyclerMorejs(_PageCycler):
+class PageCyclerMorejs(benchmark.Benchmark):
+ test = page_cycler.PageCycler
page_set = page_sets.MorejsPageSet
+ options = {'pageset_repeat': 10}
-class PageCyclerMoz(_PageCycler):
+class PageCyclerMoz(benchmark.Benchmark):
+ test = page_cycler.PageCycler
page_set = page_sets.MozPageSet
+ options = {'pageset_repeat': 10}
@benchmark.Disabled('linux', 'win') # crbug.com/353260
-class PageCyclerNetsimTop10(_PageCycler):
+class PageCyclerNetsimTop10(benchmark.Benchmark):
"""Measures load time of the top 10 sites under simulated cable network."""
tag = 'netsim'
+ test = page_cycler.PageCycler
page_set = page_sets.Top10PageSet
options = {
'cold_load_percent': 100,
@@ -85,22 +77,30 @@
@benchmark.Enabled('android')
-class PageCyclerTop10Mobile(_PageCycler):
+class PageCyclerTop10Mobile(benchmark.Benchmark):
+ test = page_cycler.PageCycler
page_set = page_sets.Top10MobilePageSet
+ options = {'pageset_repeat': 10}
@benchmark.Disabled
-class PageCyclerKeyMobileSites(_PageCycler):
+class PageCyclerKeyMobileSites(benchmark.Benchmark):
+ test = page_cycler.PageCycler
page_set = page_sets.KeyMobileSitesPageSet
+ options = {'pageset_repeat': 10}
@benchmark.Disabled('android') # crbug.com/357326
-class PageCyclerToughLayoutCases(_PageCycler):
+class PageCyclerToughLayoutCases(benchmark.Benchmark):
+ test = page_cycler.PageCycler
page_set = page_sets.ToughLayoutCasesPageSet
+ options = {'pageset_repeat': 10}
# crbug.com/273986: This test is really flakey on xp.
# cabug.com/341843: This test is always timing out on Android.
@benchmark.Disabled('android', 'win')
-class PageCyclerTypical25(_PageCycler):
+class PageCyclerTypical25(benchmark.Benchmark):
+ test = page_cycler.PageCycler
page_set = page_sets.Typical25PageSet
+ options = {'pageset_repeat': 10}
« no previous file with comments | « tools/perf/benchmarks/benchmark_smoke_unittest.py ('k') | tools/perf/benchmarks/rasterize_and_record_micro.py » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698