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

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

Issue 22286010: [telemetry] Implement Forwarder using RNDIS. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: add --android_rndis flag Created 7 years, 4 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/chrome/android_browser_backend.py
diff --git a/tools/telemetry/telemetry/core/chrome/android_browser_backend.py b/tools/telemetry/telemetry/core/chrome/android_browser_backend.py
index 34aa933ee18e62871f1261e8fb2cf83e037c7059..ba57b3bdd0862f8bcb8d5004c5ecc3f7d0e7534a 100644
--- a/tools/telemetry/telemetry/core/chrome/android_browser_backend.py
+++ b/tools/telemetry/telemetry/core/chrome/android_browser_backend.py
@@ -171,6 +171,17 @@ class AndroidBrowserBackend(chrome_browser_backend.ChromeBrowserBackend):
if options.profile_dir:
self._backend_settings.PushProfile(options.profile_dir)
+ self._have_rndis = False
+ if options.android_rndis:
+ try:
+ forwarder = adb_commands.RndisForwarderWithRoot(self._adb)
+ self.WEBPAGEREPLAY_HOST = forwarder.host_ip
+ self._have_rndis = True
+ except Exception, e:
+ logging.warning('Could not setup RNDIS: %s\n' % str(e))
+ # TODO(szym): only override DNS if WPR has privileges to proxy on port 25.
+ self._override_dns = False
+
# Set up the command line.
self._saved_cmdline = ''.join(self._adb.Adb().GetProtectedFileContents(
self._backend_settings.cmdline_file) or [])
@@ -231,6 +242,9 @@ class AndroidBrowserBackend(chrome_browser_backend.ChromeBrowserBackend):
def GetBrowserStartupArgs(self):
args = super(AndroidBrowserBackend, self).GetBrowserStartupArgs()
+ if self._override_dns:
+ args = [arg for arg in args
+ if not arg.startswith('--host-resolver-rules')]
args.append('--enable-remote-debugging')
args.append('--no-restore-state')
args.append('--disable-fre')
@@ -300,5 +314,20 @@ class AndroidBrowserBackend(chrome_browser_backend.ChromeBrowserBackend):
stdout=subprocess.PIPE).communicate()[0])
return ret
+ def AddReplayServerOptions(self, options):
+ """Override. Only add --no-dns_forwarding if not using RNDIS."""
+ if not self._override_dns:
+ options.append('--no-dns_forwarding')
+
def CreateForwarder(self, *port_pairs):
+ if self._have_rndis:
+ forwarder = adb_commands.RndisForwarderWithRoot(self._adb, *port_pairs)
+ assert self.WEBPAGEREPLAY_HOST == forwarder.host_ip, (
+ 'Host IP address on the RNDIS interface changed. Must restart browser!')
+ if self._override_dns:
+ forwarder.OverrideDns()
+ return forwarder
+ assert not self._override_dns, ('The user-space forwarder does not support '
+ 'DNS override!')
+ logging.warning('Using the user-space forwarder.\n')
return adb_commands.Forwarder(self._adb, *port_pairs)

Powered by Google App Engine
This is Rietveld 408576698