Make java_binary wrapper scripts deps rather than data_deps.

Make java_binary wrapper scripts deps rather than data_deps.

1. They don't require the scripts to run (they aren't actually runtime
deps of the binary itself.
2. In the case of jmake, where the wrapper script is called during
build, the async nature of data_deps means it may not be created in
time.
3. This will make gn gen fail if the output location of jmake changes, since it properly lists it as an input now.

BUG=none

Committed: https://crrev.com/7cca7851502a8cdd54954fce9c6624259c99c76b
Cr-Commit-Position: refs/heads/master@{#379556}

[agrieve, 2016-03-04 14:39:09]

Description was changed from

==========
Make java_binary wrapper scripts deps rather than data_deps.

Two reasons:
1. They don't require the scripts to run (they aren't actually runtime
deps of the binary itself.
2. In the case of jmake, where the wrapper script is called during
build, the async nature of data_deps means it may not be created in
time.

BUG=none
==========

to

==========
Make java_binary wrapper scripts deps rather than data_deps.

Two reasons:
1. They don't require the scripts to run (they aren't actually runtime
deps of the binary itself.
2. In the case of jmake, where the wrapper script is called during
build, the async nature of data_deps means it may not be created in
time.

BUG=none
==========

[agrieve, 2016-03-04 14:39:09]

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

[agrieve, 2016-03-04 14:39:22]

On 2016/03/04 14:39:09, agrieve wrote:
> mailto:agrieve@chromium.org changed reviewers:
> + mailto:mikecase@chromium.org

🔴

[agrieve, 2016-03-04 15:06:02]

Description was changed from

==========
Make java_binary wrapper scripts deps rather than data_deps.

Two reasons:
1. They don't require the scripts to run (they aren't actually runtime
deps of the binary itself.
2. In the case of jmake, where the wrapper script is called during
build, the async nature of data_deps means it may not be created in
time.

BUG=none
==========

to

==========
Make java_binary wrapper scripts deps rather than data_deps.

1. They don't require the scripts to run (they aren't actually runtime
deps of the binary itself.
2. In the case of jmake, where the wrapper script is called during
build, the async nature of data_deps means it may not be created in
time.
3. This will make gn gen fail if the output location of jmake changes, since it
properly lists it as an input now.

BUG=none
==========

[mikecase (-- gone --), 2016-03-05 01:38:23]

lgtm

[agrieve, 2016-03-07 15:29:16]

The CQ bit was checked by agrieve@chromium.org

[commit-bot: I haz the power, 2016-03-07 15:29:24]

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

[commit-bot: I haz the power, 2016-03-07 16:07:28]

Description was changed from

==========
Make java_binary wrapper scripts deps rather than data_deps.

1. They don't require the scripts to run (they aren't actually runtime
deps of the binary itself.
2. In the case of jmake, where the wrapper script is called during
build, the async nature of data_deps means it may not be created in
time.
3. This will make gn gen fail if the output location of jmake changes, since it
properly lists it as an input now.

BUG=none
==========

to

==========
Make java_binary wrapper scripts deps rather than data_deps.

1. They don't require the scripts to run (they aren't actually runtime
deps of the binary itself.
2. In the case of jmake, where the wrapper script is called during
build, the async nature of data_deps means it may not be created in
time.
3. This will make gn gen fail if the output location of jmake changes, since it
properly lists it as an input now.

BUG=none
==========

[commit-bot: I haz the power, 2016-03-07 16:07:30]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-03-07 16:08:38]

Description was changed from

==========
Make java_binary wrapper scripts deps rather than data_deps.

1. They don't require the scripts to run (they aren't actually runtime
deps of the binary itself.
2. In the case of jmake, where the wrapper script is called during
build, the async nature of data_deps means it may not be created in
time.
3. This will make gn gen fail if the output location of jmake changes, since it
properly lists it as an input now.

BUG=none
==========

to

==========
Make java_binary wrapper scripts deps rather than data_deps.

1. They don't require the scripts to run (they aren't actually runtime
deps of the binary itself.
2. In the case of jmake, where the wrapper script is called during
build, the async nature of data_deps means it may not be created in
time.
3. This will make gn gen fail if the output location of jmake changes, since it
properly lists it as an input now.

BUG=none

Committed: https://crrev.com/7cca7851502a8cdd54954fce9c6624259c99c76b
Cr-Commit-Position: refs/heads/master@{#379556}
==========

[commit-bot: I haz the power, 2016-03-07 16:08:41]

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