Replace ProfileManagerObserver with callback

chrome/browser/chromeos/login/login_performer.h

Index: chrome/browser/chromeos/login/login_performer.h
diff --git a/chrome/browser/chromeos/login/login_performer.h b/chrome/browser/chromeos/login/login_performer.h
index 1372a10e7f3c339092bf9109a9e45b35ee877cb5..c67ca5ea7fa64c65750eef53b2c05e5ed8db3d7c 100644
--- a/chrome/browser/chromeos/login/login_performer.h
+++ b/chrome/browser/chromeos/login/login_performer.h
@@ -52,7 +52,7 @@ namespace chromeos {
 // 2. Pending online auth request.
 class LoginPerformer : public LoginStatusConsumer,
                        public content::NotificationObserver,
-                       public ProfileManagerObserver {
+                       public base::SupportsWeakPtr<LoginUtilsImpl> {

[Robert Sesek, 2011/12/07 16:36:13]

This class already holds a WeakPtrFactory weak_factory_. I don't think you need
this (also, this seems like the wrong template param).

[sail, 2011/12/07 17:18:13]

Done.

  public:
   // Delegate class to get notifications from the LoginPerformer.
   class Delegate : public LoginStatusConsumer {
@@ -126,8 +126,9 @@ class LoginPerformer : public LoginStatusConsumer,
                        const content::NotificationSource& source,
                        const content::NotificationDetails& details) OVERRIDE;
 
-  // ProfileManagerObserver implementation:
-  virtual void OnProfileCreated(Profile* profile, Status status) OVERRIDE;
+  // Callback for asynchronous profile creation.
+  void OnProfileCreated(Profile* profile,
+                        Profile::CreateStatus status);
 
   // Requests screen lock and subscribes to screen lock notifications.
   void RequestScreenLock();