suppress user/password dialog when re-enabling sync

chrome/browser/ui/webui/sync_setup_handler.cc

 // Copyright (c) 2012 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/webui/sync_setup_handler.h"
 
 #include "base/bind.h"
 #include "base/basictypes.h"
 #include "base/bind_helpers.h"
 #include "base/compiler_specific.h"
 #include "base/json/json_reader.h"
 #include "base/json/json_writer.h"
 #include "base/utf_string_conversions.h"
 #include "base/values.h"
 #include "chrome/app/chrome_command_ids.h"
 #include "chrome/browser/google/google_util.h"
 #include "chrome/browser/profiles/profile.h"
 #include "chrome/browser/profiles/profile_info_cache.h"
 #include "chrome/browser/profiles/profile_manager.h"
 #include "chrome/browser/profiles/profile_metrics.h"
 #include "chrome/browser/signin/signin_manager.h"
 #include "chrome/browser/signin/signin_manager_factory.h"
+#include "chrome/browser/signin/token_service.h"
+#include "chrome/browser/signin/token_service_factory.h"
 #include "chrome/browser/sync/profile_sync_service.h"
 #include "chrome/browser/sync/profile_sync_service_factory.h"
 #include "chrome/browser/ui/browser_list.h"
 #include "chrome/browser/ui/webui/signin/login_ui_service.h"
 #include "chrome/browser/ui/webui/signin/login_ui_service_factory.h"
 #include "chrome/browser/ui/webui/sync_promo/sync_promo_ui.h"
 #include "chrome/common/net/gaia/gaia_constants.h"
 #include "chrome/common/url_constants.h"
 #include "content/public/browser/render_view_host.h"
 #include "content/public/browser/render_view_host_delegate.h"
@@ skipping 450 matching lines @@
   if (!error_message.empty())
     args.SetString("error_message", error_message);
   if (fatal_error)
     args.SetBoolean("fatalError", true);
   args.SetString("captchaUrl", captcha);
   StringValue page("login");
   web_ui()->CallJavascriptFunction(
       "SyncSetupOverlay.showSyncSetupPage", page, args);
 }
 
+// TODO(kochi): Handle error conditions (timeout, other failures).
+void SyncSetupHandler::DisplaySpinner() {
+  configuring_sync_ = true;
+  StringValue page("spinner");
+  web_ui()->CallJavascriptFunction(
+      "SyncSetupOverlay.showSyncSetupPage", page, DictionaryValue());
+}
+
 void SyncSetupHandler::RecordSignin() {
   // By default, do nothing - subclasses can override.
 }
 
 void SyncSetupHandler::DisplayGaiaSuccessAndClose() {
   RecordSignin();
   web_ui()->CallJavascriptFunction("SyncSetupOverlay.showSuccessAndClose");
 }
 
 void SyncSetupHandler::DisplayGaiaSuccessAndSettingUp() {
@@ skipping 76 matching lines @@
   DCHECK(IsActiveLogin());
 
   // Gaia credentials are valid - update the UI.
   DisplayGaiaSuccessAndSettingUp();
 }
 
 void SyncSetupHandler::SigninFailed(const GoogleServiceAuthError& error) {
   last_signin_error_ = error;
   // Got a failed signin - this is either just a typical auth error, or a
   // sync error (treat sync errors as "fatal errors" - i.e. non-auth errors).
+  // On ChromeOS, this condition should trigger the orange badge on wrench menu
+  // and prompt to sign out.
+#if !defined(OS_CHROMEOS)
   DisplayGaiaLogin(GetSyncService()->unrecoverable_error_detected());
+#endif

[Andrew T Wilson (Slow), 2012/04/03 17:45:13]

We should probably close the overlay here though, to make sure the spinner is
hidden?

[kochi, 2012/04/03 21:55:29]

Done.

 }
 
 Profile* SyncSetupHandler::GetProfile() const {
   return Profile::FromWebUI(web_ui());
 }
 
 ProfileSyncService* SyncSetupHandler::GetSyncService() const {
   return ProfileSyncServiceFactory::GetForProfile(GetProfile());
 }
 
 void SyncSetupHandler::SigninSuccess() {
+  LOG(WARNING) << "SigninSuccess()";
   DCHECK(GetSyncService()->sync_initialized());
   // If we have signed in while sync is already setup, it must be due to some
   // kind of re-authentication flow. In that case, just close the signin dialog
   // rather than forcing the user to go through sync configuration.
-  if (GetSyncService()->HasSyncSetupCompleted())
+  if (GetSyncService()->HasSyncSetupCompleted()) {
     DisplayGaiaSuccessAndClose();
-  else
+  } else {
+#if !defined(OS_CHROMEOS)
     DisplayConfigureSync(false, false);
+#else
+    // On ChromeOS, clicking on "Set up Sync" should show advanced settings.
+    DisplayConfigureSync(true, false);

[Andrew T Wilson (Slow), 2012/04/03 17:45:13]

Are you sure that's what we want here (not the "Sync Everything" button
instead)?

[kochi, 2012/04/03 21:55:29]

I thought this was the intention (before enabling sync,
give the user an opportunity to configure which data types
to be synced), but could be okay as the user can configure
data types anyway.  To make the diversity less, removed this
ifdef.

+#endif
+  }
 }
 
 void SyncSetupHandler::HandleConfigure(const ListValue* args) {
   std::string json;
   if (!args->GetString(0, &json)) {
     NOTREACHED() << "Could not read JSON argument";
     return;
   }
   if (json.empty()) {
     NOTREACHED();
@@ skipping 130 matching lines @@
     }
 
     // Let the various services know that we're no longer active.
     GetLoginUIService()->LoginUIClosed(web_ui());
     if (sync_service)
       sync_service->set_setup_in_progress(false);
 
     // Make sure user isn't left half-logged-in (signed in, but without sync
     // started up). If the user hasn't finished setting up sync, then sign out
     // and shut down sync.
-
     if (sync_service && !sync_service->HasSyncSetupCompleted()) {
       DVLOG(1) << "Signin aborted by user action";
       sync_service->DisableForUser();
       GetSignin()->SignOut();
     }
   }
 
   configuring_sync_ = false;
   signin_tracker_.reset();
 }
@@ skipping 18 matching lines @@
 
   // Notify services that we are now active.
   GetLoginUIService()->SetLoginUI(web_ui());
   service->set_setup_in_progress(true);
 
   if (!force_login && service->HasSyncSetupCompleted()) {
     // User is already logged in. They must have brought up the config wizard
     // via the "Advanced..." button or the wrench menu.
     DisplayConfigureSync(true, false);
   } else {
+#if !defined(OS_CHROMEOS)
     // User is not logged in - need to display login UI.
     DisplayGaiaLogin(false);
+#else
+    // On Chrome OS user is always logged in. Instead of showing login dialog,
+    // show spinner until the backend gets ready to configure sync.
+
+    // 1. To make PSS:AreCredentialsAvailable() to return true, first we need
+    //    to unsppress the flag.
+    browser_sync::SyncPrefs sync_prefs(GetProfile()->GetPrefs());

[Andrew T Wilson (Slow), 2012/04/03 17:45:13]

Should probably just move this into a helper function.

Also, if you never want to display a GAIA login on cros, you should probably #if
it out of the header and .cc file to make sure calls don't sneak back in.

[kochi, 2012/04/03 21:55:29]

Split this part into a helper function.

For #ifdef'ing the non-ChromeOS part makes this
CL bigger, so i'd like to defer it for later CLs.

On 2012/04/03 17:45:13, Andrew T Wilson wrote:

+    sync_prefs.SetStartSuppressed(false);
+
+    // 2. To listen to the token available notifications, start SigninTracker.
+    signin_tracker_.reset(new SigninTracker(GetProfile(), this));

[Andrew T Wilson (Slow), 2012/04/03 17:45:13]

We don't need a signin tracker in the case that sync is already initialized.

[kochi, 2012/04/03 21:55:29]

But for the case sync backend is stopped, we need to have
SigninTracker.

+    TokenService* token_service =
+        TokenServiceFactory::GetForProfile(GetProfile());
+    DCHECK(token_service);
+    if (token_service->AreCredentialsValid()) {
+      token_service->StartFetchingTokens();

[Andrew T Wilson (Slow), 2012/04/03 17:45:13]

Shouldn't need to do this once you remove SigninManager::SignOut() on cros.

[kochi, 2012/04/03 21:55:29]

Removed.

+    } else {
+      LOG(WARNING) << "TokenService: credentials are invalid!";
+      token_service->LoadTokensFromDB();
+    }
+    if (!service->sync_initialized()) {
+      service->set_setup_in_progress(true);

[Andrew T Wilson (Slow), 2012/04/03 17:45:13]

This looks good. In fact, we might want to have this in general (not just for
os_chromeos) in case someone tries to bring up the configure sync dialog before
the backend is initialized.

[kochi, 2012/04/03 21:55:29]

So as not to break non-ChromeOS for now, I'd like to
keep this as is...

On 2012/04/03 17:45:13, Andrew T Wilson wrote:

+      service->UnsuppressAndStart();
+      DisplaySpinner();
+    } else {
+      DisplayConfigureSync(true, false);
+    }
+#endif
   }
 
   ShowSetupUI();
 }
 
 // Private member functions.
 
 bool SyncSetupHandler::FocusExistingWizardIfPresent() {
   LoginUIService* service = GetLoginUIService();
   if (!service->current_login_ui())
@@ skipping 30 matching lines @@
     if (i != current_profile_index && AreUserNamesEqual(
         cache.GetUserNameOfProfileAtIndex(i), username_utf16)) {
       *error_message = l10n_util::GetStringUTF16(
           IDS_SYNC_USER_NAME_IN_USE_ERROR);
       return false;
     }
   }
 
   return true;
 }