Add a method to LoginUIService to open the sign in UI in a popup.

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

 // Copyright (c) 2012 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/ui/webui/signin/login_ui_service.h"
 
 #include "chrome/browser/profiles/profile.h"
 #include "chrome/browser/ui/chrome_pages.h"
 #include "chrome/common/url_constants.h"
 
 LoginUIService::LoginUIService() : ui_(NULL) {
 }
 
 LoginUIService::~LoginUIService() {}
 
+void LoginUIService::AddObserver(Observer* o) {
+  if (o)

[jam, 2012/07/10 16:13:07]

why do you need to handle someone calling add/remove observer with a null
pointer?

[Munjal (Google), 2012/07/10 16:46:15]

Done.

+    observer_list_.AddObserver(o);
+}
+
+void LoginUIService::RemoveObserver(Observer* o) {
+  if (o)
+    observer_list_.RemoveObserver(o);
+}
+
 void LoginUIService::SetLoginUI(LoginUI* ui) {
   DCHECK(!current_login_ui() || current_login_ui() == ui);
   ui_ = ui;
+  FOR_EACH_OBSERVER(Observer, observer_list_, OnLoginUIShown(ui_));
 }
 
 void LoginUIService::LoginUIClosed(LoginUI* ui) {
-  if (current_login_ui() == ui)
-    ui_ = NULL;
+  if (current_login_ui() != ui)
+    return;
+
+  FOR_EACH_OBSERVER(Observer, observer_list_, OnLoginUIClosed(ui_));
+  ui_ = NULL;
 }
 
 void LoginUIService::ShowLoginUI(Browser* browser) {
   if (ui_) {
     // We already have active login UI - make it visible.
     ui_->FocusUI();
     return;
   }
 
   // Need to navigate to the settings page and display the UI.
   chrome::ShowSettingsSubPage(browser, chrome::kSyncSetupSubPage);
 }