[gn] Switch off java targets for v8 standalone

[gn] Switch off java targets for v8 standalone

BUG=chromium:629806
Committed: https://crrev.com/a06f63a2d3661e1d1d27adadf8a054223418a165
Cr-Commit-Position: refs/heads/master@{#38054}

[Michael Achenbach, 2016-07-26 10:19:33]

Description was changed from

==========
[gn] Switch off java targets for v8 standalone

BUG=
==========

to

==========
[gn] Switch off java targets for v8 standalone

BUG=chromium:629806
==========

[Michael Achenbach, 2016-07-26 10:21:02]

machenbach@chromium.org changed reviewers:
+ jbudorick@chromium.org, jochen@chromium.org, vogelheim@chromium.org

[Michael Achenbach, 2016-07-26 10:21:02]

PTAL

This will switch off using
https://cs.chromium.org/chromium/src/v8/BUILD.gn?q=v8_external_startup_data_a...

Guess we don't need that in the standalone case? Using it currently fails due to
a missing deps:
https://build.chromium.org/p/client.v8.ports/builders/V8%20Android%20Arm%20-%...

[Michael Achenbach, 2016-07-26 11:11:45]

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

[commit-bot: I haz the power, 2016-07-26 11:11:49]

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-26 11:46:52]

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

[commit-bot: I haz the power, 2016-07-26 11:46:53]

Dry run: This issue passed the CQ dry run.

[jbudorick, 2016-07-26 14:07:04]

not really familiar with v8 standalone, but the change itself lgtm

[Michael Achenbach, 2016-07-26 14:09:54]

On 2016/07/26 14:07:04, jbudorick wrote:
> not really familiar with v8 standalone, but the change itself lgtm

I understood that the assets are only needed for making APKs, which we don't do
when building v8 standalone.

[Michael Achenbach, 2016-07-26 14:11:00]

The CQ bit was checked by machenbach@chromium.org

[commit-bot: I haz the power, 2016-07-26 14:11:08]

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-26 14:13:24]

Description was changed from

==========
[gn] Switch off java targets for v8 standalone

BUG=chromium:629806
==========

to

==========
[gn] Switch off java targets for v8 standalone

BUG=chromium:629806
==========

[commit-bot: I haz the power, 2016-07-26 14:13:25]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-07-26 14:15:40]

Description was changed from

==========
[gn] Switch off java targets for v8 standalone

BUG=chromium:629806
==========

to

==========
[gn] Switch off java targets for v8 standalone

BUG=chromium:629806

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

[commit-bot: I haz the power, 2016-07-26 14:15:40]

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