Refcount browser process in BaseLoginDisplayHost so message loop does not exit

chrome/browser/chromeos/login/base_login_display_host.cc

 // Copyright (c) 2011 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/chromeos/login/base_login_display_host.h"
 
 #include "base/file_util.h"
 #include "base/logging.h"
 #include "base/threading/thread_restrictions.h"
 #include "chrome/browser/browser_process.h"
@@ skipping 72 matching lines @@
 // BaseLoginDisplayHost --------------------------------------------------------
 
 BaseLoginDisplayHost::BaseLoginDisplayHost(const gfx::Rect& background_bounds)
     : background_bounds_(background_bounds) {
   registrar_.Add(
       this,
       NotificationType::APP_TERMINATING,
       NotificationService::AllSources());
   DCHECK(default_host_ == NULL);
   default_host_ = this;
+
+  // Add a reference count so the message loop won't exit when other
+  // message loop clients (e.g. menus) do.
+  g_browser_process->AddRefModule();
 }
 
 BaseLoginDisplayHost::~BaseLoginDisplayHost() {
+  // A browser should already exist when destructor is called since
+  // deletion is scheduled with MessageLoop::DeleteSoon() from
+  // OnSessionStart(), so the browser will already have incremented
+  // the reference count.
+  g_browser_process->ReleaseModule();
+
   default_host_ = NULL;
 }
 
 // LoginDisplayHost implementation ---------------------------------------------
 
 void BaseLoginDisplayHost::OnSessionStart() {
   MessageLoop::current()->DeleteSoon(FROM_HERE, this);
 }
 
 void BaseLoginDisplayHost::StartWizard(
@@ skipping 169 matching lines @@
     // user has changed to during OOBE.
     if (!timezone_name.empty()) {
       icu::TimeZone* timezone = icu::TimeZone::createTimeZone(
           icu::UnicodeString::fromUTF8(timezone_name));
       chromeos::CrosLibrary::Get()->GetSystemLibrary()->SetTimezone(timezone);
     }
   }
 }
 
 }  // namespace browser