Refactored signin/sync error controllers for the avatar button

chrome/browser/ui/avatar_button_error_controller.cc

+// Copyright 2016 The Chromium Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+#include "chrome/browser/ui/avatar_button_error_controller.h"
+
+#include "chrome/browser/profiles/profiles_state.h"
+#include "chrome/browser/sync/profile_sync_service_factory.h"
+#include "components/browser_sync/browser/profile_sync_service.h"
+#include "components/signin/core/browser/signin_error_controller.h"
+#include "components/signin/core/common/profile_management_switches.h"
+
+AvatarButtonErrorController::AvatarButtonErrorController(Profile* profile)
+    : avatar_signin_error_controller_(profile, this),
+      avatar_sync_error_controller_(profile, this),
+      has_signin_error_(false),
+      has_sync_error_(false) {}
+
+void AvatarButtonErrorController::UpdateSigninError(bool has_signin_error) {
+  has_signin_error_ = has_signin_error;
+  OnAvatarErrorChanged();
+}
+
+void AvatarButtonErrorController::UpdateSyncError(bool has_sync_error) {
+  has_sync_error_ = has_sync_error;
+  OnAvatarErrorChanged();
+}
+
+void AvatarButtonErrorController::GetAvatarErrorUpdate() {
+  avatar_signin_error_controller_.OnErrorChanged();
+  avatar_sync_error_controller_.OnErrorChanged();
+}
+
+AvatarButtonErrorController::AvatarSigninErrorController::
+    AvatarSigninErrorController(
+        Profile* profile,
+        AvatarButtonErrorController* avatar_button_error_controller)
+    : profile_(profile),
+      avatar_button_error_controller_(avatar_button_error_controller) {
+  SigninErrorController* signin_error_controller =
+      profiles::GetSigninErrorController(profile_);
+  if (signin_error_controller)
+    signin_error_controller->AddObserver(this);
+}
+
+AvatarButtonErrorController::AvatarSigninErrorController::
+    ~AvatarSigninErrorController() {
+  SigninErrorController* signin_error_controller =
+      profiles::GetSigninErrorController(profile_);
+  if (signin_error_controller)
+    signin_error_controller->RemoveObserver(this);
+}
+
+void AvatarButtonErrorController::AvatarSigninErrorController::
+    OnErrorChanged() {
+  const SigninErrorController* signin_error_controller =
+      profiles::GetSigninErrorController(profile_);
+  avatar_button_error_controller_->UpdateSigninError(
+      signin_error_controller && signin_error_controller->HasError());
+}
+
+AvatarButtonErrorController::AvatarSyncErrorController::
+    AvatarSyncErrorController(
+        Profile* profile,
+        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::AvatarSyncErrorController::
+    ~AvatarSyncErrorController() {
+  SyncErrorController* sync_error_controller = GetSyncErrorControllerIfNeeded();
+  if (sync_error_controller)
+    sync_error_controller->RemoveObserver(this);
+}
+
+void AvatarButtonErrorController::AvatarSyncErrorController::OnErrorChanged() {
+  ProfileSyncService* sync_service =
+      ProfileSyncServiceFactory::GetForProfile(profile_);
+  if (switches::IsMaterialDesignUserMenu() && sync_service) {
+    SyncErrorController* sync_error_controller =
+        sync_service->sync_error_controller();
+    ProfileSyncService::Status status;
+    sync_service->QueryDetailedSyncStatus(&status);
+    avatar_button_error_controller_->UpdateSyncError(
+        sync_service->HasUnrecoverableError() ||
+        status.sync_protocol_error.action == syncer::UPGRADE_CLIENT ||
+        (sync_error_controller && sync_error_controller->HasError()));
+  } else {
+    avatar_button_error_controller_->UpdateSyncError(false);
+  }
+}
+
+SyncErrorController* AvatarButtonErrorController::AvatarSyncErrorController::
+    GetSyncErrorControllerIfNeeded() {
+  if (!switches::IsMaterialDesignUserMenu())
+    return nullptr;
+  ProfileSyncService* sync_service =
+      ProfileSyncServiceFactory::GetForProfile(profile_);
+  return sync_service ? sync_service->sync_error_controller() : nullptr;
+}