ChromeOS: Fix removeUser button on signin screen.

chrome/browser/chromeos/login/signin_screen_controller.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_CHROMEOS_LOGIN_SIGNIN_SCREEN_CONTROLLER_H_
 #define CHROME_BROWSER_CHROMEOS_LOGIN_SIGNIN_SCREEN_CONTROLLER_H_
 
 #include "chrome/browser/chromeos/login/screens/gaia_screen.h"
 #include "chrome/browser/chromeos/login/screens/user_selection_screen.h"
 #include "chrome/browser/chromeos/login/ui/login_display_host.h"
 #include "chrome/browser/chromeos/login/ui/oobe_display.h"
 #include "components/user_manager/remove_user_delegate.h"
 #include "components/user_manager/user.h"
 #include "content/public/browser/notification_observer.h"
 #include "content/public/browser/notification_registrar.h"
 
+class AccountId;
+
 namespace chromeos {
 
 class LoginDisplayWebUIHandler;
 
 // Class that manages control flow between wizard screens. Wizard controller
 // interacts with screen controllers to move the user between screens.
 class SignInScreenController : public user_manager::RemoveUserDelegate,
                                public content::NotificationObserver {
  public:
   SignInScreenController(OobeDisplay* oobe_display,
@@ skipping 13 matching lines @@
   void OnSigninScreenReady();
 
   // Query to send list of users to user selection screen.
   void SendUserList();
 
   // Runs OAauth token validity check.
   void CheckUserStatus(const AccountId& account_id);
 
   // Query to remove user with specified id.
   // TODO(antrim): move to user selection screen handler.
-  void RemoveUser(const std::string& user_id);
+  void RemoveUser(const AccountId& account_id);
 
   // user_manager::RemoveUserDelegate implementation:

[achuithb, 2015/11/23 21:17:39]

could you make everything below here private while you're here?

[Alexander Alekseev, 2015/11/24 06:04:11]

Hm. Do you think we should make overridden public methods private?

[Alexander Alekseev, 2015/11/24 06:58:00]

Done.

-  void OnBeforeUserRemoved(const std::string& username) override;
-  void OnUserRemoved(const std::string& username) override;
+  void OnBeforeUserRemoved(const AccountId& account_id) override;
+  void OnUserRemoved(const AccountId& account_id) override;
 
   // content::NotificationObserver implementation.
   void Observe(int type,
                const content::NotificationSource& source,
                const content::NotificationDetails& details) override;
 
  private:
   static SignInScreenController* instance_;
 
   OobeDisplay* oobe_display_;
 
   // Reference to the WebUI handling layer for the login screen
   LoginDisplayWebUIHandler* webui_handler_;
 
   scoped_ptr<GaiaScreen> gaia_screen_;
   scoped_ptr<UserSelectionScreen> user_selection_screen_;
 
   // Used for notifications during the login process.
   content::NotificationRegistrar registrar_;
 
   DISALLOW_COPY_AND_ASSIGN(SignInScreenController);
 };
 
 }  // namespace chromeos
 
 #endif  // CHROME_BROWSER_CHROMEOS_LOGIN_SIGNIN_SCREEN_CONTROLLER_H_