🔟 Remove Parcelable -keep from base proguard rules

Remove Parcelable -keep from base proguard rules

And tidy up proguard_config -> proguard_preprocess_config rename

Verified locally that ChromePublic.apk had no size change from
this patch.

BUG=640836
Committed: https://crrev.com/c41ec1ae1555bbb04c06f80a5fcbd86978e8735d
Cr-Commit-Position: refs/heads/master@{#416428}

[agrieve, 2016-09-02 18:01:41]

Description was changed from

==========
Remove Parcelable -keep from base proguard rules

And tidy up proguard_config -> proguard_preprocess_config rename

BUG=640836
==========

to

==========
Remove Parcelable -keep from base proguard rules

And tidy up proguard_config -> proguard_preprocess_config rename

Verified locally that ChromePublic.apk had no size change from
this patch.

BUG=640836
==========

[agrieve, 2016-09-02 18:01:41]

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

[agrieve, 2016-09-02 18:01:50]

On 2016/09/02 18:01:41, agrieve wrote:
> mailto:agrieve@chromium.org changed reviewers:
> + mailto:ianwen@chromium.org

[Ian Wen, 2016-09-02 20:25:54]

lgtm

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

https://codereview.chromium.org/2310473002/diff/1/base/android/base_proguard_...
base/android/base_proguard_config.flags:46: # If we annotated all Parcelables
that get put into Bundles other than
The content of the new comment sounds like a TODO. If we plan to improve this,
should we mark it as a TODO?

[agrieve, 2016-09-02 20:34:58]

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

https://codereview.chromium.org/2310473002/diff/1/base/android/base_proguard_...
base/android/base_proguard_config.flags:46: # If we annotated all Parcelables
that get put into Bundles other than
On 2016/09/02 20:25:54, Ian Wen wrote:
> The content of the new comment sounds like a TODO. If we plan to improve this,
> should we mark it as a TODO?

It's more of a NEVERDO I think :P

[agrieve, 2016-09-02 20:35:02]

The CQ bit was checked by agrieve@chromium.org

[commit-bot: I haz the power, 2016-09-02 20:35:44]

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-09-02 22:48:47]

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

[commit-bot: I haz the power, 2016-09-02 22:48:47]

Try jobs failed on following builders:
  win_chromium_rel_ng on master.tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_rel_...)

[agrieve, 2016-09-03 00:51:54]

The CQ bit was checked by agrieve@chromium.org

[commit-bot: I haz the power, 2016-09-03 00:52:25]

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-09-03 02:39:04]

Description was changed from

==========
Remove Parcelable -keep from base proguard rules

And tidy up proguard_config -> proguard_preprocess_config rename

Verified locally that ChromePublic.apk had no size change from
this patch.

BUG=640836
==========

to

==========
Remove Parcelable -keep from base proguard rules

And tidy up proguard_config -> proguard_preprocess_config rename

Verified locally that ChromePublic.apk had no size change from
this patch.

BUG=640836
==========

[commit-bot: I haz the power, 2016-09-03 02:39:05]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-09-03 02:41:21]

Description was changed from

==========
Remove Parcelable -keep from base proguard rules

And tidy up proguard_config -> proguard_preprocess_config rename

Verified locally that ChromePublic.apk had no size change from
this patch.

BUG=640836
==========

to

==========
Remove Parcelable -keep from base proguard rules

And tidy up proguard_config -> proguard_preprocess_config rename

Verified locally that ChromePublic.apk had no size change from
this patch.

BUG=640836

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

[commit-bot: I haz the power, 2016-09-03 02:41:22]

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