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

Unified Diff: tools/perf/measurements/session_restore.py

Issue 980073005: Move PageTest.ValidatePageSet special-case into profile creator. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 5 years, 10 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | tools/perf/profile_creators/small_profile_creator.py » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: tools/perf/measurements/session_restore.py
diff --git a/tools/perf/measurements/session_restore.py b/tools/perf/measurements/session_restore.py
index 6c35a1d5822ad3130eb078462839be5b934a7175..41b995703686fcefc0a90374cb4a8230d635c8de 100644
--- a/tools/perf/measurements/session_restore.py
+++ b/tools/perf/measurements/session_restore.py
@@ -46,20 +46,6 @@ class SessionRestore(startup.Startup):
# Overridden so that no page navigation occurs.
pass
- def ValidatePageSet(self, page_set):
- wpr_archive_names_to_page_urls = collections.defaultdict(list)
- # Construct the map from pages' wpr archive names to pages' urls.
- for page in page_set:
- if page.is_local:
- continue
- wpr_archive_name = page_set.WprFilePathForUserStory(page)
- wpr_archive_names_to_page_urls[wpr_archive_name].append(page.url)
-
- # Reject any pageset that contains more than one WPR archive.
- if len(wpr_archive_names_to_page_urls.keys()) > 1:
- raise Exception("Invalid pageset: more than 1 WPR archive found.: " +
- repr(wpr_archive_names_to_page_urls))
-
def DidStartBrowser(self, browser):
self._cpu_metric = cpu.CpuMetric(browser)
self._cpu_metric.Start(None, None)
« no previous file with comments | « no previous file | tools/perf/profile_creators/small_profile_creator.py » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698