API for connection type (Ethernet/WIFI/WWAN ...) in NetworkChangeNotifier.

net/base/network_change_notifier_linux_unittest.cc

 // 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.
 
 #include "net/base/network_change_notifier_linux.h"
 
 #include "base/bind.h"
 #include "base/message_loop_proxy.h"
 #include "base/synchronization/waitable_event.h"
 #include "dbus/mock_bus.h"
@@ skipping 101 matching lines @@
 
   // Valid only after initialized_ is signaled.
   scoped_refptr<base::MessageLoopProxy> notifier_thread_proxy_;
 
   scoped_refptr<dbus::MockBus> mock_bus_;
   scoped_refptr<dbus::MockObjectProxy> mock_object_proxy_;
 };
 
 namespace {
 
-class OfflineObserver : public NetworkChangeNotifier::OnlineStateObserver {
+class OfflineObserver : public NetworkChangeNotifier::ConnectionTypeObserver {
  public:
   OfflineObserver()
       : notification_count(0),
         last_online_value(true) {
-    NetworkChangeNotifier::AddOnlineStateObserver(this);
+    NetworkChangeNotifier::AddConnectionTypeObserver(this);
   }
 
   ~OfflineObserver() {
-    NetworkChangeNotifier::RemoveOnlineStateObserver(this);
+    NetworkChangeNotifier::RemoveConnectionTypeObserver(this);
   }
 
-  virtual void OnOnlineStateChanged(bool online) OVERRIDE {
+  virtual void OnConnectionTypeChanged(
+      NetworkChangeNotifier::ConnectionType type) OVERRIDE {
     notification_count++;
-    last_online_value = online;
+    last_online_value = type != NetworkChangeNotifier::CONNECTION_NONE;
   }
 
   int notification_count;
   bool last_online_value;
 };
 
 TEST_F(NetworkChangeNotifierLinuxTest, Offline) {
   SendResponse(NM_STATE_DISCONNECTED);
-  EXPECT_TRUE(NetworkChangeNotifier::IsOffline());
+  EXPECT_TRUE(NetworkChangeNotifier::GetConnectionType() ==
+                  NetworkChangeNotifier::CONNECTION_NONE);

[wtc, 2012/05/11 01:35:05]

Nit: these should ideally be replaced by EXPECT_EQ and
EXPECT_NE.  For example, this one can be replaced by

  EXPECT_EQ(NetworkChangeNotifier::CONNECTION_NONE,
            NetworkChangeNotifier::GetConnectionType());

 }
 
 TEST_F(NetworkChangeNotifierLinuxTest, Online) {
   SendResponse(NM_STATE_CONNECTED_GLOBAL);
-  EXPECT_FALSE(NetworkChangeNotifier::IsOffline());
+  EXPECT_FALSE(NetworkChangeNotifier::GetConnectionType() ==
+                   NetworkChangeNotifier::CONNECTION_NONE);
 }
 
 TEST_F(NetworkChangeNotifierLinuxTest, OfflineThenOnline) {
   OfflineObserver observer;
 
   SendResponse(NM_STATE_DISCONNECTED);
-  EXPECT_TRUE(NetworkChangeNotifier::IsOffline());
+  EXPECT_TRUE(NetworkChangeNotifier::GetConnectionType() ==
+                  NetworkChangeNotifier::CONNECTION_NONE);
   EXPECT_EQ(0, observer.notification_count);
 
   SendSignal(NM_STATE_CONNECTED_GLOBAL);
-  EXPECT_FALSE(NetworkChangeNotifier::IsOffline());
+  EXPECT_FALSE(NetworkChangeNotifier::GetConnectionType() ==
+                   NetworkChangeNotifier::CONNECTION_NONE);
   EXPECT_EQ(1, observer.notification_count);
   EXPECT_TRUE(observer.last_online_value);
 }
 
 TEST_F(NetworkChangeNotifierLinuxTest, MultipleStateChanges) {
   OfflineObserver observer;
 
   SendResponse(NM_STATE_CONNECTED_GLOBAL);
-  EXPECT_FALSE(NetworkChangeNotifier::IsOffline());
+  EXPECT_FALSE(NetworkChangeNotifier::GetConnectionType() ==
+                   NetworkChangeNotifier::CONNECTION_NONE);
   EXPECT_EQ(0, observer.notification_count);
 
   SendSignal(NM_STATE_DISCONNECTED);
-  EXPECT_TRUE(NetworkChangeNotifier::IsOffline());
+  EXPECT_TRUE(NetworkChangeNotifier::GetConnectionType() ==
+                  NetworkChangeNotifier::CONNECTION_NONE);
   EXPECT_EQ(1, observer.notification_count);
   EXPECT_FALSE(observer.last_online_value);
 
   SendSignal(NM_STATE_CONNECTED_GLOBAL);
-  EXPECT_FALSE(NetworkChangeNotifier::IsOffline());
+  EXPECT_FALSE(NetworkChangeNotifier::GetConnectionType() ==
+                   NetworkChangeNotifier::CONNECTION_NONE);
   EXPECT_EQ(2, observer.notification_count);
   EXPECT_TRUE(observer.last_online_value);
 }
 
 TEST_F(NetworkChangeNotifierLinuxTest, IgnoreContinuedOnlineState) {
   OfflineObserver observer;
 
   SendResponse(NM_STATE_CONNECTED_SITE);
-  EXPECT_FALSE(NetworkChangeNotifier::IsOffline());
+  EXPECT_FALSE(NetworkChangeNotifier::GetConnectionType() ==
+                   NetworkChangeNotifier::CONNECTION_NONE);
   EXPECT_EQ(0, observer.notification_count);
 
   SendSignal(NM_STATE_CONNECTED_GLOBAL);
-  EXPECT_FALSE(NetworkChangeNotifier::IsOffline());
+  EXPECT_FALSE(NetworkChangeNotifier::GetConnectionType() ==
+                   NetworkChangeNotifier::CONNECTION_NONE);
   EXPECT_EQ(0, observer.notification_count);
 }
 
 TEST_F(NetworkChangeNotifierLinuxTest, IgnoreContinuedOfflineState) {
   OfflineObserver observer;
 
   SendResponse(NM_STATE_DISCONNECTING);
-  EXPECT_TRUE(NetworkChangeNotifier::IsOffline());
+  EXPECT_TRUE(NetworkChangeNotifier::GetConnectionType() ==
+                  NetworkChangeNotifier::CONNECTION_NONE);
   EXPECT_EQ(0, observer.notification_count);
 
   SendSignal(NM_STATE_DISCONNECTED);
-  EXPECT_TRUE(NetworkChangeNotifier::IsOffline());
+  EXPECT_TRUE(NetworkChangeNotifier::GetConnectionType() ==
+                  NetworkChangeNotifier::CONNECTION_NONE);
   EXPECT_EQ(0, observer.notification_count);
 }
 
 TEST_F(NetworkChangeNotifierLinuxTest, NullResponse) {
   RunOnNotifierThread(base::Bind(
       response_callback_, static_cast<dbus::Response*>(NULL)));
-  EXPECT_FALSE(NetworkChangeNotifier::IsOffline());
+  EXPECT_FALSE(NetworkChangeNotifier::GetConnectionType() ==
+                   NetworkChangeNotifier::CONNECTION_NONE);
 }
 
 TEST_F(NetworkChangeNotifierLinuxTest, EmptyResponse) {
   scoped_ptr<dbus::Response> response(dbus::Response::CreateEmpty());
   RunOnNotifierThread(base::Bind(response_callback_, response.get()));
-  EXPECT_FALSE(NetworkChangeNotifier::IsOffline());
+  EXPECT_FALSE(NetworkChangeNotifier::GetConnectionType() ==
+                   NetworkChangeNotifier::CONNECTION_NONE);
 }
 
 TEST_F(NetworkChangeNotifierLinuxTest, InvalidResponse) {
   scoped_ptr<dbus::Response> response(dbus::Response::CreateEmpty());
   dbus::MessageWriter writer(response.get());
   writer.AppendUint16(20);  // Uint16 instead of the expected Uint32
   RunOnNotifierThread(base::Bind(response_callback_, response.get()));
-  EXPECT_FALSE(NetworkChangeNotifier::IsOffline());
+  EXPECT_FALSE(NetworkChangeNotifier::GetConnectionType() ==
+                   NetworkChangeNotifier::CONNECTION_NONE);
 }
 
 }  // namespace
 }  // namespace net