Remove SigninManager::StartSignIn() since /ClientLogin is no longer used.

chrome/browser/signin/signin_manager.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.
 //
 // The signin manager encapsulates some functionality tracking
 // which user is signed in. See SigninManagerBase for full description of
 // responsibilities. The class defined in this file provides functionality
 // required by all platforms except Chrome OS.
 //
 // When a user is signed in, a ClientLogin request is run on their behalf.
@@ skipping 57 matching lines @@
   // OneClickSigninHelper.
   static const char* kChromeSigninEffectiveSite;
 
   explicit SigninManager(scoped_ptr<SigninManagerDelegate> delegate);
   virtual ~SigninManager();
 
   // Returns true if the username is allowed based on the policy string.
   static bool IsUsernameAllowedByPolicy(const std::string& username,
                                         const std::string& policy);
 
-  // Attempt to sign in this user with ClientLogin. If successful, set a
-  // preference indicating the signed in user and send out a notification,
-  // then start fetching tokens for the user.
-  // This is overridden for test subclasses that don't want to issue auth
-  // requests.
-  virtual void StartSignIn(const std::string& username,
-                           const std::string& password,
-                           const std::string& login_token,
-                           const std::string& login_captcha);
-
-  // Used when a second factor access code was required to complete a signin
-  // attempt.
-  void ProvideSecondFactorAccessCode(const std::string& access_code);
-
   // Attempt to sign in this user with existing credentials from the cookie jar.
   // |session_index| indicates which user account to use if the cookie jar
   // contains a multi-login session. Otherwise the end result of this call is
   // the same as StartSignIn().
   // If non-null, the passed |signin_complete| callback is invoked once signin
   // has been completed and the oauth login token has been generated - the
   // callback will not be invoked if no token is generated (either because of
   // a failed signin or because web-based signin is not enabled).
   // The callback should invoke SignOut() or CompletePendingSignin() to either
   // continue or cancel the in-process signin.
@@ skipping 88 matching lines @@
   bool HasSigninProcess() const;
 
  protected:
   // If user was signed in, load tokens from DB if available.
   virtual void InitTokenService() OVERRIDE;
 
   // Flag saying whether signing out is allowed.
   bool prohibit_signout_;
 
  private:
+  // TODO(guohui): Leaving the sign in type here because a follow up CL by
+  // Hui will add a new type.  Hui: please remove this comment in your CL.
   enum SigninType {
     SIGNIN_TYPE_NONE,
-    SIGNIN_TYPE_CLIENT_LOGIN,
     SIGNIN_TYPE_WITH_CREDENTIALS,
   };
 
   std::string SigninTypeToString(SigninType type);
-
   friend class FakeSigninManager;
   FRIEND_TEST_ALL_PREFIXES(SigninManagerTest, ClearTransientSigninData);
   FRIEND_TEST_ALL_PREFIXES(SigninManagerTest, ProvideSecondFactorSuccess);
   FRIEND_TEST_ALL_PREFIXES(SigninManagerTest, ProvideSecondFactorFailure);
 
   // Called to setup the transient signin data during one of the
   // StartSigninXXX methods.  |type| indicates which of the methods is being
   // used to perform the signin while |username| and |password| identify the
   // account to be signed in. Returns false and generates an auth error if the
   // passed |username| is not allowed by policy.
@@ skipping 78 matching lines @@
 
   // Helper object to listen for changes to the signin allowed preference.
   BooleanPrefMember signin_allowed_;
 
   DISALLOW_COPY_AND_ASSIGN(SigninManager);
 };
 
 #endif  // !defined(OS_CHROMEOS)
 
 #endif  // CHROME_BROWSER_SIGNIN_SIGNIN_MANAGER_H_