Android gtest runner: Use device.Install(reinstall=True)

build/android/pylib/local/device/local_device_gtest_run.py

 # Copyright 2014 The Chromium Authors. All rights reserved.
 # Use of this source code is governed by a BSD-style license that can be
 # found in the LICENSE file.
 
 import imp
 import itertools
 import os
 import posixpath
 
 from devil.android import device_errors
@@ skipping 89 matching lines @@
     self._apk_helper = test_instance.apk_helper
     self._package = test_instance.package
     self._runner = test_instance.runner
     self._permissions = test_instance.permissions
     self._suite = test_instance.suite
     self._component = '%s/%s' % (self._package, self._runner)
     self._extras = test_instance.extras
 
   def Install(self, device, incremental=False):
     if not incremental:
-      device.Install(self._apk_helper, permissions=self._permissions)
+      device.Install(self._apk_helper, reinstall=True,

[jbudorick, 2015/10/22 15:14:15]

Hrm, I was worried about how the bots would handle this given that most of our
gtest suites share a package name. Looks like they were ok with it.

+                     permissions=self._permissions)
       return
 
     installer_script = os.path.join(constants.GetOutDirectory(), 'bin',
                                     'install_%s_apk_incremental' % self._suite)
     try:
       install_wrapper = imp.load_source('install_wrapper', installer_script)
     except IOError:
       raise Exception(('Incremental install script not found: %s\n'
                        'Make sure to first build "%s_incremental"') %
                       (installer_script, self._suite))
@@ skipping 233 matching lines @@
   def TearDown(self):
     @local_device_test_run.handle_shard_failures
     def individual_device_tear_down(dev):
       for s in self._servers.get(str(dev), []):
         s.TearDown()
 
       tool = self.GetTool(dev)
       tool.CleanUpEnvironment()
 
     self._env.parallel_devices.pMap(individual_device_tear_down)