Clean up LoginUIService. Remove Singleton behavior from PeopleHandler

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

 // Copyright 2015 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_WEBUI_SETTINGS_PEOPLE_HANDLER_H_
 #define CHROME_BROWSER_UI_WEBUI_SETTINGS_PEOPLE_HANDLER_H_
 
 #include <memory>
 
 #include "base/gtest_prod_util.h"
@@ skipping 85 matching lines @@
   void RegisterMessages() override;
   void OnJavascriptAllowed() override;
   void OnJavascriptDisallowed() override;
 
   // SyncStartupTracker::Observer implementation.
   void SyncStartupCompleted() override;
   void SyncStartupFailed() override;
 
   // LoginUIService::LoginUI implementation.
   void FocusUI() override;
-  void CloseUI() override;
 
   // SigninManagerBase::Observer implementation.
   void GoogleSigninSucceeded(const std::string& account_id,
                              const std::string& username,
                              const std::string& password) override;
   void GoogleSignedOut(const std::string& account_id,
                        const std::string& username) override;
 
   // sync_driver::SyncServiceObserver implementation.
   void OnStateChanged() override;
@@ skipping 24 matching lines @@
 #if !defined(OS_CHROMEOS)
   // Displays the GAIA login form.
   void DisplayGaiaLogin(signin_metrics::AccessPoint access_point);
 
   // When web-flow is enabled, displays the Gaia login form in a new tab.
   // This function is virtual so that tests can override.
   virtual void DisplayGaiaLoginInNewTabOrWindow(
       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
   // is running in the background.
   void DisplaySpinner();
 
   // 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();
 
   // Sends the current sync status to the JavaScript WebUI code.
   void UpdateSyncStatus();
 
   // Suppresses any further signin promos, since the user has signed in once.
   void MarkFirstSetupComplete();
 
@@ skipping 18 matching lines @@
   // Manages observer lifetimes.
   ScopedObserver<SigninManagerBase, PeopleHandler> signin_observer_;
   ScopedObserver<ProfileSyncService, PeopleHandler> sync_service_observer_;
 
   DISALLOW_COPY_AND_ASSIGN(PeopleHandler);
 };
 
 }  // namespace settings
 
 #endif  // CHROME_BROWSER_UI_WEBUI_SETTINGS_PEOPLE_HANDLER_H_