Enable "Let me choose what to sync" checkbox in gaia sign in page.

chrome/browser/ui/sync/one_click_signin_helper.h

 // 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.
 
 #ifndef CHROME_BROWSER_UI_SYNC_ONE_CLICK_SIGNIN_HELPER_H_
 #define CHROME_BROWSER_UI_SYNC_ONE_CLICK_SIGNIN_HELPER_H_
 
 #include <string>
 
 #include "base/gtest_prod_util.h"
@@ skipping 17 matching lines @@
 // Per-tab one-click signin helper.  When a user signs in to a Google service
 // and the profile is not yet connected to a Google account, will start the
 // process of helping the user connect his profile with one click.  The process
 // begins with an infobar and is followed with a confirmation dialog explaining
 // more about what this means.
 class OneClickSigninHelper
     : public content::WebContentsObserver,
       public content::WebContentsUserData<OneClickSigninHelper>,
       public SigninTracker::Observer {
  public:
+  // Represents user's decision about sign in process.
   enum AutoAccept {
-    AUTO_ACCEPT,
-    NO_AUTO_ACCEPT,
+    // User decision not yet known.  Assume cancel.
+    AUTO_ACCEPT_NONE,
+
+    // User has explicitly accepted to sign in.  A bubble is shown with the
+    // option to start sync, configure it first, or abort.
+    AUTO_ACCEPT_ACCEPTED,
+
+    // User has explicitly accepted to sign in, but wants to configure sync
+    // settings before turing it on.
     AUTO_ACCEPT_CONFIGURE,
-    REJECTED_FOR_PROFILE,
+
+    // User has explicitly rejected to sign in.  Furthermore, the user does
+    // not want to be prompted to see the interstitial again in this profile.
+    AUTO_ACCEPT_REJECTED_FOR_PROFILE,
+
+    // This is an explicit sign in from either first run, NTP, wrench menu,
+    // or settings page.  The user will be signed in automatically with sync
+    // enabled using default settings.
     AUTO_ACCEPT_EXPLICIT
   };
 
   // Return value of CanOfferOnIOThread().
   enum Offer {
     CAN_OFFER,
     DONT_OFFER,
     IGNORE_REQUEST
   };
 
   // Argument to CanOffer().
   enum CanOfferFor {
     CAN_OFFER_FOR_ALL,
-    CAN_OFFER_FOR_INTERSTITAL_ONLY,

[tim (not reviewing), 2012/12/14 19:48:29]

Did you intend to get rid of this comma?

[Roger Tawa OOO till Jul 10th, 2012/12/16 03:03:46]

yes.

+    CAN_OFFER_FOR_INTERSTITAL_ONLY
   };
 
   virtual ~OneClickSigninHelper();
 
   // Called only by tests to associate information with a given request.
   static void AssociateWithRequestForTesting(base::SupportsUserData* request,
                                              const std::string& email);
 
   // Returns true if the one-click signin feature can be offered at this time.
   // If |email| is not empty, then the profile is checked to see if it's
@@ skipping 62 matching lines @@
 
   // Returns true if the one-click signin feature can be offered at this time.
   // It can be offered if the io_data is not in an incognito window and if the
   // origin of |url| is a valid Gaia sign in origin.  This function is meant
   // to called only from the IO thread.
   static Offer CanOfferOnIOThreadImpl(const GURL& url,
                                       const std::string& referrer,
                                       base::SupportsUserData* request,
                                       ProfileIOData* io_data);
 
   // The portion of ShowInfoBarIfPossible() that needs to run on the UI thread.

[tim (not reviewing), 2012/12/14 19:48:29]

Can you explain the parameters?  As in, a high level comment, when will
continue_url be used, etc.

[Roger Tawa OOO till Jul 10th, 2012/12/16 03:03:46]

Done.

   static void ShowInfoBarUIThread(const std::string& session_index,
                                   const std::string& email,
                                   AutoAccept auto_accept,
                                   SyncPromoUI::Source source,
+                                  const GURL& continue_url,
                                   int child_id,
                                   int route_id);
 
   void RedirectToNTP();
 
   // Clear all data member of the helper, except for the error.
   void CleanTransientState();
 
   // content::WebContentsObserver overrides.
   virtual void DidNavigateAnyFrame(
       const content::LoadCommittedDetails& details,
       const content::FrameNavigateParams& params) OVERRIDE;
   virtual void DidStopLoading(
       content::RenderViewHost* render_view_host) OVERRIDE;
 
   // SigninTracker::Observer override.
   virtual void GaiaCredentialsValid() OVERRIDE;
   virtual void SigninFailed(const GoogleServiceAuthError& error) OVERRIDE;
   virtual void SigninSuccess() OVERRIDE;
 
   // Information about the account that has just logged in.
   std::string session_index_;
   std::string email_;
   std::string password_;
   AutoAccept auto_accept_;
   SyncPromoUI::Source source_;
+  GURL continue_url_;
   std::string error_message_;
 
   scoped_ptr<SigninTracker> signin_tracker_;
 
   DISALLOW_COPY_AND_ASSIGN(OneClickSigninHelper);
 };
 
 #endif  // CHROME_BROWSER_UI_SYNC_ONE_CLICK_SIGNIN_HELPER_H_