GYP dependencies for JNI _java targets moved to device targets that require them.

GYP dependencies for JNI _java targets moved to device targets that require them.

Previously _java targets for android JNI were indirectly included in build
dependencies by higher level targets. This change moves the dependencies
directly to the targets that require them.

Committed: https://crrev.com/f1242e526088837d9cac99bce9fbf1e9de47423f
Cr-Commit-Position: refs/heads/master@{#331068}

[scheib, 2015-05-13 22:48:34]

Patchset #1 (id:1) has been deleted

[scheib, 2015-05-13 22:49:02]

scheib@chromium.org changed reviewers:
+ nyquist@chromium.org

[scheib, 2015-05-13 22:50:08]

https://codereview.chromium.org/1127043006/diff/20001/content/content.gyp
File content/content.gyp (right):

https://codereview.chromium.org/1127043006/diff/20001/content/content.gyp#new...
content/content.gyp:431: '../device/battery/battery.gyp:device_battery_java',
Note, content actually has java dependencies on device_battery_java sources, and
thus does need to keep a dependency here.

[nyquist, 2015-05-14 00:33:18]

nyquist@chromium.org changed reviewers:
+ cjhopman@chromium.org

[nyquist, 2015-05-14 00:33:19]

I was under the impression that currently, the _java targets are usually not
depended on by C++ targets, and instead the C++ targets depend on the generated
JNI headers instead.

+cjhopman to keep me honest

[cjhopman, 2015-05-20 02:34:12]

lgtm

[nyquist, 2015-05-20 05:59:19]

given our recent discussion, lgtm

[scheib, 2015-05-20 15:25:02]

The CQ bit was checked by scheib@chromium.org

[commit-bot: I haz the power, 2015-05-20 15:25:30]

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

[commit-bot: I haz the power, 2015-05-20 15:31:24]

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

[commit-bot: I haz the power, 2015-05-20 15:31:26]

Try jobs failed on following builders:
  chromium_presubmit on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromium_presub...)

[scheib, 2015-05-20 15:34:18]

scheib@chromium.org changed reviewers:
+ armansito@chromium.org

[scheib, 2015-05-20 15:34:19]

armansito, PTAL device/*

[armansito, 2015-05-21 02:31:24]

lgtm

[scheib, 2015-05-21 16:16:04]

scheib@chromium.org changed reviewers:
+ jam@chromium.org

[scheib, 2015-05-21 16:16:04]

jam@chromium.org: content/content.gyp

[jam, 2015-05-22 04:18:54]

lgtm

[scheib, 2015-05-22 04:36:12]

The CQ bit was checked by scheib@chromium.org

[commit-bot: I haz the power, 2015-05-22 04:36:29]

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

[commit-bot: I haz the power, 2015-05-22 05:31:16]

Committed patchset #1 (id:20001)

[commit-bot: I haz the power, 2015-05-22 05:32:17]

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