Start moving signin code out of browser/sync.

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 109 matching lines @@
                  base::Unretained(this)));
   web_ui()->RegisterMessageCallback("SyncPromo:UserFlowAction",
       base::Bind(&SyncPromoHandler::HandleUserFlowAction,
                  base::Unretained(this)));
   web_ui()->RegisterMessageCallback("SyncPromo:UserSkipped",
       base::Bind(&SyncPromoHandler::HandleUserSkipped,
                  base::Unretained(this)));
   SyncSetupHandler::RegisterMessages();
 }
 
-void SyncPromoHandler::ShowGaiaSuccessAndClose() {
+void SyncPromoHandler::RecordSignin() {
   sync_promo_trial::RecordUserSignedIn(web_ui());
-  SyncSetupHandler::ShowGaiaSuccessAndClose();
-}
-
-void SyncPromoHandler::ShowGaiaSuccessAndSettingUp() {
-  sync_promo_trial::RecordUserSignedIn(web_ui());
-  SyncSetupHandler::ShowGaiaSuccessAndSettingUp();
 }
 
 void SyncPromoHandler::ShowConfigure(const base::DictionaryValue& args) {
   bool usePassphrase = false;
   args.GetBoolean("usePassphrase", &usePassphrase);
 
   if (usePassphrase) {
     // If a passphrase is required then we must show the configure pane.
     SyncSetupHandler::ShowConfigure(args);
   } else {
@@ skipping 26 matching lines @@
         window_already_closed_ = true;
       }
       break;
     }
     default: {
       NOTREACHED();
     }
   }
 }
 
-void SyncPromoHandler::StepWizardForShowSetupUI() {
-}
-
 void SyncPromoHandler::ShowSetupUI() {
-  // SyncSetupWizard::Step should be called in StepWizardForShowSetupUI above,
-  // but it causes the sync promo page to not set focus properly to the login
-  // email address. This happens because focus is lost between the call to
-  // StepWizardForShowSetupUI and ShowSetupUI.
-  // TODO(binji): Move this function back and fix the focus the right way.
-  ProfileSyncService* service =
-      ProfileSyncServiceFactory::GetInstance()->GetForProfile(
-          Profile::FromWebUI(web_ui()));
-  service->get_wizard().Step(SyncSetupWizard::GetLoginState());
 }
 
 void SyncPromoHandler::HandleCloseSyncPromo(const base::ListValue* args) {
   CloseSyncSetup();
 
   // If the user has signed in then set the pref to show them NTP bubble
   // confirming that they're signed in.
   std::string username = prefs_->GetString(prefs::kGoogleServicesUsername);
   if (!username.empty())
     prefs_->SetBoolean(prefs::kSyncPromoShowNTPBubble, true);
@@ skipping 110 matching lines @@
   // The following call does not use the standard UMA macro because the
   // histogram name is only known at runtime.  The standard macros declare
   // static variables that won't work if the name changes on differnt calls.
   if (!histogram_name_.empty()) {
     base::Histogram* histogram = base::LinearHistogram::FactoryGet(
         histogram_name_, 1, SYNC_PROMO_BUCKET_BOUNDARY,
         SYNC_PROMO_BUCKET_BOUNDARY + 1, base::Histogram::kNoFlags);
     histogram->Add(action);
   }
 }