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

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

Issue 2917313003: Verify explicitly set names for memory benchmarks (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/long_running_idle_google_cases.py ('k') | no next file » | 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)
72 73
73 for url in top_10_mobile.URL_LIST: 74 for url in top_10_mobile.URL_LIST:
74 # We name pages so their foreground/background counterparts are easy 75 # We name pages so their foreground/background counterparts are easy
75 # to identify. For example 'http://google.com' becomes 76 # to identify. For example 'http://google.com' becomes
76 # 'http_google_com' and 'after_http_google_com' respectively. 77 # 'http_google_com' and 'after_http_google_com' respectively.
77 name = re.sub(r'\W+', '_', url) 78 name = re.sub(r'\W+', '_', url)
78 self.AddStory(ForegroundPage(self, name, url)) 79 self.AddStory(ForegroundPage(self, name, url))
79 self.AddStory(BackgroundPage(self, 'after_' + name)) 80 self.AddStory(BackgroundPage(self, 'after_' + name))
80 81
81 82
82 class MemoryTop10MobileRealistic(MemoryTop10Mobile): 83 class MemoryTop10MobileRealistic(MemoryTop10Mobile):
83 """Top 10 mobile user story in realistic mode. 84 """Top 10 mobile user story in realistic mode.
84 85
85 This means, in particular, neither forced GCs nor clearing caches. 86 This means, in particular, neither forced GCs nor clearing caches.
86 """ 87 """
87 DETERMINISTIC_MODE = False 88 DETERMINISTIC_MODE = False
OLDNEW
« no previous file with comments | « tools/perf/page_sets/long_running_idle_google_cases.py ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698