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

Unified Diff: chrome/browser/chromeos/extensions/file_browser_event_router.cc

Issue 12729002: Add a unified observer to replace NetworkManagerObserver (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: . Created 7 years, 9 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/chromeos/extensions/file_browser_event_router.cc
diff --git a/chrome/browser/chromeos/extensions/file_browser_event_router.cc b/chrome/browser/chromeos/extensions/file_browser_event_router.cc
index d16dd15098f8e5099f33b31e4056b72bf7b930d8..ab3fcc7ea76916aac0e05b844773bb6149b7ea9c 100644
--- a/chrome/browser/chromeos/extensions/file_browser_event_router.cc
+++ b/chrome/browser/chromeos/extensions/file_browser_event_router.cc
@@ -12,7 +12,6 @@
#include "base/prefs/public/pref_change_registrar.h"
#include "base/stl_util.h"
#include "base/values.h"
-#include "chrome/browser/chromeos/cros/cros_library.h"
#include "chrome/browser/chromeos/drive/drive_cache.h"
#include "chrome/browser/chromeos/drive/drive_file_system_interface.h"
#include "chrome/browser/chromeos/drive/drive_file_system_util.h"
@@ -22,6 +21,7 @@
#include "chrome/browser/chromeos/login/base_login_display_host.h"
#include "chrome/browser/chromeos/login/screen_locker.h"
#include "chrome/browser/chromeos/login/user_manager.h"
+#include "chrome/browser/chromeos/net/connectivity_state_helper.h"
#include "chrome/browser/extensions/event_names.h"
#include "chrome/browser/extensions/event_router.h"
#include "chrome/browser/extensions/extension_service.h"
@@ -272,14 +272,10 @@ void FileBrowserEventRouter::Shutdown() {
system_service->drive_service()->RemoveObserver(this);
}
- chromeos::CrosLibrary* cros_library = chromeos::CrosLibrary::Get();
- if (cros_library) {
- chromeos::NetworkLibrary* network_library =
- cros_library->GetNetworkLibrary();
- if (network_library)
- network_library->RemoveNetworkManagerObserver(this);
+ if (chromeos::ConnectivityStateHelper::IsInitialized()) {
+ chromeos::ConnectivityStateHelper::Get()->
+ RemoveNetworkManagerObserver(this);
}
-
profile_ = NULL;
}
@@ -305,14 +301,10 @@ void FileBrowserEventRouter::ObserveFileSystemEvents() {
system_service->file_system()->AddObserver(this);
}
- chromeos::CrosLibrary* cros_library = chromeos::CrosLibrary::Get();
- if (cros_library) {
- chromeos::NetworkLibrary* network_library =
- cros_library->GetNetworkLibrary();
- if (network_library)
- network_library->AddNetworkManagerObserver(this);
+ if (chromeos::ConnectivityStateHelper::IsInitialized()) {
+ chromeos::ConnectivityStateHelper::Get()->
+ AddNetworkManagerObserver(this);
}
-
suspend_state_delegate_.reset(new SuspendStateDelegateImpl());
pref_change_registrar_->Init(profile_->GetPrefs());
@@ -533,8 +525,7 @@ void FileBrowserEventRouter::OnFormatEvent(
}
}
-void FileBrowserEventRouter::OnNetworkManagerChanged(
- chromeos::NetworkLibrary* network_library) {
+void FileBrowserEventRouter::NetworkManagerChanged() {
if (!profile_ ||
!extensions::ExtensionSystem::Get(profile_)->event_router()) {
NOTREACHED();
« no previous file with comments | « chrome/browser/chromeos/extensions/file_browser_event_router.h ('k') | chrome/browser/chromeos/imageburner/burn_manager.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698