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

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

Issue 860913003: Set use_rndis_forwarder in platform backend. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: move to platform. Created 5 years, 11 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_backend.py
diff --git a/tools/telemetry/telemetry/core/backends/chrome/android_browser_backend.py b/tools/telemetry/telemetry/core/backends/chrome/android_browser_backend.py
index 9170014530c71746b162154eaaf3cb4d9fde2e4a..d18ee012e28df3e7c590119ba1861c22b75fb840 100644
--- a/tools/telemetry/telemetry/core/backends/chrome/android_browser_backend.py
+++ b/tools/telemetry/telemetry/core/backends/chrome/android_browser_backend.py
@@ -25,8 +25,8 @@ from pylib.device import intent # pylint: disable=F0401
class AndroidBrowserBackend(chrome_browser_backend.ChromeBrowserBackend):
"""The backend for controlling a browser instance running on Android."""
def __init__(self, android_platform_backend, browser_options,
- backend_settings, use_rndis_forwarder, output_profile_path,
- extensions_to_load, target_arch):
+ backend_settings, output_profile_path, extensions_to_load,
+ target_arch):
assert isinstance(android_platform_backend,
android_platform_backend_module.AndroidPlatformBackend)
super(AndroidBrowserBackend, self).__init__(
@@ -64,13 +64,9 @@ class AndroidBrowserBackend(chrome_browser_backend.ChromeBrowserBackend):
self._backend_settings.package,
self._backend_settings.profile_ignore_list)
- # Configure platform to use the rndis forwarder.
- # TODO(ariblue): Move this setting out of a browser-specific path, since
- # apps may want to use the forwarder, as well.
- self._platform_backend.SetRndisForwarder(use_rndis_forwarder)
-
- if self.browser_options.netsim or use_rndis_forwarder:
- assert use_rndis_forwarder, 'Netsim requires RNDIS forwarding.'
+ if self.browser_options.netsim or self.platform_backend.use_rndis_forwarder:
+ assert self.platform_backend.use_rndis_forwarder, (
nednguyen 2015/01/23 04:16:07 I don't understand this. if X or Y: assert Y
+ 'Netsim requires RNDIS forwarding.')
self.wpr_port_pairs = forwarders.PortPairs(
http=forwarders.PortPair(0, 80),
https=forwarders.PortPair(0, 443),

Powered by Google App Engine
This is Rietveld 408576698