Clean up LoginUIService. Remove Singleton behavior from PeopleHandler

chrome/browser/ui/webui/settings/people_handler.h

Index: chrome/browser/ui/webui/settings/people_handler.h
diff --git a/chrome/browser/ui/webui/settings/people_handler.h b/chrome/browser/ui/webui/settings/people_handler.h
index 4a98df8a598fc3ba6334f5dc12870158f381783e..3132759b49f93aef525e4eece2c66291cde2aca8 100644
--- a/chrome/browser/ui/webui/settings/people_handler.h
+++ b/chrome/browser/ui/webui/settings/people_handler.h
@@ -103,7 +103,6 @@ class PeopleHandler : public SettingsPageUIHandler,
 
   // LoginUIService::LoginUI implementation.
   void FocusUI() override;
-  void CloseUI() override;
 
   // SigninManagerBase::Observer implementation.
   void GoogleSigninSucceeded(const std::string& account_id,
@@ -148,10 +147,6 @@ class PeopleHandler : public SettingsPageUIHandler,
       signin_metrics::AccessPoint access_point);
 #endif
 
-  // A utility function to call before actually showing setup dialog. Makes sure
-  // that a new dialog can be shown and sets flag that setup is in progress.
-  bool PrepareSyncSetup();
-
   // Displays spinner-only UI indicating that something is going on in the
   // background.
   // TODO(kochi): better to show some message that the user can understand what
@@ -161,14 +156,8 @@ class PeopleHandler : public SettingsPageUIHandler,
   // Displays an error dialog which shows timeout of starting the sync backend.
   void DisplayTimeout();
 
-  // Returns true if this object is the active login object.
-  bool IsActiveLogin() const;
-
-  // If a wizard already exists, return true. Otherwise, return false.
-  bool IsExistingWizardPresent();
-
-  // If a wizard already exists, focus it and return true.
-  bool FocusExistingWizardIfPresent();
+  // Closes the associated sync settings page.
+  void CloseUI();
 
   // Pushes the updated sync prefs to JavaScript.
   void PushSyncPrefs();