[Android] Sanitize APK signing file timestamps.

[Android] Sanitize APK signing file timestamps.

BUG=630931
Committed: https://crrev.com/61e1a7ac087039d43e694c08584724252f9785ce
Cr-Commit-Position: refs/heads/master@{#408389}

[jbudorick, 2016-07-23 04:02:27]

The CQ bit was checked by jbudorick@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-07-23 04:02:32]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-07-23 04:55:09]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-07-23 04:55:10]

Dry run: This issue passed the CQ dry run.

[ghost stip (do not use), 2016-07-25 05:46:05]

stip@chromium.org changed reviewers:
+ stip@chromium.org

[ghost stip (do not use), 2016-07-25 05:46:06]

bug: https://bugs.chromium.org/p/chromium/issues/detail?id=630931

[jbudorick, 2016-07-25 13:12:23]

I looked at my runs of chrome_public_test_apk and android_webview_test_apk. None
of them seem to have been deduped. In two arbitrary runs of
chrome_public_test_apk, ChromePublic.apk and snapshot_blob.bin were still
different.

[jbudorick, 2016-07-25 13:14:19]

On 2016/07/25 13:12:23, jbudorick wrote:
> I looked at my runs of chrome_public_test_apk and android_webview_test_apk.
None
> of them seem to have been deduped. In two arbitrary runs of
> chrome_public_test_apk, ChromePublic.apk and snapshot_blob.bin were still
> different.

ah, I guess this is why you filed
https://bugs.chromium.org/p/chromium/issues/detail?id=630935

[ghost stip (do not use), 2016-07-26 23:55:51]

now that the v8 bug has been fixed, this should work for all binaries.

from
https://build.chromium.org/p/tryserver.chromium.android/builders/linux_androi...
and
https://build.chromium.org/p/tryserver.chromium.android/builders/linux_androi...,
both android_webview_test_apk and chrome_public_test_apk have the same binary

[jbudorick, 2016-07-27 23:15:21]

The CQ bit was checked by jbudorick@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-07-27 23:15:51]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-07-28 00:16:54]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-07-28 00:16:55]

Dry run: This issue passed the CQ dry run.

[jbudorick, 2016-07-28 13:24:51]

Description was changed from

==========
CLEAN BEFORE COMMITTING: Make APKs great again!

BUG=
==========

to

==========
[Android] Sanitize APK signing file timestamps.

BUG=
==========

[jbudorick, 2016-07-28 13:35:29]

Description was changed from

==========
[Android] Sanitize APK signing file timestamps.

BUG=
==========

to

==========
[Android] Sanitize APK signing file timestamps.

BUG=630931
==========

[jbudorick, 2016-07-28 13:35:47]

jbudorick@chromium.org changed reviewers:
+ agrieve@chromium.org

[jbudorick, 2016-07-28 13:35:47]

agrieve: ptal

[agrieve, 2016-07-28 13:40:58]

lgtm

[jbudorick, 2016-07-28 13:45:39]

The CQ bit was checked by jbudorick@chromium.org

[commit-bot: I haz the power, 2016-07-28 13:45:56]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-07-28 14:34:33]

Description was changed from

==========
[Android] Sanitize APK signing file timestamps.

BUG=630931
==========

to

==========
[Android] Sanitize APK signing file timestamps.

BUG=630931
==========

[commit-bot: I haz the power, 2016-07-28 14:34:34]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-07-28 14:36:31]

Description was changed from

==========
[Android] Sanitize APK signing file timestamps.

BUG=630931
==========

to

==========
[Android] Sanitize APK signing file timestamps.

BUG=630931

Committed: https://crrev.com/61e1a7ac087039d43e694c08584724252f9785ce
Cr-Commit-Position: refs/heads/master@{#408389}
==========

[commit-bot: I haz the power, 2016-07-28 14:36:32]

Patchset 3 (id:??) landed as
https://crrev.com/61e1a7ac087039d43e694c08584724252f9785ce
Cr-Commit-Position: refs/heads/master@{#408389}

[agrieve, 2016-07-28 17:33:15]

A revert of this CL (patchset #3 id:40001) has been created in
https://codereview.chromium.org/2191673004/ by agrieve@chromium.org.

The reason for reverting is: Breaking builds:
https://bugs.chromium.org/p/chromium/issues/detail?id=632400.