Added ChromeOS-specific NetworkChangeNotifier specialization that is wired directly to flimflam.

chrome/browser/chromeos/net/network_change_notifier_chromeos.h

+// Copyright (c) 2011 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 CHROME_BROWSER_CHROMEOS_NET_NETWORK_CHANGE_NOTIFIER_CHROMEOS_H_
+#define CHROME_BROWSER_CHROMEOS_NET_NETWORK_CHANGE_NOTIFIER_CHROMEOS_H_
+#pragma once
+
+#include "base/basictypes.h"
+#include "base/memory/scoped_ptr.h"

[oshima, 2011/05/17 17:33:05]

not used?

[zel, 2011/05/18 00:43:33]

Done.

+#include "chrome/browser/chromeos/cros/network_library.h"
+#include "net/base/network_change_notifier.h"
+
+namespace net {
+
+class NetworkChangeNotifierCros

[oshima, 2011/05/17 17:33:05]

NetworkChangeNotifierChromeos
(to be sync with filename)

+    : public NetworkChangeNotifier,
+      public chromeos::NetworkLibrary::NetworkManagerObserver {
+ public:
+  NetworkChangeNotifierCros();
+
+ private:
+  bool connected_;
+  bool has_active_network_;
+  chromeos::ConnectivityState connectivity_state_;
+  std::string service_path_;
+  std::string ip_address_;

[oshima, 2011/05/17 17:33:05]

data member after methods. Please add description.
Do we need both connected and connectivity_state_?

+
+  virtual ~NetworkChangeNotifierCros();
+
+  // NetworkChangeNotifier overrides.
+  virtual bool IsCurrentlyOffline() const;

[oshima, 2011/05/17 17:33:05]

OVERRIDE

[zel, 2011/05/18 00:43:33]

Done.

+
+  // NetworkManagerObserver overrides:
+  virtual void OnNetworkManagerChanged(chromeos::NetworkLibrary* obj);

[oshima, 2011/05/17 17:33:05]

OVERRIDE

[zel, 2011/05/18 00:43:33]

Done.

+
+  // Updates data members that keep the track the network stack state.
+  void UpdateNetworkState(chromeos::NetworkLibrary* cros);
+
+  DISALLOW_COPY_AND_ASSIGN(NetworkChangeNotifierCros);
+};
+
+}  // namespace net
+
+#endif  // CHROME_BROWSER_CHROMEOS_NET_NETWORK_CHANGE_NOTIFIER_CHROMEOS_H_