Moving all for test keeps into for test flags file.

Moving all for test keeps into for test flags file.

This saves 137kb of .dex size and 16kb of dirty memory per process.

BUG=626710
Committed: https://crrev.com/385a49f0676c8cba918439054e21c9bf5a24b914
Cr-Commit-Position: refs/heads/master@{#406544}

[smaier, 2016-07-08 17:35:35]

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

[commit-bot: I haz the power, 2016-07-08 17:35:59]

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

[smaier, 2016-07-08 17:42:45]

smaier@chromium.org changed reviewers:
+ agrieve@chromium.org, wnwen@chromium.org, yfriedman@chromium.org

[Yaron, 2016-07-08 21:29:27]

lgtm but obviously depends on the other cl

https://codereview.chromium.org/2131353002/diff/20001/chrome/android/java/pro...
File chrome/android/java/proguard.flags (left):

https://codereview.chromium.org/2131353002/diff/20001/chrome/android/java/pro...
chrome/android/java/proguard.flags:225: -keep class org.chromium.sync.protocol.*
{ *; }
wonder if this will be still be needed. probably not but trybots will tell you.

[smaier, 2016-07-11 14:54:25]

https://codereview.chromium.org/2131353002/diff/20001/chrome/android/java/pro...
File chrome/android/java/proguard.flags (left):

https://codereview.chromium.org/2131353002/diff/20001/chrome/android/java/pro...
chrome/android/java/proguard.flags:225: -keep class org.chromium.sync.protocol.*
{ *; }
On 2016/07/08 21:29:27, Yaron wrote:
> wonder if this will be still be needed. probably not but trybots will tell
you.

As of ~3 weeks ago, it was needed. Is there anything that has changed with
chrome_sync_shell_test_apk since then?
https://codereview.chromium.org/2079613002/#ps80001

[smaier, 2016-07-19 21:46:03]

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

[commit-bot: I haz the power, 2016-07-19 21:46:44]

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-19 22:50:39]

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

[commit-bot: I haz the power, 2016-07-19 22:50:40]

Dry run: This issue passed the CQ dry run.

[smaier, 2016-07-20 13:18:17]

The CQ bit was checked by smaier@chromium.org

[smaier, 2016-07-20 13:18:18]

The patchset sent to the CQ was uploaded after l-g-t-m from
yfriedman@chromium.org
Link to the patchset: https://codereview.chromium.org/2131353002/#ps40001
(title: "rebased")

[commit-bot: I haz the power, 2016-07-20 13:18:36]

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-20 13:23:01]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-07-20 13:24:31]

Description was changed from

==========
Moving all for test keeps into for test flags file.

This saves 137kb of .dex size and 16kb of dirty memory per process.

BUG=626710
==========

to

==========
Moving all for test keeps into for test flags file.

This saves 137kb of .dex size and 16kb of dirty memory per process.

BUG=626710

Committed: https://crrev.com/385a49f0676c8cba918439054e21c9bf5a24b914
Cr-Commit-Position: refs/heads/master@{#406544}
==========

[commit-bot: I haz the power, 2016-07-20 13:24:32]

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