Remove browser::FindLastActiveWithProfile call in login_ui_service.cc by plumbing the Browser* thro…

chrome/browser/ui/webui/signin/login_ui_service.h

Index: chrome/browser/ui/webui/signin/login_ui_service.h
===================================================================
--- chrome/browser/ui/webui/signin/login_ui_service.h	(revision 144404)
+++ chrome/browser/ui/webui/signin/login_ui_service.h	(working copy)
@@ -9,7 +9,7 @@
 #include "base/basictypes.h"
 #include "chrome/browser/profiles/profile_keyed_service.h"
 
-class Profile;
+class Browser;
 
 // The LoginUIService helps track per-profile information for the login UI -
 // for example, whether there is login UI currently on-screen.
@@ -29,10 +29,8 @@
     virtual ~LoginUI() {}
   };
 
-  // Creates a LoginUIService associated with the passed |profile|. |profile|
-  // is used to create a new browser in the case that ShowLoginUI() is invoked
-  // when no browser windows are open (e.g. via the Mac menu bar).

[jam, 2012/06/27 21:43:27]

this comment wasn't right, i.e. Mac menu bar goes through a different code path
when there's no browser. verified on mac.

http://code.google.com/searchframe#OAMlx_jo-ck/src/chrome/browser/app_control...

-  explicit LoginUIService(Profile* profile);
+  // Creates a LoginUIService

[Andrew T Wilson (Slow), 2012/06/27 22:32:36]

Remove this comment since it adds nothing?

[jam, 2012/06/27 22:39:30]

Done.

+  LoginUIService();
   virtual ~LoginUIService();
 
   // Gets the currently active login UI, or null if no login UI is active.
@@ -51,17 +49,13 @@
   // Brings the login UI to the foreground, or if there is no login UI,
   // navigates to the login UI page in a new browser tab.

[Andrew T Wilson (Slow), 2012/06/27 22:32:36]

Update this comment, since it sounds like we use the passed browser?

[jam, 2012/06/27 22:39:30]

Done.

   // Virtual for mocking purposes.
-  virtual void ShowLoginUI();
+  virtual void ShowLoginUI(Browser* browser);
 
  private:
   // Weak pointer to the currently active login UI, or null if none.
   LoginUI* ui_;
 
-  // Weak pointer to the profile this service is associated with.
-  Profile* profile_;
-
   DISALLOW_COPY_AND_ASSIGN(LoginUIService);
 };
 
-
 #endif  // CHROME_BROWSER_UI_WEBUI_SIGNIN_LOGIN_UI_SERVICE_H_