Expanding signin histogram coverage.

chrome/browser/ui/sync/one_click_signin_helper.h

Index: chrome/browser/ui/sync/one_click_signin_helper.h
diff --git a/chrome/browser/ui/sync/one_click_signin_helper.h b/chrome/browser/ui/sync/one_click_signin_helper.h
index fab944358cab36bdf1c0723ea02f8d19bafc270d..894f6e901979c855294acf95ec60b33910c578b1 100644
--- a/chrome/browser/ui/sync/one_click_signin_helper.h
+++ b/chrome/browser/ui/sync/one_click_signin_helper.h
@@ -195,12 +195,20 @@ class OneClickSigninHelper
   // ProfileSyncServiceObserver.
   virtual void OnStateChanged() OVERRIDE;
 
+  // Tracks if we are in the process of showing the signin or one click
+  // interstitial page. It's set to true the first time we load one of those
+  // pages and set to false when transient state is cleaned.
+  bool showing_signin_;
+
   // 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_;
+  bool switched_to_advanced_;
+  // When switching to advanced settings, we want to track the original source.
+  SyncPromoUI::Source advanced_source_;

[Roger Tawa OOO till Jul 10th, 2013/02/28 20:23:54]

Maybe call this member original_source_ ?  Also, it should be initialized in the
ctor.

[jwd, 2013/02/28 20:45:38]

Done.

   GURL continue_url_;
   // Redirect URL after sync setup is complete.
   GURL redirect_url_;