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

Unified Diff: chrome/browser/chromeos/login/user.h

Issue 23095006: If user profile doesn't contain language setting, default to his Google account settings. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Added comment on "list of preferred languages". Created 7 years, 3 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/chromeos/login/user.h
diff --git a/chrome/browser/chromeos/login/user.h b/chrome/browser/chromeos/login/user.h
index 35911bbdc69de905836cd8b87d1d97066d8a3677..54db9371265d3ddc53cefa9d531bc138903e3a75 100644
--- a/chrome/browser/chromeos/login/user.h
+++ b/chrome/browser/chromeos/login/user.h
@@ -176,7 +176,18 @@ class User {
explicit User(const std::string& email);
virtual ~User();
+ // Used only by UserManagerImpl
Dmitry Polukhin 2013/09/06 20:30:05 Do we need such comment? It will be outdated very
Alexander Alekseev 2013/09/11 18:53:02 Done.
+ bool is_profile_created() const {
+ return profile_is_created_;
+ }
+
+ const std::string* GetAccountLocale() const {
+ return account_locale_.get();
+ }
+
// Setters are private so only UserManager can call them.
+ void SetAccountLocale(const std::string& raw_account_locale);
+
void SetImage(const UserImage& user_image, int image_index);
void SetImageURL(const GURL& image_url);
@@ -212,6 +223,13 @@ class User {
is_active_ = is_active;
}
+ void set_profile_is_created() {
+ profile_is_created_ = true;
+ }
+
+ // True if user has google account (not a guest or managed user).
+ bool has_account() const;
Dmitry Polukhin 2013/09/06 20:30:05 I would call it has_gaia_account to make it clear
Alexander Alekseev 2013/09/11 18:53:02 Done.
+
private:
std::string email_;
string16 display_name_;
@@ -220,6 +238,12 @@ class User {
UserImage user_image_;
OAuthTokenStatus oauth_token_status_;
+ // This is set to chromeos locale if account data has been downloaded.
+ // (Or failed to download, but at least one download attempt finished).
+ // An empty string indicates error in data load, or in
+ // translation of Account locale to chromeos locale.
+ scoped_ptr<std::string> account_locale_;
+
// Used to identify homedir mount point.
std::string username_hash_;
@@ -239,6 +263,9 @@ class User {
// True if user is currently logged in and active in current session.
bool is_active_;
+ // True if user Profile is created
+ bool profile_is_created_;
+
DISALLOW_COPY_AND_ASSIGN(User);
};

Powered by Google App Engine
This is Rietveld 408576698