Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(22)

Unified Diff: build/android/pylib/android_commands.py

Issue 14601004: [Android] Only reinstall test apk if needed (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 7 years, 8 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: build/android/pylib/android_commands.py
diff --git a/build/android/pylib/android_commands.py b/build/android/pylib/android_commands.py
index 65e2681d520c0b261e178546ac56b482f7353dbf..dbc52572f17e88ddca56bd856ed9351d462a507f 100644
--- a/build/android/pylib/android_commands.py
+++ b/build/android/pylib/android_commands.py
@@ -672,7 +672,7 @@ class AndroidCommands(object):
"""
self.RunShellCommand('input keyevent %d' % keycode)
- def CheckMd5Sum(self, local_path, device_path):
+ def CheckMd5Sum(self, local_path, device_path, ignore_paths=False):
craigdh 2013/05/06 16:59:56 please add a docstring for this function, includin
cjhopman 2013/05/06 22:32:18 Done.
assert os.path.exists(local_path), 'Local path not found %s' % local_path
if not self._md5sum_build_dir:
@@ -697,6 +697,10 @@ class AndroidCommands(object):
['%s/md5sum_bin_host' % self._md5sum_build_dir, local_path])
hashes_on_host = _ComputeFileListHash(md5sum_output.splitlines())
+ if ignore_paths:
+ hashes_on_device = [h.split()[0] for h in hashes_on_device]
+ hashes_on_host = [h.split()[0] for h in hashes_on_host]
+
return hashes_on_device == hashes_on_host
def PushIfNeeded(self, local_path, device_path):
« no previous file with comments | « no previous file | build/android/pylib/gtest/test_package_apk.py » ('j') | build/android/pylib/gtest/test_package_apk.py » ('J')

Powered by Google App Engine
This is Rietveld 408576698