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

Unified Diff: chrome/browser/ui/app_list/app_list_service_impl.cc

Issue 22268009: Move signin status and current user information into AppListModel. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: rebase Created 7 years, 4 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/app_list/app_list_service_impl.cc
diff --git a/chrome/browser/ui/app_list/app_list_service_impl.cc b/chrome/browser/ui/app_list/app_list_service_impl.cc
index 102809c0aa0ba7ea67bc039a3d4e1e50722aa222..83921a702e67d3ab2486c7cd11a3b3b9288bdf8e 100644
--- a/chrome/browser/ui/app_list/app_list_service_impl.cc
+++ b/chrome/browser/ui/app_list/app_list_service_impl.cc
@@ -10,14 +10,11 @@
#include "base/prefs/pref_service.h"
#include "base/time/time.h"
#include "chrome/browser/browser_process.h"
-#include "chrome/browser/chrome_notification_types.h"
#include "chrome/browser/profiles/profile_manager.h"
#include "chrome/common/chrome_constants.h"
#include "chrome/common/chrome_switches.h"
#include "chrome/common/pref_names.h"
#include "content/public/browser/browser_thread.h"
-#include "content/public/browser/notification_details.h"
-#include "content/public/browser/notification_source.h"
namespace {
@@ -158,7 +155,6 @@ AppListControllerDelegate* AppListServiceImpl::CreateControllerDelegate() {
}
void AppListServiceImpl::CreateShortcut() {}
-void AppListServiceImpl::OnSigninStatusChanged() {}
// We need to watch for profile removal to keep kAppListProfile updated.
void AppListServiceImpl::OnProfileWillBeRemoved(
@@ -174,13 +170,6 @@ void AppListServiceImpl::OnProfileWillBeRemoved(
}
}
-void AppListServiceImpl::Observe(
- int type,
- const content::NotificationSource& source,
- const content::NotificationDetails& details) {
- OnSigninStatusChanged();
-}
-
void AppListServiceImpl::Show() {
profile_loader_.LoadProfileInvalidatingOtherLoads(
GetProfilePath(g_browser_process->profile_manager()->user_data_dir()),
@@ -202,17 +191,7 @@ Profile* AppListServiceImpl::GetCurrentAppListProfile() {
}
void AppListServiceImpl::SetProfile(Profile* new_profile) {
- registrar_.RemoveAll();
profile_ = new_profile;
- if (!profile_)
- return;
-
- registrar_.Add(this, chrome::NOTIFICATION_GOOGLE_SIGNIN_SUCCESSFUL,
- content::Source<Profile>(profile_));
- registrar_.Add(this, chrome::NOTIFICATION_GOOGLE_SIGNIN_FAILED,
- content::Source<Profile>(profile_));
- registrar_.Add(this, chrome::NOTIFICATION_GOOGLE_SIGNED_OUT,
- content::Source<Profile>(profile_));
}
void AppListServiceImpl::InvalidatePendingProfileLoads() {
« no previous file with comments | « chrome/browser/ui/app_list/app_list_service_impl.h ('k') | chrome/browser/ui/app_list/app_list_service_mac.mm » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698