Suppress bubble view for CONNECTION_FAILED error

chrome/browser/sync/sync_global_error.cc

 // Copyright (c) 2011 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/sync/sync_global_error.h"
 
 #include "chrome/app/chrome_command_ids.h"
 #include "chrome/browser/sync/profile_sync_service.h"
 #include "chrome/browser/sync/profile_sync_service_observer.h"
 #include "chrome/browser/sync/sync_ui_util.h"
 #include "chrome/browser/ui/global_error_service.h"
 #include "chrome/browser/ui/global_error_service_factory.h"
 #include "chrome/common/net/gaia/google_service_auth_error.h"
 #include "grit/chromium_strings.h"
 #include "grit/generated_resources.h"
 #include "ui/base/l10n/l10n_util.h"
 
 typedef GoogleServiceAuthError AuthError;
 
-using namespace sync_ui_util;
-
 SyncGlobalError::SyncGlobalError(ProfileSyncService* service)
-    : has_error_(false),
-      service_(service) {
+    : service_(service) {
   OnStateChanged();
 }
 
 SyncGlobalError::~SyncGlobalError() {
 }
 
 bool SyncGlobalError::HasBadge() {
-  return GetStatusLabelsForSyncGlobalError(service_, NULL, NULL, NULL) ==
-      SYNC_ERROR;
+  return !menu_label_.empty();
 }
 
 bool SyncGlobalError::HasMenuItem() {
   // When we're on Chrome OS we need to add a separate menu item to the wrench
   // menu to the show the error. On other platforms we can just reuse the
   // "Sign in to Chrome..." menu item to show the error.
 #if defined(OS_CHROMEOS)
-  return GetStatusLabelsForSyncGlobalError(service_, NULL, NULL, NULL) ==
-      SYNC_ERROR;
+  return !menu_label_.empty();
 #else
   return false;
 #endif
 }
 
 int SyncGlobalError::MenuItemCommandID() {
   return IDC_SHOW_SYNC_ERROR;
 }
 
 string16 SyncGlobalError::MenuItemLabel() {
-  string16 label;
-  GetStatusLabelsForSyncGlobalError(service_, &label, NULL, NULL);
-  return label;
+  return menu_label_;
 }
 
 void SyncGlobalError::ExecuteMenuItem(Browser* browser) {
   service_->ShowErrorUI();
 }
 
 bool SyncGlobalError::HasBubbleView() {
-  return GetStatusLabelsForSyncGlobalError(service_, NULL, NULL, NULL) ==
-      SYNC_ERROR;
+  return !bubble_message_.empty() && !bubble_accept_label_.empty();
 }
 
 string16 SyncGlobalError::GetBubbleViewTitle() {
   return l10n_util::GetStringUTF16(IDS_SYNC_ERROR_BUBBLE_VIEW_TITLE);
 }
 
 string16 SyncGlobalError::GetBubbleViewMessage() {
-  string16 label;
-  GetStatusLabelsForSyncGlobalError(service_, NULL, &label, NULL);
-  return label;
+  return bubble_message_;
 }
 
 string16 SyncGlobalError::GetBubbleViewAcceptButtonLabel() {
-  string16 label;
-  GetStatusLabelsForSyncGlobalError(service_, NULL, NULL, &label);
-  return label;
+  return bubble_accept_label_;
 }
 
 string16 SyncGlobalError::GetBubbleViewCancelButtonLabel() {
   return string16();
 }
 
 void SyncGlobalError::BubbleViewDidClose() {
 }
 
 void SyncGlobalError::BubbleViewAcceptButtonPressed() {
   service_->ShowErrorUI();
 }
 
 void SyncGlobalError::BubbleViewCancelButtonPressed() {
   NOTREACHED();
 }
 
 void SyncGlobalError::OnStateChanged() {
-  bool new_has_error = GetStatusLabelsForSyncGlobalError(
-      service_, NULL, NULL, NULL) == SYNC_ERROR;
-  if (new_has_error != has_error_) {
-    has_error_ = new_has_error;
-    GlobalErrorServiceFactory::GetForProfile(
-        service_->profile())->NotifyErrorsChanged(this);
+  string16 menu_label;
+  string16 bubble_message;
+  string16 bubble_accept_label;
+  sync_ui_util::GetStatusLabelsForSyncGlobalError(
+      service_, &menu_label, &bubble_message, &bubble_accept_label);
+
+  // All the labels should be empty or all of them non-empty.
+  DCHECK((menu_label.empty() && bubble_message.empty() &&
+          bubble_accept_label.empty()) ||
+         (!menu_label.empty() && !bubble_message.empty() &&
+          !bubble_accept_label.empty()));
+
+  if (menu_label != menu_label_ || bubble_message != bubble_message_ ||
+      bubble_accept_label != bubble_accept_label_) {
+    menu_label_ = menu_label;
+    bubble_message_ = bubble_message;
+    bubble_accept_label_ = bubble_accept_label;
+
+    // Profile can be NULL during tests.
+    Profile* profile = service_->profile();
+    if (profile) {
+      GlobalErrorServiceFactory::GetForProfile(
+          profile)->NotifyErrorsChanged(this);
+    }
   }
 }
 
 bool SyncGlobalError::HasCustomizedSyncMenuItem() {
-  return GetStatusLabelsForSyncGlobalError(service_, NULL, NULL, NULL) ==
-      SYNC_ERROR;
+  return !menu_label_.empty();
 }