Fix Android ninja sdk build

Fix Android ninja sdk build

For some reason, the way we build jars needs us to specify java_in_dir
this way with ninja.

BUG=


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

[cjhopman, 2012-09-12 19:42:01]

PTAL

[Yaron, 2012-09-12 20:47:00]

http://codereview.chromium.org/10908223/diff/1/base/base.gyp
File base/base.gyp (right):

http://codereview.chromium.org/10908223/diff/1/base/base.gyp#newcode966
base/base.gyp:966: 'java_in_dir': 'android/java',
Please make the change here as well

[jar (doing other things), 2012-09-12 21:12:42]

https://codereview.chromium.org/10908223/diff/5001/base/base.gyp
File base/base.gyp (right):

https://codereview.chromium.org/10908223/diff/5001/base/base.gyp#newcode954
base/base.gyp:954:
'android/java/src/org/chromium/base/SystemMessageHandler.java',
I'm not a gyp pro... and have not played with java...

...but why isn't your change done here as well?

[cjhopman, 2012-09-12 22:09:21]

http://codereview.chromium.org/10908223/diff/5001/base/base.gyp
File base/base.gyp (right):

http://codereview.chromium.org/10908223/diff/5001/base/base.gyp#newcode954
base/base.gyp:954:
'android/java/src/org/chromium/base/SystemMessageHandler.java',
On 2012/09/12 21:12:42, jar wrote:
> I'm not a gyp pro... and have not played with java...
> 
> ...but why isn't your change done here as well?

The change is needed below due to a bug in variable expansion in included gypi
files. That doesn't affect us here.

[Yaron, 2012-09-12 22:28:22]

lgtm

[jar (doing other things), 2012-09-12 23:01:18]

lgtm

[commit-bot: I haz the power, 2012-09-12 23:19:24]

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

[commit-bot: I haz the power, 2012-09-13 01:44:05]

Change committed as 156457