[android] Explicitly specify whether to ignore file paths when pushing files.

[android] Explicitly specify whether to ignore file paths when pushing files.

When pushing data files avoids matching files whose relative paths do not match a file being pushed from the host.

BUG=284906
TEST=None
NOTRY=True

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=221477

[frankf, 2013-09-04 18:42:51]

https://codereview.chromium.org/23513019/diff/1/build/android/pylib/android_c...
File build/android/pylib/android_commands.py (right):

https://codereview.chromium.org/23513019/diff/1/build/android/pylib/android_c...
build/android/pylib/android_commands.py:817: ignore_filenames: If True only the
md5sum is considered when checking if
It's not only filename but the path.

https://codereview.chromium.org/23513019/diff/1/build/android/pylib/android_c...
build/android/pylib/android_commands.py:817: ignore_filenames: If True only the
md5sum is considered when checking if
Also this should hide how it's computing the files changed. md5sum is just one
algorithm you can use.

https://codereview.chromium.org/23513019/diff/1/build/android/pylib/android_c...
build/android/pylib/android_commands.py:827: # Ignore extra files on the device.
Still not deleting the old one?

[craigdh, 2013-09-04 21:25:54]

https://codereview.chromium.org/23513019/diff/1/build/android/pylib/android_c...
File build/android/pylib/android_commands.py (right):

https://codereview.chromium.org/23513019/diff/1/build/android/pylib/android_c...
build/android/pylib/android_commands.py:817: ignore_filenames: If True only the
md5sum is considered when checking if
On 2013/09/04 18:42:51, frankf wrote:
> It's not only filename but the path.

Done.

https://codereview.chromium.org/23513019/diff/1/build/android/pylib/android_c...
build/android/pylib/android_commands.py:817: ignore_filenames: If True only the
md5sum is considered when checking if
On 2013/09/04 18:42:51, frankf wrote:
> Also this should hide how it's computing the files changed. md5sum is just one
> algorithm you can use.

Done.

[frankf, 2013-09-04 21:52:20]

lgtm

https://codereview.chromium.org/23513019/diff/10001/build/android/pylib/andro...
File build/android/pylib/android_commands.py (right):

https://codereview.chromium.org/23513019/diff/10001/build/android/pylib/andro...
build/android/pylib/android_commands.py:809: def GetFilesChanged(self,
host_path, device_path, ignore_filenames=False):
This needs to be cleaned up and move to its own utility module. As it stands,
it's pretty much unreadable.

[commit-bot: I haz the power, 2013-09-05 18:37:10]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/craigdh@chromium.org/23513019/10001

[commit-bot: I haz the power, 2013-09-05 18:38:55]

Change committed as 221477