Handle refresh on sync setup page.

chrome/browser/ui/webui/sync_promo/sync_promo_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_promo/sync_promo_handler.h"
 
 #include "base/bind.h"
 #include "base/bind_helpers.h"
 #include "base/metrics/histogram.h"
 #include "base/time.h"
@@ skipping 191 matching lines @@
     } else {
       GURL url = SyncPromoUI::GetNextPageURLForSyncPromoURL(sync_url);
       OpenURLParams params(
           url, Referrer(), CURRENT_TAB, content::PAGE_TRANSITION_LINK, false);
       web_ui()->GetWebContents()->OpenURL(params);
     }
   }
 }
 
 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);

[Evan Stade, 2012/08/11 00:36:12]

the code was calling ShowSetupUI (which called OpenSyncSetup), then Initialize
(this function). Therefore we called: OpenSyncSetup, LoginUIClosed,
OpenSyncSetup. This broke the identity api (the only subscriber to
OnLoginUIClosed).

+  if (IsActiveLogin()) {
+    // Page has been reloaded. We don't need to do initialization of the setup
+    // handler but we do have to refresh the page.
+    DisplayGaiaLogin(false);

[Andrew T Wilson (Slow), 2012/08/11 01:10:33]

Hmmm. Is this the correct thing to do here? I'm concerned about the case where:

1) The user has a custom passphrase set for his acct.
2) He logs in to the promo
3) He is presented with the "advanced" sync setup page to enter his custom
passphrase
4) He hits reload.

Not sure what happens in this case if you call DisplayGaiaLogin(false). The user
is in a signed-in state so displaying a login seems like the wrong thing (note
the code in SyncSetupHandler::OpenSyncSetup() that displays different UI
depending on the current state of sync - you are bypassing this code by calling
DisplayGaiaLogin() directly).

It feels like the right thing to do is always call OpenSyncSetup(true) as the
code was doing before - I don't quite understand what the fail case is.

[Evan Stade, 2012/08/11 01:28:55]

that code always displays the gaia login when force_login is true, as it is
below. The difference is that OpenSyncSetup frequently does nothing (it only
works the first time you call it).

I just tested the scenario you suggested by adding a custom passphrase to my
test account, signing out and trying to sign back in with chrome://signin. When
I reloaded on the advanced page, it dumped me back to the login page with my
username filled in but the password field blank. This seems reasonable and it is
the same thing that happens when I sign in via chrome://settings, but reload
before configuring.
When you reload, OpenSyncSetup(true) does nothing because PrepareSyncSetup
returns false.

+  } else {
+    // Do setup and show the login page.
+    OpenSyncSetup(true);
+  }
 
-  // Open the sync wizard to the login screen.
-  OpenSyncSetup(true);
   // We don't need to compute anything for this, just do this every time.
   RecordUserFlowAction(SYNC_PROMO_VIEWED);
   // Increment view count first and show natural numbers in stats rather than 0
   // based starting point (if it happened to be our first time showing this).
   IncrementViewCountBy(1);
   // Record +1 for every view.  This is the only thing we record that's not part
   // of the user flow histogram.
   UMA_HISTOGRAM_COUNTS("SyncPromo.NumTimesViewed", GetViewCount());
 }
 
@@ skipping 56 matching lines @@
   // Send an enumeration to our single user flow histogram.
   UMA_HISTOGRAM_ENUMERATION("SyncPromo.UserFlow", action,
                             SYNC_PROMO_BUCKET_BOUNDARY);
 }
 
 void SyncPromoHandler::CloseUI() {
   // Callers should not ever try to close the promo UI (should only call
   // CloseUI() if the user is already logged in).
   NOTREACHED() << "Cannot close the promo UI";
 }