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

Side by Side Diff: tools/perf/page_sets/intl_es_fr_pt-BR.py

Issue 2935783004: Explicitly setting names for stories in used page_sets (Closed)
Patch Set: Explicitly setting names for stories in used page_sets Created 3 years, 6 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 2014 The Chromium Authors. All rights reserved. 1 # Copyright 2014 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 page_sets import page_cycler_story 5 from page_sets import page_cycler_story
6 from telemetry import story 6 from telemetry import story
7 from telemetry.page import cache_temperature as cache_temperature_module 7 from telemetry.page import cache_temperature as cache_temperature_module
8 from telemetry.page import shared_page_state 8 from telemetry.page import shared_page_state
9 9
10 10
11 class IntlEsFrPtBrPage(page_cycler_story.PageCyclerStory): 11 class IntlEsFrPtBrPage(page_cycler_story.PageCyclerStory):
12 12
13 def __init__(self, url, page_set, cache_temperature=None): 13 def __init__(self, url, page_set, cache_temperature=None):
14 super(IntlEsFrPtBrPage, self).__init__( 14 super(IntlEsFrPtBrPage, self).__init__(
15 url=url, page_set=page_set, 15 url=url, page_set=page_set,
16 shared_page_state_class=shared_page_state.SharedDesktopPageState, 16 shared_page_state_class=shared_page_state.SharedDesktopPageState,
17 cache_temperature=cache_temperature) 17 cache_temperature=cache_temperature,
18 name=url)
18 self.archive_data_file = 'data/intl_es_fr_pt-BR.json' 19 self.archive_data_file = 'data/intl_es_fr_pt-BR.json'
19 20
20 21
21 class IntlEsFrPtBrPageSet(story.StorySet): 22 class IntlEsFrPtBrPageSet(story.StorySet):
22 23
23 """ 24 """
24 Popular pages in Romance languages Spanish, French and Brazilian Portuguese. 25 Popular pages in Romance languages Spanish, French and Brazilian Portuguese.
25 """ 26 """
26 27
27 def __init__(self, cache_temperatures=None): 28 def __init__(self, cache_temperatures=None):
28 super(IntlEsFrPtBrPageSet, self).__init__( 29 super(IntlEsFrPtBrPageSet, self).__init__(
29 archive_data_file='data/intl_es_fr_pt-BR.json', 30 archive_data_file='data/intl_es_fr_pt-BR.json',
30 cloud_storage_bucket=story.PARTNER_BUCKET) 31 cloud_storage_bucket=story.PARTNER_BUCKET,
32 verify_names=True)
31 if cache_temperatures is None: 33 if cache_temperatures is None:
32 cache_temperatures = [cache_temperature_module.ANY] 34 cache_temperatures = [cache_temperature_module.ANY]
33 35
34 urls_list = [ 36 urls_list = [
35 'http://elmundo.es/', 37 'http://elmundo.es/',
36 'http://terra.es/', 38 'http://terra.es/',
37 # pylint: disable=line-too-long 39 # pylint: disable=line-too-long
38 'http://www.ebay.es/sch/i.html?_sacat=382&_trkparms=clkid%3D65489713890604 85883&_qi=RTM1381637', 40 'http://www.ebay.es/sch/i.html?_sacat=382&_trkparms=clkid%3D65489713890604 85883&_qi=RTM1381637',
39 'http://www.eltiempo.es/talavera-de-la-reina.html', 41 'http://www.eltiempo.es/talavera-de-la-reina.html',
40 'http://www.free.fr/adsl/index.html', 42 'http://www.free.fr/adsl/index.html',
41 'http://www.voila.fr/', 43 'http://www.voila.fr/',
42 'http://www.leboncoin.fr/annonces/offres/limousin/', 44 'http://www.leboncoin.fr/annonces/offres/limousin/',
43 'http://www.orange.fr/', 45 'http://www.orange.fr/',
44 # Why: #5 site in Brazil 46 # Why: #5 site in Brazil
45 'http://www.uol.com.br/', 47 'http://www.uol.com.br/',
46 # Why: #10 site in Brazil 48 # Why: #10 site in Brazil
47 # pylint: disable=line-too-long 49 # pylint: disable=line-too-long
48 'http://produto.mercadolivre.com.br/MLB-468424957-pelicula-protetora-smart phone-h5500-e-h5300-43-frete-free-_JM' 50 'http://produto.mercadolivre.com.br/MLB-468424957-pelicula-protetora-smart phone-h5500-e-h5300-43-frete-free-_JM'
49 ] 51 ]
50 52
51 for url in urls_list: 53 for url in urls_list:
52 for temp in cache_temperatures: 54 for temp in cache_temperatures:
53 self.AddStory(IntlEsFrPtBrPage(url, self, cache_temperature=temp)) 55 self.AddStory(IntlEsFrPtBrPage(url, self, cache_temperature=temp))
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698