Move UI specific implementation from ProfileSyncService to SyncSetupFlowHandler.

chrome/browser/sync/sync_setup_wizard.cc

Index: chrome/browser/sync/sync_setup_wizard.cc
diff --git a/chrome/browser/sync/sync_setup_wizard.cc b/chrome/browser/sync/sync_setup_wizard.cc
index b8dc3dd27d94d098911a45d81cdf6818a39a8a12..a9e94e759032a41b4e760f021454b827bd0e0f66 100644
--- a/chrome/browser/sync/sync_setup_wizard.cc
+++ b/chrome/browser/sync/sync_setup_wizard.cc
@@ -83,6 +83,12 @@ void SyncSetupWizard::Focus() {
     flow->Focus();
 }
 
+void SyncSetupWizard::ShowSyncSetup() {
+  SyncSetupFlow* flow = flow_container_->get_flow();
+  if (flow)
+    flow->ShowSyncSetup();

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

this just looks like a sub-case of Step as implemented already... what am I
missing?

[qsr (NOT THE RIGHT qsr), 2011/07/08 08:51:43]

 This is calling ShowSyncSetup method on flow, that will ultimately call:
 
BrowserList::GetLastActiveWithProfile(profile)->ShowOptionsTab(chrome::kSyncSetupSubPage);

 Step itself is never doing it, and the SyncService can call either of the 2
methods depending on the context.

 I'm not sure to understand what you propose. The aim of this patch is to move
all UI related code to the implementation of the flow handler.

+}
+
 SyncSetupFlow* SyncSetupWizard::AttachSyncSetupHandler(
     SyncSetupFlowHandler* handler) {
   SyncSetupFlow* flow = flow_container_->get_flow();