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

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

Issue 2941663002: Removing verify_names=True now that the default for verify_names is True (Closed)
Patch Set: 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
« no previous file with comments | « tools/perf/page_sets/media_cns_cases.py ('k') | tools/perf/page_sets/mse_cases.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 50 matching lines...) Expand 10 before | Expand all | Expand 10 after
61 keyevent.KEYCODE_BACK) 61 keyevent.KEYCODE_BACK)
62 62
63 63
64 class MemoryTop10Mobile(story.StorySet): 64 class MemoryTop10Mobile(story.StorySet):
65 """User story to measure foreground/background memory in top 10 mobile.""" 65 """User story to measure foreground/background memory in top 10 mobile."""
66 DETERMINISTIC_MODE = True 66 DETERMINISTIC_MODE = True
67 67
68 def __init__(self): 68 def __init__(self):
69 super(MemoryTop10Mobile, self).__init__( 69 super(MemoryTop10Mobile, self).__init__(
70 archive_data_file='data/memory_top_10_mobile.json', 70 archive_data_file='data/memory_top_10_mobile.json',
71 cloud_storage_bucket=story.PARTNER_BUCKET, 71 cloud_storage_bucket=story.PARTNER_BUCKET)
72 verify_names=True)
73 72
74 for url in top_10_mobile.URL_LIST: 73 for url in top_10_mobile.URL_LIST:
75 # We name pages so their foreground/background counterparts are easy 74 # We name pages so their foreground/background counterparts are easy
76 # to identify. For example 'http://google.com' becomes 75 # to identify. For example 'http://google.com' becomes
77 # 'http_google_com' and 'after_http_google_com' respectively. 76 # 'http_google_com' and 'after_http_google_com' respectively.
78 name = re.sub(r'\W+', '_', url) 77 name = re.sub(r'\W+', '_', url)
79 self.AddStory(ForegroundPage(self, name, url)) 78 self.AddStory(ForegroundPage(self, name, url))
80 self.AddStory(BackgroundPage(self, 'after_' + name)) 79 self.AddStory(BackgroundPage(self, 'after_' + name))
OLDNEW
« no previous file with comments | « tools/perf/page_sets/media_cns_cases.py ('k') | tools/perf/page_sets/mse_cases.py » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698