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

Side by Side Diff: tools/perf/page_sets/memory_top_10_mobile.py

Issue 2228413003: [tools/perf] Enable many more lint checks for perf directory. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Address Petr's comment Created 4 years, 4 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
« no previous file with comments | « tools/perf/page_sets/dual_browser_story.py ('k') | tools/perf/page_sets/typical_25.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 2015 The Chromium Authors. All rights reserved. 1 # Copyright 2015 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 import re 5 import re
6 6
7 from telemetry.page import page as page_module 7 from telemetry.page import page as page_module
8 from telemetry.page import shared_page_state 8 from telemetry.page import shared_page_state
9 from telemetry import story 9 from telemetry import story
10 10
(...skipping 59 matching lines...) Expand 10 before | Expand all | Expand 10 after
70 70
71 def __init__(self): 71 def __init__(self):
72 super(MemoryTop10Mobile, self).__init__( 72 super(MemoryTop10Mobile, self).__init__(
73 archive_data_file='data/memory_top_10_mobile.json', 73 archive_data_file='data/memory_top_10_mobile.json',
74 cloud_storage_bucket=story.PARTNER_BUCKET) 74 cloud_storage_bucket=story.PARTNER_BUCKET)
75 75
76 for url in top_10_mobile.URL_LIST: 76 for url in top_10_mobile.URL_LIST:
77 # We name pages so their foreground/background counterparts are easy 77 # We name pages so their foreground/background counterparts are easy
78 # to identify. For example 'http://google.com' becomes 78 # to identify. For example 'http://google.com' becomes
79 # 'http_google_com' and 'after_http_google_com' respectively. 79 # 'http_google_com' and 'after_http_google_com' respectively.
80 name = re.sub('\W+', '_', url) 80 name = re.sub(r'\W+', '_', url)
81 self.AddStory(ForegroundPage(self, name, url)) 81 self.AddStory(ForegroundPage(self, name, url))
82 self.AddStory(BackgroundPage(self, 'after_' + name)) 82 self.AddStory(BackgroundPage(self, 'after_' + name))
83 83
84 84
85 class MemoryTop10MobileRealistic(MemoryTop10Mobile): 85 class MemoryTop10MobileRealistic(MemoryTop10Mobile):
86 """Top 10 mobile user story in realistic mode. 86 """Top 10 mobile user story in realistic mode.
87 87
88 This means, in particular, neither forced GCs nor clearing caches. 88 This means, in particular, neither forced GCs nor clearing caches.
89 """ 89 """
90 DETERMINISTIC_MODE = False 90 DETERMINISTIC_MODE = False
OLDNEW
« no previous file with comments | « tools/perf/page_sets/dual_browser_story.py ('k') | tools/perf/page_sets/typical_25.py » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698