Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(6307)

Unified Diff: chrome/browser/ui/webui/sync_setup_handler.h

Issue 10192005: Refactor LoginUIService to not rely on WebUI. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Review feedback. Created 8 years, 8 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/ui/webui/sync_setup_handler.h
diff --git a/chrome/browser/ui/webui/sync_setup_handler.h b/chrome/browser/ui/webui/sync_setup_handler.h
index a608b1188615e1c0e57e2f4a5d157bdd3a822558..148ccccb23ea753c8545e816172429f90f25e299 100644
--- a/chrome/browser/ui/webui/sync_setup_handler.h
+++ b/chrome/browser/ui/webui/sync_setup_handler.h
@@ -9,6 +9,7 @@
#include "base/memory/scoped_ptr.h"
#include "chrome/browser/signin/signin_tracker.h"
#include "chrome/browser/ui/webui/options2/options_ui2.h"
+#include "chrome/browser/ui/webui/signin/login_ui_service.h"
class LoginUIService;
class ProfileManager;
@@ -16,7 +17,8 @@ class ProfileSyncService;
class SigninManager;
class SyncSetupHandler : public options2::OptionsPageUIHandler,
- public SigninTracker::Observer {
+ public SigninTracker::Observer,
+ public LoginUIService::LoginUI {
public:
// Constructs a new SyncSetupHandler. |profile_manager| may be NULL.
explicit SyncSetupHandler(ProfileManager* profile_manager);
@@ -27,11 +29,15 @@ class SyncSetupHandler : public options2::OptionsPageUIHandler,
OVERRIDE;
virtual void RegisterMessages() OVERRIDE;
- // SigninTracker::Observer implementation
+ // SigninTracker::Observer implementation.
virtual void GaiaCredentialsValid() OVERRIDE;
virtual void SigninFailed(const GoogleServiceAuthError& error) OVERRIDE;
virtual void SigninSuccess() OVERRIDE;
+ // LoginUIService::LoginUI implementation.
+ virtual void FocusUI() OVERRIDE;
+ virtual void CloseUI() OVERRIDE;
+
static void GetStaticLocalizedValues(
base::DictionaryValue* localized_strings,
content::WebUI* web_ui);
« no previous file with comments | « chrome/browser/ui/webui/sync_promo/sync_promo_handler.cc ('k') | chrome/browser/ui/webui/sync_setup_handler.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698