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

Side by Side Diff: tools/perf/benchmarks/maps.py

Issue 816353008: [Telemetry] Explicitly define Name() method for all perf benchmarks (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 5 years, 11 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 unified diff | Download patch
OLDNEW
1 # Copyright 2013 The Chromium Authors. All rights reserved. 1 # Copyright 2013 The Chromium Authors. All rights reserved.
2 # Use of this source code is governed by a BSD-style license that can be 2 # Use of this source code is governed by a BSD-style license that can be
3 # found in the LICENSE file. 3 # found in the LICENSE file.
4 4
5 """Runs a Google Maps performance test. 5 """Runs a Google Maps performance test.
6 Rerforms several common navigation actions on the map (pan, zoom, rotate)""" 6 Rerforms several common navigation actions on the map (pan, zoom, rotate)"""
7 7
8 import os 8 import os
9 import re 9 import re
10 10
(...skipping 32 matching lines...) Expand 10 before | Expand all | Expand 10 after
43 def RunNavigateSteps(self, action_runner): 43 def RunNavigateSteps(self, action_runner):
44 action_runner.NavigateToPage(self) 44 action_runner.NavigateToPage(self)
45 action_runner.WaitForJavaScriptCondition('window.testDone') 45 action_runner.WaitForJavaScriptCondition('window.testDone')
46 46
47 47
48 @benchmark.Disabled 48 @benchmark.Disabled
49 class MapsBenchmark(benchmark.Benchmark): 49 class MapsBenchmark(benchmark.Benchmark):
50 """Basic Google Maps benchmarks.""" 50 """Basic Google Maps benchmarks."""
51 test = _MapsMeasurement 51 test = _MapsMeasurement
52 52
53 @classmethod
54 def Name(cls):
55 return 'maps'
56
53 def CreatePageSet(self, options): 57 def CreatePageSet(self, options):
54 page_set_path = os.path.join( 58 page_set_path = os.path.join(
55 util.GetChromiumSrcDir(), 'tools', 'perf', 'page_sets') 59 util.GetChromiumSrcDir(), 'tools', 'perf', 'page_sets')
56 ps = page_set_module.PageSet( 60 ps = page_set_module.PageSet(
57 archive_data_file='data/maps.json', file_path=page_set_path, 61 archive_data_file='data/maps.json', file_path=page_set_path,
58 bucket=page_set_module.PUBLIC_BUCKET) 62 bucket=page_set_module.PUBLIC_BUCKET)
59 ps.AddUserStory(MapsPage(ps, ps.base_dir)) 63 ps.AddUserStory(MapsPage(ps, ps.base_dir))
60 return ps 64 return ps
61 65
62 class MapsNoVsync(MapsBenchmark): 66 class MapsNoVsync(MapsBenchmark):
63 """Runs the Google Maps benchmark with Vsync disabled""" 67 """Runs the Google Maps benchmark with Vsync disabled"""
64 tag = 'novsync' 68 tag = 'novsync'
65 69
70 @classmethod
71 def Name(cls):
72 return 'maps.novsync'
73
66 def CustomizeBrowserOptions(self, options): 74 def CustomizeBrowserOptions(self, options):
67 options.AppendExtraBrowserArgs('--disable-gpu-vsync') 75 options.AppendExtraBrowserArgs('--disable-gpu-vsync')
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698