Commit b21095a7 authored by estade@chromium.org's avatar estade@chromium.org

fix sync login UI on reload

BUG=138338
TEST=manual

Review URL: https://chromiumcodereview.appspot.com/10823191

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@150244 0039d316-1c4b-4281-b951-d872f2087c98
parent ac2473f4
......@@ -209,6 +209,10 @@ void SyncPromoHandler::HandleCloseSyncPromo(const base::ListValue* args) {
}
void SyncPromoHandler::HandleInitializeSyncPromo(const base::ListValue* args) {
// If this is a page reload, then we have to inform the login service
// the old UI closed. If this is an initial load, this call will do nothing.
GetLoginUIService()->LoginUIClosed(this);
// Open the sync wizard to the login screen.
OpenSyncSetup(true);
// We don't need to compute anything for this, just do this every time.
......
......@@ -97,6 +97,9 @@ class SyncSetupHandler : public options2::OptionsPageUIHandler,
// profile.
ProfileSyncService* GetSyncService() const;
// Returns the LoginUIService for the parent profile.
LoginUIService* GetLoginUIService() const;
private:
// Callbacks from the page.
void OnDidClosePage(const base::ListValue* args);
......@@ -170,9 +173,6 @@ class SyncSetupHandler : public options2::OptionsPageUIHandler,
// Returns the SigninManager for the parent profile.
SigninManager* GetSignin() const;
// Returns the LoginUIService for the parent profile.
LoginUIService* GetLoginUIService() const;
// The SigninTracker object used to determine when the user has fully signed
// in (this requires waiting for various services to initialize and tracking
// errors from multiple sources). Should only be non-null while the login UI
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment