Revert "Reland r212020: Move Python setup/tear down logic into Forwarder ..."

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 be46a1a2ed066a457876480fd3b33b7579b4dd75..dcf3d20613d6e5b867688d5c83d0c492da6f8554 100644
--- a/build/android/pylib/instrumentation/test_runner.py
+++ b/build/android/pylib/instrumentation/test_runner.py
@@ -154,13 +154,11 @@ 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.ForwardPorts([(port, port) for port in self.ports_to_forward])
+      self.StartForwarder([(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([(port, port) for port in self.ports_to_forward])
     super(TestRunner, self).TearDown()
 
   def TestSetup(self, test):
@@ -174,7 +172,7 @@ class TestRunner(base_test_runner.BaseTestRunner):
     self.tool.SetupEnvironment()
 
     # Make sure the forwarder is still running.
-    self._RestartHttpServerForwarderIfNecessary()
+    self.RestartHttpServerForwarderIfNecessary()
 
   def _IsPerfTest(self, test):
     """Determines whether a test is a performance test.