Allow test and unittest_apk GN targets to use proguard

Allow test and unittest_apk GN targets to use proguard

This CL adds proguard arguments to test and unittest_apk GN targets.
This does not change the default behavior for these two targets.

BUG=359249
Committed: https://crrev.com/c35ba6ea4eee6e6b0bb87c4048e357edf3d9b4a4
Cr-Commit-Position: refs/heads/master@{#396031}

[huapengl, 2016-05-23 23:06:39]

huapengl@amazon.com changed reviewers:
+ dpranke@chromium.org

[huapengl, 2016-05-23 23:06:40]

Hi Dirk, 
Can you take a look on this change?

Thanks

[Dirk Pranke, 2016-05-24 22:07:15]

https://codereview.chromium.org/2000333003/diff/1/build/config/android/rules.gni
File build/config/android/rules.gni (right):

https://codereview.chromium.org/2000333003/diff/1/build/config/android/rules....
build/config/android/rules.gni:2140: proguard_configs +=
invoker.proguard_configs
shouldn't this be "proguard_configs = invoker.proguard_configs"?

[huapengl, 2016-05-25 20:47:51]

On 2016/05/24 22:07:15, Dirk Pranke (slow) wrote:
>
https://codereview.chromium.org/2000333003/diff/1/build/config/android/rules.gni
> File build/config/android/rules.gni (right):
> 
>
https://codereview.chromium.org/2000333003/diff/1/build/config/android/rules....
> build/config/android/rules.gni:2140: proguard_configs +=
> invoker.proguard_configs
> shouldn't this be "proguard_configs = invoker.proguard_configs"?

Sure, good idea.
I will upload a new revision.

[huapengl, 2016-05-25 20:49:05]

removed the "+"

[Dirk Pranke, 2016-05-25 21:20:52]

lgtm

[huapengl, 2016-05-25 21:22:00]

The CQ bit was checked by huapengl@amazon.com

[commit-bot: I haz the power, 2016-05-25 21:23:03]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/2000333003/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/2000333003/20001

[commit-bot: I haz the power, 2016-05-25 23:08:18]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-05-25 23:10:38]

Description was changed from

==========
Allow test and unittest_apk GN targets to use proguard

This CL adds proguard arguments to test and unittest_apk GN targets.
This does not change the default behavior for these two targets.

BUG=359249
==========

to

==========
Allow test and unittest_apk GN targets to use proguard

This CL adds proguard arguments to test and unittest_apk GN targets.
This does not change the default behavior for these two targets.

BUG=359249

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

[commit-bot: I haz the power, 2016-05-25 23:10:39]

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