[gmscore] Enable multidexing for various test targets

[gmscore] Enable multidexing for various test targets

Java tests that involve building dex files can reach the dex
method limit when their dependencies pull at lot of content.
This CL enables multidexing for a few targets that fail
building when we stop pre-proguarding the play services library

BUG=668211
Committed: https://crrev.com/a2515b11698058cef320f53ec4df844a2b558d06
Cr-Commit-Position: refs/heads/master@{#437593}

[dgn, 2016-12-08 14:30:20]

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

[commit-bot: I haz the power, 2016-12-08 14:30:35]

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

[dgn, 2016-12-08 14:31:46]

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

[dgn, 2016-12-08 14:31:47]

PTAL

I removed the current downstream intermediate proguarding and ran `ninja -C
out_android_gn/Debug all`, then enabled multidex for the targets that failed
until everything could build.

Another way to do this could be to enable multidexing directly in the `test`
target on android, or for `test_only` apks.

[dgn, 2016-12-08 15:41:48]

dgn@chromium.org changed reviewers:
+ nyquist@chromium.org, torne@chromium.org

[dgn, 2016-12-08 15:41:48]

nyquist@chromium.org: Please review changes in //blimp

torne@chromium.org: Please review changes in //android_webview

[Torne, 2016-12-08 15:45:28]

android_webview lgtm

[commit-bot: I haz the power, 2016-12-08 16:00:52]

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

[commit-bot: I haz the power, 2016-12-08 16:00:54]

Dry run: This issue passed the CQ dry run.

[nyquist, 2016-12-08 16:30:54]

blimp lgtm

[dgn, 2016-12-08 17:38:35]

dgn@chromium.org changed reviewers:
+ sdefresne@chromium.org

[dgn, 2016-12-08 17:38:36]

sdefresne@chromium.org: Please review changes in components/BUILD.gn

[dgn, 2016-12-08 17:39:23]

Thanks, I will land it tomorrow morning when I have time to watch the bots.

[agrieve, 2016-12-09 15:44:05]

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

[agrieve, 2016-12-09 15:44:05]

lgtm. I think it's fine for now, and we'll see if other targets start hitting
the limit.

[blundell, 2016-12-09 16:52:57]

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

[blundell, 2016-12-09 16:52:58]

rubberstamp lgtm for //components

[dgn, 2016-12-09 17:11:04]

The CQ bit was checked by dgn@chromium.org

[commit-bot: I haz the power, 2016-12-09 17:11:31]

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-12-09 18:40:52]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1481303464162260, "parent_rev":
"3e30a8cea2cb65dbb0b94b1e036743a49b08ea17", "commit_rev":
"25c7933ac7e4e1b143fc4791648d5094e2b79418"}

[commit-bot: I haz the power, 2016-12-09 18:41:18]

Description was changed from

==========
[gmscore] Enable multidexing for various test targets

Java tests that involve building dex files can reach the dex
method limit when their dependencies pull at lot of content.
This CL enables multidexing for a few targets that fail
building when we stop pre-proguarding the play services library

BUG=668211
==========

to

==========
[gmscore] Enable multidexing for various test targets

Java tests that involve building dex files can reach the dex
method limit when their dependencies pull at lot of content.
This CL enables multidexing for a few targets that fail
building when we stop pre-proguarding the play services library

BUG=668211

Review-Url: https://codereview.chromium.org/2563663002
==========

[commit-bot: I haz the power, 2016-12-09 18:41:19]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-12-12 14:36:24]

Description was changed from

==========
[gmscore] Enable multidexing for various test targets

Java tests that involve building dex files can reach the dex
method limit when their dependencies pull at lot of content.
This CL enables multidexing for a few targets that fail
building when we stop pre-proguarding the play services library

BUG=668211

Review-Url: https://codereview.chromium.org/2563663002
==========

to

==========
[gmscore] Enable multidexing for various test targets

Java tests that involve building dex files can reach the dex
method limit when their dependencies pull at lot of content.
This CL enables multidexing for a few targets that fail
building when we stop pre-proguarding the play services library

BUG=668211

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

[commit-bot: I haz the power, 2016-12-12 14:36:26]

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