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

Unified Diff: telemetry/telemetry/internal/backends/chrome/android_browser_backend.py

Issue 2398683006: [telemetry] Install PushAppsToBackground apk (Closed)
Patch Set: add method to test fake Created 4 years, 2 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
Index: telemetry/telemetry/internal/backends/chrome/android_browser_backend.py
diff --git a/telemetry/telemetry/internal/backends/chrome/android_browser_backend.py b/telemetry/telemetry/internal/backends/chrome/android_browser_backend.py
index a90604a818cb9f2070c2f4392c3c59b09154a8b4..d4c7493bd7adc881189f7a68aca33057300a629f 100644
--- a/telemetry/telemetry/internal/backends/chrome/android_browser_backend.py
+++ b/telemetry/telemetry/internal/backends/chrome/android_browser_backend.py
@@ -172,14 +172,14 @@ class AndroidBrowserBackend(chrome_browser_backend.ChromeBrowserBackend):
'Timed out waiting for browser to come back foreground.')
def Background(self):
- # Launch clock app, pushing Chrome to the background.
- # TODO(crbug.com/586148): The clock app isn't necessarily on every phone,
- # replace this with the PushAppsToBackground instead.
+ package = 'org.chromium.push_apps_to_background'
+ activity = package + '.PushAppsToBackgroundActivity'
self.device.StartActivity(
- intent.Intent(package='com.google.android.deskclock',
- activity='com.android.deskclock.DeskClock',
- action=None,
- flags=[intent.FLAG_ACTIVITY_RESET_TASK_IF_NEEDED]),
+ intent.Intent(
+ package=package,
+ activity=activity,
+ action=None,
+ flags=[intent.FLAG_ACTIVITY_RESET_TASK_IF_NEEDED]),
blocking=True)
def GetBrowserStartupArgs(self):
« no previous file with comments | « telemetry/telemetry/internal/backends/android_app_backend.py ('k') | telemetry/telemetry/internal/binary_dependencies.json » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698