Removing pre-material design menu setting.

chrome/browser/ui/avatar_button_error_controller.cc

Index: chrome/browser/ui/avatar_button_error_controller.cc
diff --git a/chrome/browser/ui/avatar_button_error_controller.cc b/chrome/browser/ui/avatar_button_error_controller.cc
index 647ba4353b459add54a4f9e10346acb8e0e78f5d..8244d5889ceecc4f6438c8b7f25cd710301f7ef4 100644
--- a/chrome/browser/ui/avatar_button_error_controller.cc
+++ b/chrome/browser/ui/avatar_button_error_controller.cc
@@ -70,16 +70,9 @@ AvatarButtonErrorController::SyncErrorObserver::SyncErrorObserver(
     AvatarButtonErrorController* avatar_button_error_controller)
     : profile_(profile),
       avatar_button_error_controller_(avatar_button_error_controller) {
-  SyncErrorController* sync_error_controller = GetSyncErrorControllerIfNeeded();
-  if (sync_error_controller)
-    sync_error_controller->AddObserver(this);
 }
 
-AvatarButtonErrorController::SyncErrorObserver::~SyncErrorObserver() {
-  SyncErrorController* sync_error_controller = GetSyncErrorControllerIfNeeded();
-  if (sync_error_controller)
-    sync_error_controller->RemoveObserver(this);
-}
+AvatarButtonErrorController::SyncErrorObserver::~SyncErrorObserver() {}
 
 void AvatarButtonErrorController::SyncErrorObserver::OnErrorChanged() {

[sky, 2017/02/23 20:44:52]

Is this function ever called now? Who adds this class as an observer now? If no
one does, does it need to implement SigninErrorController::Observer?

[jlebel, 2017/03/06 13:09:29]

Sorry I definitely made a mistake here. GetSyncErrorControllerIfNeeded() should
not have been removed.

   avatar_button_error_controller_->UpdateSyncError(HasSyncError());
@@ -88,7 +81,7 @@ void AvatarButtonErrorController::SyncErrorObserver::OnErrorChanged() {
 bool AvatarButtonErrorController::SyncErrorObserver::HasSyncError() {
   browser_sync::ProfileSyncService* sync_service =
       ProfileSyncServiceFactory::GetForProfile(profile_);
-  if (switches::IsMaterialDesignUserMenu() && sync_service) {
+  if (sync_service) {
     SyncErrorController* sync_error_controller =
         sync_service->sync_error_controller();
     browser_sync::ProfileSyncService::Status status;
@@ -99,12 +92,3 @@ bool AvatarButtonErrorController::SyncErrorObserver::HasSyncError() {
   }
   return false;
 }
-
-SyncErrorController* AvatarButtonErrorController::SyncErrorObserver::
-    GetSyncErrorControllerIfNeeded() {
-  if (!switches::IsMaterialDesignUserMenu())
-    return nullptr;
-  browser_sync::ProfileSyncService* sync_service =
-      ProfileSyncServiceFactory::GetForProfile(profile_);
-  return sync_service ? sync_service->sync_error_controller() : nullptr;
-}