[Android] Don't let chromium_commands modify all_dependent_settings via java.gypi

[Android] Don't let chromium_commands modify all_dependent_settings via java.gypi

BUG=423025
Committed: https://crrev.com/e1adb7edbdfa6a76da39a841c8a150bc54cb78fe
Cr-Commit-Position: refs/heads/master@{#300086}

[jbudorick, 2014-10-14 12:20:26]

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

[jbudorick, 2014-10-14 12:20:27]

Not sure that this is the best way to have chromium_commands use java.gypi but
avoid changing all_dependent_settings. wdyt?

[cjhopman, 2014-10-16 18:11:18]

On 2014/10/14 12:20:27, jbudorick wrote:
> Not sure that this is the best way to have chromium_commands use java.gypi but
> avoid changing all_dependent_settings. wdyt?

lgtm. In GN we've basically made java targets only affect the classpath of
direct dependents. That's probably what we should do in gyp, but it would be a
lot of work (and not worth it if we are going to switch to GN). I think what you
have here is a fine approach for now.

[jbudorick, 2014-10-17 09:22:56]

The CQ bit was checked by jbudorick@chromium.org

[commit-bot: I haz the power, 2014-10-17 09:24:00]

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

[commit-bot: I haz the power, 2014-10-17 10:35:51]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2014-10-17 10:36:42]

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