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

Unified Diff: chrome/browser/sync/sync_ui_util.cc

Issue 12502017: signin: pull basic SigninManager functionality into new SigninManagerBase class. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: deal with new enterprise_platform_keys_private_api Created 7 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
« no previous file with comments | « chrome/browser/sync/sync_ui_util.h ('k') | chrome/browser/sync/sync_ui_util_unittest.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/sync/sync_ui_util.cc
diff --git a/chrome/browser/sync/sync_ui_util.cc b/chrome/browser/sync/sync_ui_util.cc
index 4e95d331db196579bbf8b1aa6a7693708cf7e976..a8cd618e63474294d89b65926ded4a262712aacf 100644
--- a/chrome/browser/sync/sync_ui_util.cc
+++ b/chrome/browser/sync/sync_ui_util.cc
@@ -12,7 +12,7 @@
#include "chrome/browser/profiles/profile.h"
#include "chrome/browser/profiles/profile_manager.h"
#include "chrome/browser/signin/signin_global_error.h"
-#include "chrome/browser/signin/signin_manager.h"
+#include "chrome/browser/signin/signin_manager_base.h"
#include "chrome/browser/signin/signin_ui_util.h"
#include "chrome/browser/sync/profile_sync_service.h"
#include "chrome/browser/sync/profile_sync_service_factory.h"
@@ -45,7 +45,7 @@ namespace {
// and can connect to the sync server. If the user hasn't yet authenticated, an
// empty string is returned.
string16 GetSyncedStateStatusLabel(ProfileSyncService* service,
- const SigninManager& signin,
+ const SigninManagerBase& signin,
StatusLabelStyle style) {
string16 user_name = UTF8ToUTF16(signin.GetAuthenticatedUsername());
@@ -115,7 +115,7 @@ void GetStatusForActionableError(
// status_label and link_label must either be both NULL or both non-NULL.
MessageType GetStatusInfo(ProfileSyncService* service,
- const SigninManager& signin,
+ const SigninManagerBase& signin,
StatusLabelStyle style,
string16* status_label,
string16* link_label) {
@@ -253,7 +253,7 @@ MessageType GetStatusInfo(ProfileSyncService* service,
// Returns the status info for use on the new tab page, where we want slightly
// different information than in the settings panel.
MessageType GetStatusInfoForNewTabPage(ProfileSyncService* service,
- const SigninManager& signin,
+ const SigninManagerBase& signin,
string16* status_label,
string16* link_label) {
DCHECK(status_label);
@@ -291,7 +291,7 @@ MessageType GetStatusInfoForNewTabPage(ProfileSyncService* service,
} // namespace
MessageType GetStatusLabels(ProfileSyncService* service,
- const SigninManager& signin,
+ const SigninManagerBase& signin,
StatusLabelStyle style,
string16* status_label,
string16* link_label) {
@@ -302,7 +302,7 @@ MessageType GetStatusLabels(ProfileSyncService* service,
}
MessageType GetStatusLabelsForNewTabPage(ProfileSyncService* service,
- const SigninManager& signin,
+ const SigninManagerBase& signin,
string16* status_label,
string16* link_label) {
DCHECK(status_label);
@@ -312,7 +312,7 @@ MessageType GetStatusLabelsForNewTabPage(ProfileSyncService* service,
}
void GetStatusLabelsForSyncGlobalError(ProfileSyncService* service,
- const SigninManager& signin,
+ const SigninManagerBase& signin,
string16* menu_label,
string16* bubble_message,
string16* bubble_accept_label) {
@@ -343,7 +343,7 @@ void GetStatusLabelsForSyncGlobalError(ProfileSyncService* service,
}
MessageType GetStatus(
- ProfileSyncService* service, const SigninManager& signin) {
+ ProfileSyncService* service, const SigninManagerBase& signin) {
return sync_ui_util::GetStatusInfo(service, signin, WITH_HTML, NULL, NULL);
}
« no previous file with comments | « chrome/browser/sync/sync_ui_util.h ('k') | chrome/browser/sync/sync_ui_util_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698