Add support for Android aidl and support for gyp's java_in_dir

Add support for Android aidl and support for gyp's java_in_dir

Adds the android_aidl template to support Android aidl.

Adds a DEPRECATED_java_in_dir to android_library and android_apk. This
will greatly ease the transition from gyp to gn.

Adds BUILD.gn for third_party/eyesfree (using the two new features
added).

TBR=rsleevi
BUG=359249
Committed: https://crrev.com/dad5f427256371787b536ccdcf08d6493bebb379
Cr-Commit-Position: refs/heads/master@{#293386}

[cjhopman, 2014-08-27 18:36:38]

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

[cjhopman, 2014-08-27 18:36:38]

newt: *

[cjhopman, 2014-08-27 18:37:59]

https://codereview.chromium.org/512923002/diff/1/build/android/gyp/write_buil...
File build/android/gyp/write_build_config.py (right):

https://codereview.chromium.org/512923002/diff/1/build/android/gyp/write_buil...
build/android/gyp/write_build_config.py:114: raise Exception('Unknown deps: ' +
str(unknown_deps))
Looks like some random fixes have popped up in here. :/

This fixes can't add str+list

https://codereview.chromium.org/512923002/diff/1/build/config/android/rules.gni
File build/config/android/rules.gni (right):

https://codereview.chromium.org/512923002/diff/1/build/config/android/rules.g...
build/config/android/rules.gni:222: depfile =
"${target_gen_dir}/${target_name}_{{source_name_part}}.d"
This path has to be unique for each source file.

[newt (away), 2014-08-27 20:05:29]

https://codereview.chromium.org/512923002/diff/1/build/config/android/rules.gni
File build/config/android/rules.gni (right):

https://codereview.chromium.org/512923002/diff/1/build/config/android/rules.g...
build/config/android/rules.gni:749: if (defined(invoker.datadeps)) {
where are these datadeps coming from?

https://codereview.chromium.org/512923002/diff/1/build/config/android/rules.g...
build/config/android/rules.gni:809: # Variables
what about interface_file?

https://codereview.chromium.org/512923002/diff/1/build/config/android/rules.g...
build/config/android/rules.gni:811: #   import_include: Paths to directories
containing .java files imported by the
s/Paths/Path ... right?

https://codereview.chromium.org/512923002/diff/1/build/config/android/rules.g...
build/config/android/rules.gni:862: "//build/android/gyp/find.py",
is the plan to get rid of this "find" after the gn switch?

https://codereview.chromium.org/512923002/diff/1/net/net.gyp
File net/net.gyp (left):

https://codereview.chromium.org/512923002/diff/1/net/net.gyp#oldcode1669
net/net.gyp:1669: ['OS == "android"', {
why was this here to begin with?

[cjhopman, 2014-08-27 21:05:22]

https://codereview.chromium.org/512923002/diff/1/build/config/android/rules.gni
File build/config/android/rules.gni (right):

https://codereview.chromium.org/512923002/diff/1/build/config/android/rules.g...
build/config/android/rules.gni:749: if (defined(invoker.datadeps)) {
On 2014/08/27 20:05:29, newt wrote:
> where are these datadeps coming from?

The invoker. I added documentation above (this is a standard gn target variable,
but needs to be implemented separately for each template definition that wants
to support it).

https://codereview.chromium.org/512923002/diff/1/build/config/android/rules.g...
build/config/android/rules.gni:809: # Variables
On 2014/08/27 20:05:28, newt wrote:
> what about interface_file?

Done.

https://codereview.chromium.org/512923002/diff/1/build/config/android/rules.g...
build/config/android/rules.gni:811: #   import_include: Paths to directories
containing .java files imported by the
On 2014/08/27 20:05:29, newt wrote:
> s/Paths/Path ... right?

Done.

https://codereview.chromium.org/512923002/diff/1/build/config/android/rules.g...
build/config/android/rules.gni:862: "//build/android/gyp/find.py",
On 2014/08/27 20:05:29, newt wrote:
> is the plan to get rid of this "find" after the gn switch?
It's complicated.

The TODO a couple lines up describes one way that we could get rid of this find.
The other option would be to make the caller explicitly specify which java files
are needed, in that case I would want to make sure that the build doesn't
actually use any unspecified java files (I have a couple thoughts on how to do
that).

The find can cause some subtle build issues (though no worse than what we have
in gyp).

https://codereview.chromium.org/512923002/diff/1/net/net.gyp
File net/net.gyp (left):

https://codereview.chromium.org/512923002/diff/1/net/net.gyp#oldcode1669
net/net.gyp:1669: ['OS == "android"', {
On 2014/08/27 20:05:29, newt wrote:
> why was this here to begin with?

idk, the condition block above it has the exact same condition. I expect it was
here just because gyp is hard to read.

[newt (away), 2014-08-27 21:09:52]

lgtm

[cjhopman, 2014-08-27 21:14:26]

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

[cjhopman, 2014-08-27 21:14:26]

And brettw@ for the OWNERS stamp of approval

[brettw, 2014-09-04 21:52:35]

lgtm

[cjhopman, 2014-09-04 21:54:10]

The CQ bit was checked by cjhopman@chromium.org

[commit-bot: I haz the power, 2014-09-04 21:54:56]

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

[cjhopman, 2014-09-04 23:13:29]

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

[cjhopman, 2014-09-04 23:13:35]

The CQ bit was unchecked by cjhopman@chromium.org

[cjhopman, 2014-09-04 23:13:36]

The CQ bit was checked by cjhopman@chromium.org

[commit-bot: I haz the power, 2014-09-04 23:14:02]

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

[commit-bot: I haz the power, 2014-09-05 01:03:23]

Committed patchset #4 (id:60001) as 40aaa0520c86daddc7eeee98fc5e6afd6e0ffeea

[commit-bot: I haz the power, 2014-09-10 03:34:28]

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