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

Unified Diff: chrome/browser/managed_mode/managed_user_registration_utility.h

Issue 23691045: Update managed user registration to allow updating the avatar (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: ... 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/managed_mode/managed_user_registration_utility.h
diff --git a/chrome/browser/managed_mode/managed_user_registration_utility.h b/chrome/browser/managed_mode/managed_user_registration_utility.h
index 5a87a35fdc4161bcd3c7d521ba8bb9080f7224f2..229aea81240a45a0caef92ad82d5711dd7e92f10 100644
--- a/chrome/browser/managed_mode/managed_user_registration_utility.h
+++ b/chrome/browser/managed_mode/managed_user_registration_utility.h
@@ -61,11 +61,15 @@ class ManagedUserRegistrationUtility {
// Registers a new managed user with the server. |managed_user_id| is a new
// unique ID for the new managed user. If its value is the same as that of
// of one of the existing managed users, then the same user will be created
- // on this machine. |info| contains necessary information like the display
- // name of the the user. |callback| is called with the result of the
- // registration. We use the info here and not the profile, because on
- // Chrome OS the profile of the managed user does not yet exist.
+ // on this machine. |update_avatar| can only be true when a managed user
+ // is being imported, in this case the managed user's avatar will also
+ // be updated on the sync server. |info| contains necessary information like
+ // the display name of the user and his avatar. |callback| is called
+ // with the result of the registration. We use the info here and not the
+ // profile, because on Chrome OS the profile of the managed user does not
+ // yet exist.
virtual void Register(const std::string& managed_user_id,
+ bool update_avatar,
const ManagedUserRegistrationInfo& info,
const RegistrationCallback& callback) = 0;

Powered by Google App Engine
This is Rietveld 408576698