Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(6375)

Unified Diff: chrome/browser/ui/webui/chromeos/login/network_state_informer.h

Issue 14134007: NetworkPortalDetector/NetworkStateInformer: Switch over to use NetworkStateHandler (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: rebase again Created 7 years, 7 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/ui/webui/chromeos/login/network_state_informer.h
diff --git a/chrome/browser/ui/webui/chromeos/login/network_state_informer.h b/chrome/browser/ui/webui/chromeos/login/network_state_informer.h
index 0296b4e991f70309fda747f4ccc9d6c910577cb1..180cb3b1deb317ca9fb740e32fe7ac45c1cc1ea9 100644
--- a/chrome/browser/ui/webui/chromeos/login/network_state_informer.h
+++ b/chrome/browser/ui/webui/chromeos/login/network_state_informer.h
@@ -9,18 +9,21 @@
#include <string>
#include "base/cancelable_callback.h"
+#include "base/compiler_specific.h"
#include "base/memory/ref_counted.h"
#include "base/observer_list.h"
-#include "chrome/browser/chromeos/cros/network_library.h"
#include "chrome/browser/chromeos/login/captive_portal_window_proxy.h"
#include "chrome/browser/chromeos/login/screens/error_screen_actor.h"
#include "chrome/browser/chromeos/net/network_portal_detector.h"
+#include "chromeos/network/network_state_handler_observer.h"
#include "content/public/browser/notification_observer.h"
#include "content/public/browser/notification_registrar.h"
#include "content/public/browser/notification_service.h"
namespace chromeos {
+class NetworkState;
+
class NetworkStateInformerDelegate {
public:
NetworkStateInformerDelegate() {}
@@ -34,7 +37,7 @@ class NetworkStateInformerDelegate {
// State is considered changed if connection or the active network has been
// changed. Also, it answers to the requests about current network state.
class NetworkStateInformer
- : public chromeos::NetworkLibrary::NetworkManagerObserver,
+ : public chromeos::NetworkStateHandlerObserver,
public chromeos::NetworkPortalDetector::Observer,
public content::NotificationObserver,
public CaptivePortalWindowProxyDelegate,
@@ -70,12 +73,13 @@ class NetworkStateInformer
// Removes observer.
void RemoveObserver(NetworkStateInformerObserver* observer);
- // NetworkLibrary::NetworkManagerObserver implementation:
- virtual void OnNetworkManagerChanged(chromeos::NetworkLibrary* cros) OVERRIDE;
+ // NetworkStateHandlerObserver implementation:
+ virtual void NetworkManagerChanged() OVERRIDE;
+ virtual void DefaultNetworkChanged(const NetworkState* network) OVERRIDE;
// NetworkPortalDetector::Observer implementation:
virtual void OnPortalDetectionCompleted(
- const Network* network,
+ const NetworkState* network,
const NetworkPortalDetector::CaptivePortalState& state) OVERRIDE;
// content::NotificationObserver implementation.
@@ -97,7 +101,7 @@ class NetworkStateInformer
std::string last_network_service_path() const {
return last_network_service_path_;
}
- ConnectionType last_network_type() const { return last_network_type_; }
+ std::string last_network_type() const { return last_network_type_; }
private:
struct ProxyState {
@@ -119,14 +123,14 @@ class NetworkStateInformer
virtual ~NetworkStateInformer();
- bool UpdateState(chromeos::NetworkLibrary* cros);
+ bool UpdateState();
void UpdateStateAndNotify();
void SendStateToObservers(ErrorScreenActor::ErrorReason reason);
- State GetNetworkState(const Network* network);
- bool IsProxyConfigured(const Network* network);
+ State GetNetworkState(const NetworkState* network);
+ bool IsProxyConfigured(const NetworkState* network);
content::NotificationRegistrar registrar_;
State state_;
@@ -135,7 +139,7 @@ class NetworkStateInformer
std::string last_online_service_path_;
std::string last_connected_service_path_;
std::string last_network_service_path_;
- ConnectionType last_network_type_;
+ std::string last_network_type_;
base::CancelableClosure check_state_;
// Caches proxy state for active networks.

Powered by Google App Engine
This is Rietveld 408576698