Setup from settings should allow configuration first

chrome/browser/ui/sync/one_click_signin_sync_starter.h

Index: chrome/browser/ui/sync/one_click_signin_sync_starter.h
diff --git a/chrome/browser/ui/sync/one_click_signin_sync_starter.h b/chrome/browser/ui/sync/one_click_signin_sync_starter.h
index faf561d6e0ff8567a33c833d4fbf91d0ff7b8287..9eb5e8a6a691336699a446937b8c9c0895f9d65e 100644
--- a/chrome/browser/ui/sync/one_click_signin_sync_starter.h
+++ b/chrome/browser/ui/sync/one_click_signin_sync_starter.h
@@ -16,7 +16,16 @@ class Browser;
 // the job is done.
 class OneClickSigninSyncStarter : public SigninTracker::Observer {
  public:
-  enum StartSyncMode {SYNC_WITH_DEFAULT_SETTINGS, CONFIGURE_SYNC_FIRST };
+  enum StartSyncMode {
+    // Starts the process of signing the user in with the SigninManager, and
+    // once completed automatically starts sync with all data types enabled.
+    SYNC_WITH_DEFAULT_SETTINGS,
+
+    // Starts the process of signing the user in with the SigninManager, and
+    // once completed redirects the user to the settings page to allow them
+    // to configure which data types to sync before sync is enabled.
+    CONFIGURE_SYNC_FIRST
+  };
 
   OneClickSigninSyncStarter(Browser* browser,
                             const std::string& session_index,