Add a -keep for all Parcelables in hopes it fixes crash in the wild

Add a -keep for all Parcelables in hopes it fixes crash in the wild

BUG=639823
Committed: https://crrev.com/b66c6d1ee2b9c1c4ec687c9ded1006a167fc31c5
Cr-Commit-Position: refs/heads/master@{#414250}

[agrieve, 2016-08-24 18:25:26]

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

[agrieve, 2016-08-24 18:25:34]

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

[commit-bot: I haz the power, 2016-08-24 18:26:29]

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

[nyquist, 2016-08-24 21:16:13]

lgtm

https://codereview.chromium.org/2269143005/diff/1/base/android/base_proguard_...
File base/android/base_proguard_config.flags (right):

https://codereview.chromium.org/2269143005/diff/1/base/android/base_proguard_...
base/android/base_proguard_config.flags:51: #     .aars are used.
Nit: Is there a bug for this?

[commit-bot: I haz the power, 2016-08-24 21:48:59]

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

[commit-bot: I haz the power, 2016-08-24 21:49:00]

Dry run: This issue passed the CQ dry run.

[agrieve, 2016-08-25 01:00:53]

https://codereview.chromium.org/2269143005/diff/1/base/android/base_proguard_...
File base/android/base_proguard_config.flags (right):

https://codereview.chromium.org/2269143005/diff/1/base/android/base_proguard_...
base/android/base_proguard_config.flags:51: #     .aars are used.
On 2016/08/24 21:16:12, nyquist wrote:
> Nit: Is there a bug for this?

Good call. Added.

[agrieve, 2016-08-25 01:01:01]

The CQ bit was checked by agrieve@chromium.org

[agrieve, 2016-08-25 01:01:01]

The patchset sent to the CQ was uploaded after l-g-t-m from nyquist@chromium.org
Link to the patchset: https://codereview.chromium.org/2269143005/#ps20001
(title: "added crbug")

[commit-bot: I haz the power, 2016-08-25 01:02:32]

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-08-25 02:10:59]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-08-25 02:12:53]

Description was changed from

==========
Add a -keep for all Parcelables in hopes it fixes crash in the wild

BUG=639823
==========

to

==========
Add a -keep for all Parcelables in hopes it fixes crash in the wild

BUG=639823

Committed: https://crrev.com/b66c6d1ee2b9c1c4ec687c9ded1006a167fc31c5
Cr-Commit-Position: refs/heads/master@{#414250}
==========

[commit-bot: I haz the power, 2016-08-25 02:12:54]

Patchset 2 (id:??) landed as
https://crrev.com/b66c6d1ee2b9c1c4ec687c9ded1006a167fc31c5
Cr-Commit-Position: refs/heads/master@{#414250}

[Bernhard Bauer, 2016-08-25 11:38:23]

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

[Bernhard Bauer, 2016-08-25 11:38:24]

Sorry, didn't see this yesterday. I'm not completely sure if this is necessary
after Toby's https://codereview.chromium.org/2269553003, but if we are going to
revisit this anyway per the linked bug, we can see if we can remove it then.

[Tobias Sargeant, 2016-08-25 12:11:39]

On 2016/08/25 11:38:24, Bernhard Bauer wrote:
> Sorry, didn't see this yesterday. I'm not completely sure if this is necessary
> after Toby's https://codereview.chromium.org/2269553003, but if we are going
to
> revisit this anyway per the linked bug, we can see if we can remove it then.

I think the -keep is still needed, per the comment about reflected Parcelables.
My
CL only dealt with the obfuscation problem.