Move UI specific implementation from ProfileSyncService to SyncSetupFlowHandler.

chrome/browser/sync/profile_sync_service.cc

Index: chrome/browser/sync/profile_sync_service.cc
diff --git a/chrome/browser/sync/profile_sync_service.cc b/chrome/browser/sync/profile_sync_service.cc
index 94c84596ad193099a654125f19bd21e9e939cdaf..7eeebc965f387201c1cbe3f32ab87e48a0bf08ed 100644
--- a/chrome/browser/sync/profile_sync_service.cc
+++ b/chrome/browser/sync/profile_sync_service.cc
@@ -857,8 +857,7 @@ void ProfileSyncService::PromptForExistingPassphrase() {
 
 void ProfileSyncService::ShowSyncSetup(SyncSetupWizard::State state) {

[Andrew T Wilson (Slow), 2011/07/07 17:44:23]

So, I'm wondering if we should just completely get rid of this routine, and
instead add a ShowSyncSetup(SyncSetupWizard::State state) routine to
SyncSetupWizard that does exactly what this does (steps to that state, and
displays the setup appropriately). And all calls to this routine
(ShowSyncSetup(<state>)) would instead change to wizard_.ShowSyncSetup(<state>).

[tim (not reviewing), 2011/07/07 17:49:47]

hm. wizard_.Step is supposed to do exactly that; advance, and show
appropriately... that's what the if (flow) and SyncSetupFlow::Run stuff is about
in there.

   wizard_.Step(state);
-  BrowserList::GetLastActiveWithProfile(profile())->ShowOptionsTab(
-      chrome::kSyncSetupSubPage);
+  wizard_.ShowSyncSetup();
 }
 
 SyncBackendHost::StatusSummary ProfileSyncService::QuerySyncStatusSummary() {