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

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

Issue 1037483004: Telemetry benchmark key_desktop_move_pages. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Added comments. Created 5 years, 9 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 | « tools/perf/page_sets/key_desktop_move_cases.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/top_pages.py
diff --git a/tools/perf/page_sets/top_pages.py b/tools/perf/page_sets/top_pages.py
index 69b06ac552ca52f552e6759b286a2ad60eeffb32..582cf986c3730dc44f3ddabebf83542cf59e1f6e 100644
--- a/tools/perf/page_sets/top_pages.py
+++ b/tools/perf/page_sets/top_pages.py
@@ -54,50 +54,6 @@ class GmailPage(TopPages):
action_runner.WaitForJavaScriptCondition(
'window.gmonkey !== undefined &&'
'document.getElementById("gb") !== null')
- # This check is needed for gmonkey to load completely.
- action_runner.WaitForJavaScriptCondition(
- 'document.readyState == "complete"')
-
-
-class GmailMouseScrollPage(GmailPage):
-
- """ Why: productivity, top google properties """
-
- def RunPageInteractions(self, action_runner):
- action_runner.ExecuteJavaScript('''
- gmonkey.load('2.0', function(api) {
- window.__scrollableElementForTelemetry = api.getScrollableElement();
- });''')
- action_runner.WaitForJavaScriptCondition(
- 'window.__scrollableElementForTelemetry != null')
- scrollbar_x, start_y, end_y = self._CalculateScrollBarRatios(action_runner)
-
- interaction = action_runner.BeginGestureInteraction(
- 'DragAction')
- action_runner.DragPage(left_start_ratio=scrollbar_x,
- top_start_ratio=start_y, left_end_ratio=scrollbar_x,
- top_end_ratio=end_y, speed_in_pixels_per_second=100,
- element_function='window.__scrollableElementForTelemetry')
- interaction.End()
-
- def _CalculateScrollBarRatios(self, action_runner):
- viewport_height = float(action_runner.EvaluateJavaScript(
- 'window.__scrollableElementForTelemetry.clientHeight'))
- content_height = float(action_runner.EvaluateJavaScript(
- 'window.__scrollableElementForTelemetry.scrollHeight'))
- viewport_width = float(action_runner.EvaluateJavaScript(
- 'window.__scrollableElementForTelemetry.offsetWidth'))
- scrollbar_width = float(action_runner.EvaluateJavaScript('''
- window.__scrollableElementForTelemetry.offsetWidth -
- window.__scrollableElementForTelemetry.scrollWidth'''))
-
- # This calculation is correct only when the element doesn't have border or
- # padding or scroll buttons (eg: gmail mail element).
- scrollbar_start_mid_y = viewport_height / (2 * content_height)
- scrollbar_end_mid_y = 1 - scrollbar_start_mid_y
- scrollbar_mid_x_offset = scrollbar_width / (2 * viewport_width)
- scrollbar_mid_x = 1 - scrollbar_mid_x_offset
- return scrollbar_mid_x, scrollbar_start_mid_y, scrollbar_end_mid_y
class GoogleCalendarPage(TopPages):
@@ -142,21 +98,6 @@ class GoogleDocPage(TopPages):
action_runner.WaitForJavaScriptCondition(
'document.getElementsByClassName("kix-appview-editor").length')
-class GoogleMapsPage(TopPages):
-
- """ Why: productivity, top google properties; Supports drag gesturee """
-
- def __init__(self, page_set):
- super(GoogleMapsPage, self).__init__(
- url='https://www.google.co.uk/maps/@51.5043968,-0.1526806',
- page_set=page_set,
- name='Maps')
-
- def RunNavigateSteps(self, action_runner):
- super(GoogleMapsPage, self).RunNavigateSteps(action_runner)
- action_runner.WaitForElement(selector='.widget-scene-canvas')
- action_runner.WaitForElement(selector='.widget-zoom-in')
- action_runner.WaitForElement(selector='.widget-zoom-out')
class GooglePlusPage(TopPages):
« no previous file with comments | « tools/perf/page_sets/key_desktop_move_cases.py ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698