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

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

Issue 2346773002: Revert of "[telemetry] Wire TsProxy through telemetry's network stack." (Closed) Base URL: https://github.com/catapult-project/catapult.git@master
Patch Set: Created 4 years, 3 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/chrome_browser_backend.py
diff --git a/telemetry/telemetry/internal/backends/chrome/chrome_browser_backend.py b/telemetry/telemetry/internal/backends/chrome/chrome_browser_backend.py
index cafd4b7f41dc6a519139a0a4a9f1cbe60dbbab0b..9c24f4c2cf918d49383ccdcd924de94aa112fa51 100644
--- a/telemetry/telemetry/internal/backends/chrome/chrome_browser_backend.py
+++ b/telemetry/telemetry/internal/backends/chrome/chrome_browser_backend.py
@@ -121,24 +121,16 @@
return args
def GetReplayBrowserStartupArgs(self):
+ replay_args = []
network_backend = self.platform_backend.network_controller_backend
+ proxy_port = network_backend.forwarder.port_pair.remote_port
+ replay_args.append('--proxy-server=socks://localhost:%s' % proxy_port)
if not network_backend.is_replay_active:
return []
- replay_args = []
if not network_backend.is_test_ca_installed:
# Ignore certificate errors if the platform backend has not created
# and installed a root certificate.
replay_args.append('--ignore-certificate-errors')
- # Force hostnames to resolve to the replay's host_ip.
- replay_args.append('--host-resolver-rules=MAP * %s,EXCLUDE localhost' %
- network_backend.host_ip)
- # Force the browser to send HTTP/HTTPS requests to fixed ports if they
- # are not the standard HTTP/HTTPS ports.
- device_ports = network_backend.wpr_device_ports
- if device_ports.http != 80:
- replay_args.append('--testing-fixed-http-port=%s' % device_ports.http)
- if device_ports.https != 443:
- replay_args.append('--testing-fixed-https-port=%s' % device_ports.https)
return replay_args
def HasBrowserFinishedLaunching(self):

Powered by Google App Engine
This is Rietveld 408576698