Make java.gypi use all_dependent_settings for input_jars_paths

Make java.gypi use all_dependent_settings for input_jars_paths

all_dependent_settings is more like the link_settings that we would like
to use here. This variable is used by java targets to set the classpath
for javac, i.e. to set the libraries that are needed by the target.

Use of direct_dependent_settings with export_dependent_settings causes
unnecessary duplication of all java dependencies for all of our java
library targets.

BUG=146049
TBR=sky,brettw


Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=160768

[Yaron, 2012-09-28 20:15:03]

lgtm

[cjhopman, 2012-10-08 17:44:46]

rsleevi: net/

Also, you originally objected to all_dependent_settings so I figure I should get
your lgtm on switching to it.

[Ryan Sleevi, 2012-10-08 17:50:47]

net/ LGTM.

Assuming I've read correctly, this only affects the Java targets, and is being
used to simulate for Java the (built-in) transitive dependency analysis for
C/C++ targets. In an 'ideal' world, it'd be nice to be able to support such
analysis within GYP and the generators itself, but I suspect that brings
significant complexity into GYP.

My only concern with this approach would be if it is used to justify the use of
all_dependent_settings for C/C++ targets.  How would you feel about adding a
comment to all of the places you're using all_dependent_settings and referencing
the bug and that this is only for Java targets?

[commit-bot: I haz the power, 2012-10-08 23:59:12]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/cjhopman@chromium.org/10986094/11002

[commit-bot: I haz the power, 2012-10-08 23:59:20]

Presubmit check for 10986094-11002 failed and returned exit status 1.


Running presubmit commit checks ...

** Presubmit ERRORS **
Missing LGTM from an OWNER for files in these directories:
    chrome
    content
    base
    ui

Was the presubmit check useful? Please send feedback & hate mail to
maruel@chromium.org!

[cjhopman, 2012-10-09 00:03:36]

TBR: only affects android gyp targets
sky: ui/, chrome/
brettw: base/, content/

[commit-bot: I haz the power, 2012-10-09 00:03:54]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/cjhopman@chromium.org/10986094/11002

[sky, 2012-10-09 00:33:59]

LGTM

[commit-bot: I haz the power, 2012-10-09 02:20:35]

Change committed as 160768