Handle replay args within replay module.

tools/telemetry/telemetry/core/backends/chrome/android_browser_backend.py

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 f473f41b7c4b84552034ce3119fbb2629c04c0a9..5a41d33cebac6e1ee0c6a9065349a67a9907425e 100644
--- a/tools/telemetry/telemetry/core/backends/chrome/android_browser_backend.py
+++ b/tools/telemetry/telemetry/core/backends/chrome/android_browser_backend.py
@@ -434,9 +434,3 @@ class AndroidBrowserBackend(chrome_browser_backend.ChromeBrowserBackend):
                                         self._adb.device_serial()],
                                        stdout=subprocess.PIPE).communicate()[0])
     return ret
-
-  def AddReplayServerOptions(self, extra_wpr_args):
-    if not self.forwarder_factory.does_forwarder_override_dns:

[nednguyen, 2014/10/06 22:07:50]

What is the equivalent of this logic in your change?

[slamm, 2014/10/13 21:45:54]

In webpagereplay, --no-dns_forwarding is added when the DNS port is None.

I created a spreadsheet to show make sure that matches up.

https://docs.google.com/a/chromium.org/spreadsheets/d/14fNT8Sd-bO3ZbD_2qwVqlf...

A possible improvement after this CL would be setting the ports in one place
based on a few parameters. As it is now, the ports are set in various browser
backend initializers.

-      extra_wpr_args.append('--no-dns_forwarding')
-    if self.browser_options.netsim:
-      extra_wpr_args.append('--net=%s' % self.browser_options.netsim)