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

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

Issue 8510069: Make ProfileImageDownloader available to non-chromeos code (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: fix build Created 9 years, 1 month 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/chromeos/login/user_image_loader.cc ('k') | chrome/browser/chromeos/login/user_manager.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/chromeos/login/user_manager.h
diff --git a/chrome/browser/chromeos/login/user_manager.h b/chrome/browser/chromeos/login/user_manager.h
index 4e3aa2fd370f2f34a206eea7a2e49c2963f8a824..d1137a980cbc629846e9055e3dfcdd27dbe7bb9e 100644
--- a/chrome/browser/chromeos/login/user_manager.h
+++ b/chrome/browser/chromeos/login/user_manager.h
@@ -14,15 +14,16 @@
#include "base/observer_list.h"
#include "base/synchronization/lock.h"
#include "base/time.h"
-#include "chrome/browser/chromeos/login/profile_image_downloader.h"
#include "chrome/browser/chromeos/login/user.h"
#include "chrome/browser/chromeos/login/user_image_loader.h"
+#include "chrome/browser/profiles/profile_downloader_delegate.h"
#include "content/public/browser/notification_observer.h"
#include "content/public/browser/notification_registrar.h"
#include "third_party/skia/include/core/SkBitmap.h"
class FilePath;
class PrefService;
+class ProfileDownloader;
namespace base {
template<typename> struct DefaultLazyInstanceTraits;
@@ -34,7 +35,7 @@ class RemoveUserDelegate;
// This class provides a mechanism for discovering users who have logged
// into this chromium os device before and updating that list.
-class UserManager : public ProfileImageDownloader::Delegate,
+class UserManager : public ProfileDownloaderDelegate,
public content::NotificationObserver {
public:
// Returns a shared instance of a UserManager. Not thread-safe, should only be
@@ -210,10 +211,11 @@ class UserManager : public ProfileImageDownloader::Delegate,
// Checks current user's ownership on file thread.
void CheckOwnership();
- // ProfileImageDownloader::Delegate implementation.
- virtual void OnDownloadSuccess(const SkBitmap& image) OVERRIDE;
- virtual void OnDownloadFailure() OVERRIDE;
- virtual void OnDownloadDefaultImage() OVERRIDE;
+ // ProfileDownloaderDelegate implementation.
+ virtual int GetDesiredImageSideLength() OVERRIDE;
+ virtual Profile* GetBrowserProfile() OVERRIDE;
+ virtual void OnDownloadComplete(ProfileDownloader* downloader,
+ bool success) OVERRIDE;
// Creates a new User instance.
User* CreateUser(const std::string& email) const;
@@ -259,7 +261,7 @@ class UserManager : public ProfileImageDownloader::Delegate,
ObserverList<Observer> observer_list_;
// Download user profile image on login to update it if it's changed.
- scoped_ptr<ProfileImageDownloader> profile_image_downloader_;
+ scoped_ptr<ProfileDownloader> profile_image_downloader_;
// Time when the profile image download has started.
base::Time profile_image_load_start_time_;
« no previous file with comments | « chrome/browser/chromeos/login/user_image_loader.cc ('k') | chrome/browser/chromeos/login/user_manager.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698