Add background stories to MobileSystemHealth

tools/perf/page_sets/system_health/system_health_story.py

 # Copyright 2016 The Chromium Authors. All rights reserved.
 # Use of this source code is governed by a BSD-style license that can be
 # found in the LICENSE file.
 
 from page_sets.system_health import platforms
 
 from telemetry import decorators
 from telemetry.page import page
 from telemetry.page import shared_page_state
 
+from devil.android.sdk import intent # pylint: disable=import-error
+from devil.android.sdk import keyevent # pylint: disable=import-error
 
 # Extra wait time after the page has loaded required by the loading metric. We
 # use it in all benchmarks to avoid divergence between benchmarks.
 # TODO(petrcermak): Switch the memory benchmarks to use it as well.
 _WAIT_TIME_AFTER_LOAD = 10
 
 
 class _SystemHealthSharedState(shared_page_state.SharedPageState):
   """Shared state which enables disabling stories on individual platforms.
 
@@ skipping 38 matching lines @@
   """Abstract base class for System Health user stories."""
   __metaclass__ = _MetaSystemHealthStory
 
   # The full name of a single page story has the form CASE:GROUP:PAGE (e.g.
   # 'load:search:google').
   NAME = NotImplemented
   URL = NotImplemented
   ABSTRACT_STORY = True
   SUPPORTED_PLATFORMS = platforms.ALL_PLATFORMS
 
-  def __init__(self, story_set, take_memory_measurement):
+  def __init__(
+      self, story_set, take_memory_measurement, measure_in_background):
     case, group, _ = self.NAME.split(':')
     super(SystemHealthStory, self).__init__(
         shared_page_state_class=_SystemHealthSharedState, page_set=story_set,
         name=self.NAME, url=self.URL,
         credentials_path='../data/credentials.json',
         grouping_keys={'case': case, 'group': group})
     self._take_memory_measurement = take_memory_measurement
+    self._measure_in_background = measure_in_background
 
   @classmethod
   def CanRun(cls, possible_browser):
     if (decorators.ShouldSkip(cls, possible_browser)[0] or
         cls.ShouldDisable(possible_browser)):
       return False
     return True
 
   @classmethod
   def ShouldDisable(cls, possible_browser):
     """Override this method to disable a story under specific conditions.
 
     This method is modelled after telemetry.benchmark.Benchmark.ShouldDisable().
     """
     del possible_browser
     return False
 
   def _Measure(self, action_runner):
     if self._take_memory_measurement:
       action_runner.MeasureMemory(deterministic_mode=True)
     else:
       action_runner.Wait(_WAIT_TIME_AFTER_LOAD)
 
   def _Login(self, action_runner):
     pass
 
   def _DidLoadDocument(self, action_runner):
     pass
 
+  def _BackgroundChromeIfNecessary(self, action_runner):
+    if self._measure_in_background:
+      self._BackgroundChrome(action_runner)
+
+  def _ForegroundChromeIfNecessary(self, action_runner):
+    if self._measure_in_background:
+      self._ForegroundChrome(action_runner)
+
+  def _BackgroundChrome(self, action_runner):
+    platform = action_runner.tab.browser.platform
+    platform.LaunchAndroidApplication(
+        intent.Intent(
+            package='com.google.android.deskclock',
+            activity='com.android.deskclock.DeskClock'),
+        app_has_webviews=False)

[hjd, 2016/10/03 11:23:27]

The block above seems kind of ugly, it seems like background/foreground should
be pushed into platform so we just always get the right impl for the OS we're
on?

+
+  def _ForegroundChrome(self, action_runner):
+    platform = action_runner.tab.browser.platform
+    platform.android_action_runner.InputKeyEvent(keyevent.KEYCODE_BACK)
+
   def RunNavigateSteps(self, action_runner):
     self._Login(action_runner)
     super(SystemHealthStory, self).RunNavigateSteps(action_runner)
 
   def RunPageInteractions(self, action_runner):
     action_runner.tab.WaitForDocumentReadyStateToBeComplete()
     self._DidLoadDocument(action_runner)
+    self._BackgroundChromeIfNecessary(action_runner)
     self._Measure(action_runner)
+    self._ForegroundChromeIfNecessary(action_runner)
+