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

Side by Side Diff: tools/perf/measurements/startup.py

Issue 1438533003: Revert of telemetry: Fix startup benchmarks so that all metrics get recorded. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Rebase. Created 5 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 unified diff | Download patch
« no previous file with comments | « no previous file | tools/perf/page_sets/blank_page.py » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
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 from telemetry.page import page_test 5 from telemetry.page import page_test
6 6
7 from metrics import keychain_metric 7 from metrics import keychain_metric
8 from metrics import startup_metric 8 from metrics import startup_metric
9 9
10 10
(...skipping 11 matching lines...) Expand all
22 self._cold = cold 22 self._cold = cold
23 23
24 def CustomizeBrowserOptions(self, options): 24 def CustomizeBrowserOptions(self, options):
25 if self._cold: 25 if self._cold:
26 options.clear_sytem_cache_for_browser_and_profile_on_start = True 26 options.clear_sytem_cache_for_browser_and_profile_on_start = True
27 options.AppendExtraBrowserArgs([ 27 options.AppendExtraBrowserArgs([
28 '--enable-stats-collection-bindings' 28 '--enable-stats-collection-bindings'
29 ]) 29 ])
30 keychain_metric.KeychainMetric.CustomizeBrowserOptions(options) 30 keychain_metric.KeychainMetric.CustomizeBrowserOptions(options)
31 31
32 def RunNavigateSteps(self, page, tab):
33 # Overriden so that no page navigation occurs - startup to the NTP.
34 pass
35
32 def ValidateAndMeasurePage(self, page, tab, results): 36 def ValidateAndMeasurePage(self, page, tab, results):
33 keychain_metric.KeychainMetric().AddResults(tab, results) 37 keychain_metric.KeychainMetric().AddResults(tab, results)
34 startup_metric.StartupMetric().AddResults(tab, results) 38 startup_metric.StartupMetric().AddResults(tab, results)
35 39
36 40
37 class StartWithUrl(Startup): 41 class StartWithUrl(Startup):
38 """Performs a measurement of Chromium's performance starting with a URL. 42 """Performs a measurement of Chromium's performance starting with a URL.
39 43
40 Uses cold start if cold==True, otherwise uses warm start. A cold start means 44 Uses cold start if cold==True, otherwise uses warm start. A cold start means
41 none of the Chromium files are in the disk cache. A warm start assumes the OS 45 none of the Chromium files are in the disk cache. A warm start assumes the OS
42 has already cached much of Chromium's content. For warm tests, you should 46 has already cached much of Chromium's content. For warm tests, you should
43 repeat the page set to ensure it's cached. 47 repeat the page set to ensure it's cached.
44 48
45 The startup URL is taken from the page's startup_url. This 49 The startup URL is taken from the page's startup_url. This
46 allows the testing of multiple different URLs in a single benchmark. 50 allows the testing of multiple different URLs in a single benchmark.
47 """ 51 """
48 52
49 def __init__(self, cold=False): 53 def __init__(self, cold=False):
50 super(StartWithUrl, self).__init__(cold=cold) 54 super(StartWithUrl, self).__init__(cold=cold)
OLDNEW
« no previous file with comments | « no previous file | tools/perf/page_sets/blank_page.py » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698