Provide NetworkChangeNotifierAndroid with the actual initial connection type.

net/android/network_change_notifier_delegate_android.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_DELEGATE_ANDROID_H_
 #define NET_ANDROID_NETWORK_CHANGE_NOTIFIER_DELEGATE_ANDROID_H_
 
 #include "base/android/jni_android.h"
 #include "base/basictypes.h"
 #include "base/memory/ref_counted.h"
 #include "base/observer_list_threadsafe.h"
+#include "base/synchronization/lock.h"
 #include "base/threading/thread_checker.h"
 #include "net/base/network_change_notifier.h"
 
 namespace net {
 
 // Delegate used to thread-safely notify NetworkChangeNotifierAndroid whenever a
 // network connection change notification is signaled by the Java side (on the
 // JNI thread).
 // All the methods exposed below must be called exclusively on the JNI thread
 // unless otherwise stated (e.g. AddObserver()/RemoveObserver()).
 class NET_EXPORT_PRIVATE NetworkChangeNotifierDelegateAndroid {
  public:
+  typedef NetworkChangeNotifier::ConnectionType ConnectionType;
+
   enum ConnectivityState {
     OFFLINE,
     ONLINE,
   };
 
   // Observer interface implemented by NetworkChangeNotifierAndroid which
   // subscribes to network change notifications fired by the delegate (and
   // initiated by the Java side).
   class Observer {
    public:
     virtual ~Observer() {}
 
     // Updates the current connection type.
     virtual void OnConnectionTypeChanged(
         NetworkChangeNotifier::ConnectionType new_connection_type) = 0;
   };
 
   NetworkChangeNotifierDelegateAndroid();
   ~NetworkChangeNotifierDelegateAndroid();
 
   // Called from NetworkChangeNotifierAndroid.java on the JNI thread whenever
   // the connection type changes. This updates the current connection type seen
   // by this class and forwards the notification to the observers that
   // subscribed through AddObserver().
   void NotifyConnectionTypeChanged(JNIEnv* env,
                                    jobject obj,
                                    jint new_connection_type);
   jint GetConnectionType(JNIEnv* env, jobject obj) const;
 
+  // Same as above but meant to be called by NetworkChangeNotifierAndroid from
+  // any thread.

[Ryan Sleevi, 2012/12/19 18:07:57]

This comment is confusing and would need to be re-worded.

+  ConnectionType GetCurrentConnectionType() const;
+
   // These methods can be called on any thread. Note that the provided observer
   // will be notified on the thread AddObserver() is called on.
   void AddObserver(Observer* observer);
   void RemoveObserver(Observer* observer);
 
   // Exposed for testing.
   void ForceConnectivityState(ConnectivityState state);
 
   // Initializes JNI bindings.
   static bool Register(JNIEnv* env);
 
  private:
   friend class NetworkChangeNotifierDelegateAndroidTest;
 
-  typedef NetworkChangeNotifier::ConnectionType ConnectionType;
+  void SetConnectionType(ConnectionType new_connection_type);
 
   base::ThreadChecker thread_checker_;
   scoped_refptr<ObserverListThreadSafe<Observer> > observers_;
   scoped_refptr<base::SingleThreadTaskRunner> jni_task_runner_;
   base::android::ScopedJavaGlobalRef<jobject> java_network_change_notifier_;
+  mutable base::Lock connection_type_lock_;  // Protects the state below.
   ConnectionType connection_type_;
 
   DISALLOW_COPY_AND_ASSIGN(NetworkChangeNotifierDelegateAndroid);
 };
 
 }  // namespace net
 
 #endif  // NET_ANDROID_NETWORK_CHANGE_NOTIFIER_DELEGATE_H_