Add NetworkChangeNotifier for Android.

net/android/java/java.gyp

Index: net/android/java/java.gyp
diff --git a/net/android/java/java.gyp b/net/android/java/java.gyp
new file mode 100644
index 0000000000000000000000000000000000000000..d9b48bda2797d82d5450b86b7770282ae8e261f1
--- /dev/null
+++ b/net/android/java/java.gyp
@@ -0,0 +1,33 @@
+# Copyright (c) 2012 The Chromium Authors. All rights reserved.
+# Use of this source code is governed by a BSD-style license that can be
+# found in the LICENSE file.
+
+{
+  'targets': [
+    {
+      'target_name': 'net_java',
+      'message': 'building net java sources',
+      'type': 'none',
+      'actions': [
+        {
+          'action_name': 'net_java',
+          'inputs': [
+            '<(DEPTH)/net/android/java/net.xml',

[Ryan Sleevi, 2012/04/13 22:37:58]

In Chromium code, we really, really try to minimize the use of '<(DEPTH)'. The
exception being the WebKit code, as there's Chromium-In-WebKit (over in the WK
tree) as much as there is WebKit-in-Chromium (as it is in the Chromium tree).

Can you explain why this is necessary?

[Yaron, 2012/04/17 16:18:03]

It's not. I didn't realize it was frowned upon.

+            '<!@(find <(DEPTH)/net/android/java -name "*.java")'

[Ryan Sleevi, 2012/04/13 22:37:58]

This is really not ideal. The request for this has come up several times on
gyp-developer, and this has consistently been dismissed as not consistent with
the syntax GYP tries to promote (even though yes, it does work).

I'd really like to push back on this before it becomes wide-spread, as I raised
it when base/android/java/java.gyp landed

Note also that base/android/java/java.gyp makes some implicit assumptions about
where subcommands are run that are not guaranteed to hold. I see you've avoided
this by using <(DEPTH) (which is good), but base/android/... will need to be
fixed at some point.

[Yaron, 2012/04/17 16:18:03]

This was discussed in the spin-off bug

+          ],
+          'dependencies': [
+            '<(DEPTH)/base/android/java/java.gyp:base_java',
+          ],
+          'outputs': [
+            '<(DEPTH)/net/android/java/dist/lib/chromium_net.jar',

[John Grabowski, 2012/04/13 22:37:30]

let's put output in a standard location outside the source tree
E.g. see
<property name="out.dir" location="${PRODUCT_DIR}"/>
in
http://chromiumcodereview.appspot.com/10051021/diff/8004/base/android/java/ba...
(and the gyp change that defines it)

[Ryan Sleevi, 2012/04/13 22:37:58]

Dropping files into the source tree = please avoid at all costs. See comments on
net.xml

[Yaron, 2012/04/17 16:18:03]

Yep, I agree but was again following base for consistency and figured they'd all
be fixed together. John's fixing this in http://codereview.chromium.org/10051021
and I figured he'd land first and I'd update accordingly but I'll make the
changes now.

[Yaron, 2012/04/17 16:18:03]

Done.

+          ],
+          'action': [
+            'ant',
+            '-buildfile',

[John Grabowski, 2012/04/13 22:37:30]

see more stuff here
see
http://chromiumcodereview.appspot.com/10051021/diff/8004/base/android/java/ja...

[Yaron, 2012/04/17 16:18:03]

Done.

+            '<(DEPTH)/net/android/java/net.xml',

[Ryan Sleevi, 2012/04/13 22:37:58]

Rather than repeat this (shared with src/base/android/java/java.gyp), consider
using an 'java.gypi' target combined with 'variables' to set the appropriate
outputs as needed.

'java.gypi' can then defer to 'ant'.

An example of how to do this can be seen with 'protoc.gypi'

[Yaron, 2012/04/17 16:18:03]

Done.

+          ]
+        },
+      ],
+    },
+  ],
+}