Collect the custodian's full name when a supervised user is created.

chrome/browser/managed_mode/managed_user_registration_service.cc

 // Copyright 2013 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.
 
 #include "chrome/browser/managed_mode/managed_user_registration_service.h"
 
 #include "base/base64.h"
 #include "base/bind.h"
 #include "base/command_line.h"
 #include "base/prefs/pref_service.h"
@@ skipping 56 matching lines @@
 ManagedUserRegistrationInfo::ManagedUserRegistrationInfo(const string16& name)
     : name(name) {
 }
 
 ManagedUserRegistrationService::ManagedUserRegistrationService(
     PrefService* prefs,
     scoped_ptr<ManagedUserRefreshTokenFetcher> token_fetcher)
     : weak_ptr_factory_(this),
       prefs_(prefs),
       token_fetcher_(token_fetcher.Pass()),
-      pending_managed_user_acknowledged_(false) {
+      pending_managed_user_acknowledged_(false),
+      download_profile_(NULL) {
   pref_change_registrar_.Init(prefs);
   pref_change_registrar_.Add(
       prefs::kGoogleServicesLastUsername,
       base::Bind(&ManagedUserRegistrationService::OnLastSignedInUsernameChange,
                  base::Unretained(this)));
 }
 
 ManagedUserRegistrationService::~ManagedUserRegistrationService() {
   DCHECK(pending_managed_user_id_.empty());
   DCHECK(callback_.is_null());
@@ skipping 48 matching lines @@
     SyncError error =
         sync_processor_->ProcessSyncChanges(FROM_HERE, change_list);
     DCHECK(!error.IsSet()) << error.ToString();
   }
 
   browser_sync::DeviceInfo::CreateLocalDeviceInfo(
       base::Bind(&ManagedUserRegistrationService::FetchToken,
                  weak_ptr_factory_.GetWeakPtr(), info.name));
 }
 
+void ManagedUserRegistrationService::DownloadProfile(
+    Profile* profile,
+    const DownloadProfileCallback& callback) {
+  if (profile_downloader_)

[Bernhard Bauer, 2013/06/21 15:52:07]

Should it be a DCHECK to try to download a profile more than once (or trying to
download a different profile)?

[Pam (message me for reviews), 2013/06/21 16:19:09]

Well, since the downloader isn't guaranteed to ever return either success or
failure, it seemed like there could be one hanging around without it being a
programming error. It didn't look like just destroying a leftover was safe
either, but maybe I'm missing something there.

[Bernhard Bauer, 2013/06/21 16:46:57]

But I don't think silently returning is the correct behavior either.
Why is destroying the ProfileDownloader not safe?

[Pam (message me for reviews), 2013/06/21 19:20:25]

Fair enough. On a second look, as far as I can tell it's not a problem to dump
the old downloader.

+    return;
+
+  download_callback_ = callback;
+  download_profile_ = profile;
+  profile_downloader_.reset(new ProfileDownloader(this));
+  profile_downloader_->Start();
+}
+
 void ManagedUserRegistrationService::CancelPendingRegistration() {
   AbortPendingRegistration(
       false,  // Don't run the callback. The error will be ignored.
       GoogleServiceAuthError(GoogleServiceAuthError::NONE));
 }
 
 void ManagedUserRegistrationService::Shutdown() {
   AbortPendingRegistration(
       true,  // Run the callback.
       GoogleServiceAuthError(GoogleServiceAuthError::REQUEST_CANCELED));
@@ skipping 236 matching lines @@
             sync_processor_->ProcessSyncChanges(FROM_HERE, change_list);
         DCHECK(!sync_error.IsSet());
       }
     }
   }
 
   pending_managed_user_token_.clear();
   pending_managed_user_id_.clear();
   pending_managed_user_acknowledged_ = false;
 }
+
+bool ManagedUserRegistrationService::NeedsProfilePicture() const {
+  return false;
+}
+
+int ManagedUserRegistrationService::GetDesiredImageSideLength() const {
+  return 0;
+}
+
+std::string ManagedUserRegistrationService::GetCachedPictureURL() const {
+  return std::string();
+}
+
+Profile* ManagedUserRegistrationService::GetBrowserProfile() {
+  DCHECK(download_profile_);
+  return download_profile_;
+}
+
+void ManagedUserRegistrationService::OnProfileDownloadComplete() {
+  download_callback_.Reset();
+  download_profile_ = NULL;
+  profile_downloader_.reset();
+}
+
+void ManagedUserRegistrationService::OnProfileDownloadSuccess(
+    ProfileDownloader* downloader) {
+  download_callback_.Run(downloader->GetProfileFullName());
+  OnProfileDownloadComplete();
+}
+
+void ManagedUserRegistrationService::OnProfileDownloadFailure(
+    ProfileDownloader* downloader,
+    ProfileDownloaderDelegate::FailureReason reason) {
+  // Ignore failures; proceed without the custodian's name.
+  OnProfileDownloadComplete();
+}