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

Unified Diff: tools/perf/page_sets/power_cases.py

Issue 1984473002: [Telemetry] Add BattOr benchmark. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 years, 7 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
« tools/perf/benchmarks/battor.py ('K') | « tools/perf/benchmarks/battor.py ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: tools/perf/page_sets/power_cases.py
diff --git a/tools/perf/page_sets/power_cases.py b/tools/perf/page_sets/power_cases.py
new file mode 100644
index 0000000000000000000000000000000000000000..06be3bad22353d90386a75e5a46fdeba059de3b5
--- /dev/null
+++ b/tools/perf/page_sets/power_cases.py
@@ -0,0 +1,39 @@
+# Copyright 2014 The Chromium Authors. All rights reserved.
charliea (OOO until 10-5) 2016/05/16 19:30:21 I think it makes sense to create our own directory
charliea (OOO until 10-5) 2016/05/16 19:30:21 nit: s/2014/2016
rnephew (Reviews Here) 2016/05/16 20:05:57 Those subdirectories look like they mostly contain
rnephew (Reviews Here) 2016/05/16 20:05:57 Done.
charliea (OOO until 10-5) 2016/05/17 13:44:26 It looks like https://code.google.com/p/chromium/c
+# Use of this source code is governed by a BSD-style license that can be
+# found in the LICENSE file.
charliea (OOO until 10-5) 2016/05/16 19:30:21 nit: blank line after license
rnephew (Reviews Here) 2016/05/16 20:05:57 Done.
+from telemetry.page import page as page_module
+from telemetry.page import shared_page_state
+from telemetry import story
+
+
+URL_LIST = [
+ 'https://webkit.org/blog-files/3d-transforms/poster-circle.html',
+ # Does not autoplay on android devices.
charliea (OOO until 10-5) 2016/05/16 19:30:21 s/android/Android
rnephew (Reviews Here) 2016/05/16 20:05:57 Done.
charliea (OOO until 10-5) 2016/05/17 13:44:26 Is the lack of autoplay going to be a problem on A
+ 'https://www.youtube.com/watch?v=3KANI2dpXLw?autoplay=1',
+ 'about:blank'
+]
+
+
+class PowerCasesPage(page_module.Page):
+
+ def __init__(self, url, page_set, name=''):
+ super(PowerCasesPage, self).__init__(
+ url=url, page_set=page_set, name=name,
+ credentials_path = 'data/credentials.json',
+ shared_page_state_class=shared_page_state.SharedMobilePageState)
+ self.archive_data_file = 'data/power_cases.json'
+
+ def RunPageInteractions(self, action_runner):
+ action_runner.Wait(30)
+
+
+class PowerCasesPageSet(story.StorySet):
+ """Power hungry pages, used for power testing."""
+
+ def __init__(self):
+ super(PowerCasesPageSet, self).__init__(
+ archive_data_file='data/power_cases.json',
+ cloud_storage_bucket=story.PARTNER_BUCKET)
+
+ for url in URL_LIST:
+ self.AddStory(PowerCasesPage(url, self))
« tools/perf/benchmarks/battor.py ('K') | « tools/perf/benchmarks/battor.py ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698