Clean up straggler classes to use embedded signin in page in the new profiles world.

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 151 matching lines @@
 
     OneClickSigninHelper::StartSyncArgs args_;
     chrome::HostDesktopType desktop_type_;
     OneClickSigninSyncStarter::StartSyncMode start_mode_;
     scoped_ptr<SigninOAuthHelper> signin_oauth_helper_;
     base::WeakPtrFactory<SyncStarterWrapper> weak_pointer_factory_;
 
     DISALLOW_COPY_AND_ASSIGN(SyncStarterWrapper);
   };
 
-  static void LogHistogramValue(signin::Source source, int action);
+  static void LogHistogramValue(int action);
+
+  static void LogHistogramSourceValue(signin::Source source);
 
   // 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
   // already connected to a google account or if the user has already rejected
   // one-click sign-in with this email, in which cases a one click signin
   // should not be offered.
   //
   // If |can_offer_for| is |CAN_OFFER_FOR_INTERSTITAL_ONLY|, then only do the
   // checks that would affect the interstitial page.  Otherwise, do the checks
   // that would affect the interstitial and the explicit sign ins.
@@ skipping 194 matching lines @@
 
   // Allows unittest to avoid starting sync for real.
   bool do_not_start_sync_for_testing_;
 
   base::WeakPtrFactory<OneClickSigninHelper> weak_pointer_factory_;
 
   DISALLOW_COPY_AND_ASSIGN(OneClickSigninHelper);
 };
 
 #endif  // CHROME_BROWSER_UI_SYNC_ONE_CLICK_SIGNIN_HELPER_H_