Move Python setup/tear down logic into Forwarder itself.

build/android/pylib/instrumentation/test_runner.py

Index: build/android/pylib/instrumentation/test_runner.py
diff --git a/build/android/pylib/instrumentation/test_runner.py b/build/android/pylib/instrumentation/test_runner.py
index 60000bf71be0e2d871b8430e6cee4bd9ee5c5d76..8981ebf248d92feef349de4dd8cd18daa2e3fe17 100644
--- a/build/android/pylib/instrumentation/test_runner.py
+++ b/build/android/pylib/instrumentation/test_runner.py
@@ -149,11 +149,13 @@ class TestRunner(base_test_runner.BaseTestRunner):
     http_server_ports = self.LaunchTestHttpServer(
         os.path.join(constants.DIR_SOURCE_ROOT), self._lighttp_port)
     if self.ports_to_forward:
-      self.StartForwarder([(port, port) for port in self.ports_to_forward])
+      self.ForwardPorts([(port, port) for port in self.ports_to_forward])
     self.flags.AddFlags(['--enable-test-intents'])
 
   def TearDown(self):
     """Cleans up the test harness and saves outstanding data from test run."""
+    if self.ports_to_forward:
+      self.UnmapPorts(self.ports_to_forward)
     super(TestRunner, self).TearDown()
 
   def TestSetup(self, test):
@@ -166,9 +168,6 @@ class TestRunner(base_test_runner.BaseTestRunner):
     self._SetupIndividualTestTimeoutScale(test)
     self.tool.SetupEnvironment()
 
-    # Make sure the forwarder is still running.
-    self.RestartHttpServerForwarderIfNecessary()
-
   def _IsPerfTest(self, test):
     """Determines whether a test is a performance test.