Add NetworkChangeNotifier for Android.

net/android/network_change_notifier.h

+// 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.
+
+#ifndef NET_ANDROID_NETWORK_CHANGE_NOTIFIER_ANDROID_H_
+#define NET_ANDROID_NETWORK_CHANGE_NOTIFIER_ANDROID_H_
+#pragma once
+
+#include "base/android/scoped_java_ref.h"
+#include "base/basictypes.h"
+#include "base/compiler_specific.h"
+#include "base/memory/scoped_ptr.h"
+#include "net/base/network_change_notifier.h"
+
+namespace net {
+namespace android {
+
+class NetworkChangeNotifier : public net::NetworkChangeNotifier {
+ public:
+  NetworkChangeNotifier();
+  virtual ~NetworkChangeNotifier();
+
+  void NotifyObservers(JNIEnv* env, jobject obj);
+
+ private:
+  void CreateJavaObject(JNIEnv* env);
+
+  // NetworkChangeNotifier:
+  virtual bool IsCurrentlyOffline() const OVERRIDE;
+
+  base::android::ScopedJavaGlobalRef<jobject>
+      java_network_change_notifier_android_object_;

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

nit: shorten the variable name?

Certainly, "_object" seems superfluous?

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

Done.

+
+  DISALLOW_COPY_AND_ASSIGN(NetworkChangeNotifier);
+};
+
+bool RegisterNetworkChangeNotifier(JNIEnv* env);

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

Any reason to have this free floating?

Can you not just make it a static member on NCN?

net::android::NetworkChangeNotifier::Register(JNIEnv* env) ?

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

This style is consistent with downstream but that doesn't mean it's right... I
tend to agree with you so I'll do that here and change everyone downstream. I
think the issue is that some of these registrations are just static helpers and
not an object but I can keep those as Register*

+
+}  // namespace android
+}  // namespace net
+
+#endif  // NET_ANDROID_NETWORK_CHANGE_NOTIFIER_ANDROID_H_