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

Unified Diff: tools/telemetry/telemetry/core/backends/chrome/android_browser_finder.py

Issue 994583004: Revert of Set use_rndis_forwarder in platform backend. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: 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
Index: tools/telemetry/telemetry/core/backends/chrome/android_browser_finder.py
diff --git a/tools/telemetry/telemetry/core/backends/chrome/android_browser_finder.py b/tools/telemetry/telemetry/core/backends/chrome/android_browser_finder.py
index 4525bda8b4a43e22c3934ebbb84c66297fb59d58..348d1354c7f97c748065a16b509715f6f9e48bef 100644
--- a/tools/telemetry/telemetry/core/backends/chrome/android_browser_finder.py
+++ b/tools/telemetry/telemetry/core/backends/chrome/android_browser_finder.py
@@ -103,9 +103,14 @@
def Create(self, finder_options):
self._InitPlatformIfNeeded()
+
+ use_rndis_forwarder = (finder_options.android_rndis or
+ finder_options.browser_options.netsim or
+ platform.GetHostPlatform().GetOSName() != 'linux')
browser_backend = android_browser_backend.AndroidBrowserBackend(
self._platform_backend,
finder_options.browser_options, self._backend_settings,
+ use_rndis_forwarder,
output_profile_path=finder_options.output_profile_path,
extensions_to_load=finder_options.extensions_to_load,
target_arch=finder_options.target_arch)
@@ -205,5 +210,5 @@
device = android_device.GetDevice(finder_options)
if not device:
return []
- android_platform = platform.GetPlatformForDevice(device, finder_options)
+ android_platform = platform.GetPlatformForDevice(device)
return _FindAllPossibleBrowsers(finder_options, android_platform)

Powered by Google App Engine
This is Rietveld 408576698