Remove the unused push_deps option from the test runners.

build/android/pylib/base/base_test_runner.py

Index: build/android/pylib/base/base_test_runner.py
diff --git a/build/android/pylib/base/base_test_runner.py b/build/android/pylib/base/base_test_runner.py
index cfd0c6c75a3c107099be5741edf8ea510427868f..d857f5938058e3a1a023d3f4255d4fca593506de 100644
--- a/build/android/pylib/base/base_test_runner.py
+++ b/build/android/pylib/base/base_test_runner.py
@@ -24,13 +24,11 @@ NET_TEST_SERVER_PORT_INFO_FILE = 'net-test-server-ports'
 class BaseTestRunner(object):
   """Base class for running tests on a single device."""
 
-  def __init__(self, device_serial, tool, push_deps=True,
-               cleanup_test_files=False):
+  def __init__(self, device_serial, tool, cleanup_test_files=False):
     """
       Args:
         device: Tests will run on the device of this ID.
         tool: Name of the Valgrind tool.
-        push_deps: If True, push all dependencies to the device.
         cleanup_test_files: Whether or not to cleanup test files on device.
     """
     self.device_serial = device_serial
@@ -46,7 +44,6 @@ class BaseTestRunner(object):
     # starting it in TestServerThread.
     self.test_server_spawner_port = 0
     self.test_server_port = 0
-    self._push_deps = push_deps
     self._cleanup_test_files = cleanup_test_files
 
   def _PushTestServerPortInfoToDevice(self):
@@ -79,19 +76,7 @@ class BaseTestRunner(object):
   def SetUp(self):
     """Run once before all tests are run."""
     self.InstallTestPackage()
-    push_size_before = self.device.old_interface.GetPushSizeInfo()
-    if self._push_deps:
-      logging.warning('Pushing data files to device.')
-      self.PushDataDeps()
-      push_size_after = self.device.old_interface.GetPushSizeInfo()
-      logging.warning(
-          'Total data: %0.3fMB' %
-          ((push_size_after[0] - push_size_before[0]) / float(2 ** 20)))
-      logging.warning(
-          'Total data transferred: %0.3fMB' %
-          ((push_size_after[1] - push_size_before[1]) / float(2 ** 20)))
-    else:
-      logging.warning('Skipping pushing data to device.')

[mikecase (-- gone --), 2014/11/01 00:53:54]

PushDataDeps is always an empty function except for Instrumentation tests. I
will be moving PushDataDeps to instrumentation/setup.py in a future CL. 

Also, these log statements I'm removing should have probably been inside
PushDataDeps because they appear to be meaningless for anything but
Instrumentation tests. Look at the logs here for example ...
http://build.chromium.org/p/chromium.linux/builders/Android%20Tests%20%28dbg%...

W   11.157s b2bb  Total data: 0.000MB
W   11.157s b2bb  Total data transferred: 0.000MB
W   11.183s 04ae  Pushing data files to device.
W   11.183s 04ae  Total data: 0.000MB
W   11.183s 04ae  Total data transferred: 0.000MB
W   11.416s a2b6  Pushing data files to device.

[jbudorick, 2014/11/03 15:18:47]

In theory, it'd be nice to do this at the same time that we push data deps for
gtests. In practice, this is pretty low priority because of the refactor.
+1

+    self.PushDataDeps()
 
   def TearDown(self):
     """Run once after all tests are run."""