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

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

Issue 2350623002: Deleted AppListViewDelegateObserver class (was unused). (Closed)
Patch Set: Created 4 years, 3 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_view_delegate.cc
diff --git a/chrome/browser/ui/app_list/app_list_view_delegate.cc b/chrome/browser/ui/app_list/app_list_view_delegate.cc
index 2d2b668ab469d38e54ac849f9ff5b889d183c0ec..d5e06533a1979e052470d96137480746cadad8d2 100644
--- a/chrome/browser/ui/app_list/app_list_view_delegate.cc
+++ b/chrome/browser/ui/app_list/app_list_view_delegate.cc
@@ -61,7 +61,6 @@
#include "extensions/common/manifest_constants.h"
#include "extensions/common/manifest_handlers/launcher_page_info.h"
#include "ui/app_list/app_list_switches.h"
-#include "ui/app_list/app_list_view_delegate_observer.h"
#include "ui/app_list/search_box_model.h"
#include "ui/app_list/search_controller.h"
#include "ui/app_list/speech_ui_model.h"
@@ -687,16 +686,6 @@ AppListViewDelegate::GetUsers() const {
return users_;
}
-void AppListViewDelegate::AddObserver(
- app_list::AppListViewDelegateObserver* observer) {
- observers_.AddObserver(observer);
-}
-
-void AppListViewDelegate::RemoveObserver(
- app_list::AppListViewDelegateObserver* observer) {
- observers_.RemoveObserver(observer);
-}
-
void AppListViewDelegate::OnTemplateURLServiceChanged() {
TemplateURLService* template_url_service =
TemplateURLServiceFactory::GetForProfile(profile_);
@@ -720,9 +709,6 @@ void AppListViewDelegate::Observe(int type,
const content::NotificationDetails& details) {
DCHECK_EQ(chrome::NOTIFICATION_APP_TERMINATING, type);
- FOR_EACH_OBSERVER(app_list::AppListViewDelegateObserver, observers_,
- OnShutdown());
-
SetProfile(nullptr); // Ensures launcher page web contents are torn down.
// SigninManagerFactory is not a leaky singleton (unlike this class), and

Powered by Google App Engine
This is Rietveld 408576698