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

Unified Diff: chrome/browser/chromeos/dbus/dbus_thread_manager.cc

Issue 8289021: chromeos: Change all clients of LoginLibrary to use SessionManagerClient. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: renamed Created 9 years, 2 months 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
Index: chrome/browser/chromeos/dbus/dbus_thread_manager.cc
diff --git a/chrome/browser/chromeos/dbus/dbus_thread_manager.cc b/chrome/browser/chromeos/dbus/dbus_thread_manager.cc
index 74c9f8ea2b1c7eaa87e0e52f7e52cfb614ba0b64..4087273acab4f9d38fdabc92ab426e2fb3eed975 100644
--- a/chrome/browser/chromeos/dbus/dbus_thread_manager.cc
+++ b/chrome/browser/chromeos/dbus/dbus_thread_manager.cc
@@ -7,6 +7,7 @@
#include "base/command_line.h"
#include "base/threading/thread.h"
#include "chrome/browser/chromeos/dbus/cros_dbus_service.h"
+#include "chrome/browser/chromeos/dbus/session_manager_client.h"
#include "chrome/browser/chromeos/dbus/power_manager_client.h"
#include "chrome/browser/chromeos/dbus/sensors_source.h"
#include "chrome/common/chrome_switches.h"
@@ -44,6 +45,9 @@ DBusThreadManager::DBusThreadManager() {
// Create the power manager client.
power_manager_client_.reset(PowerManagerClient::Create(system_bus_.get()));
+ // Create the session manager client.
+ session_manager_client_.reset(
+ SessionManagerClient::Create(system_bus_.get()));
}
DBusThreadManager::~DBusThreadManager() {
@@ -60,7 +64,11 @@ DBusThreadManager::~DBusThreadManager() {
}
void DBusThreadManager::Initialize() {
- CHECK(!g_dbus_thread_manager);
+ if (g_dbus_thread_manager) {
+ // This can happen in tests.
+ LOG(WARNING) << "DBusThreadManager::Initialize() was already called";
+ return;
+ }
g_dbus_thread_manager = new DBusThreadManager;
VLOG(1) << "DBusThreadManager initialized";
}
@@ -82,4 +90,9 @@ DBusThreadManager* DBusThreadManager::Get() {
return g_dbus_thread_manager;
}
+void DBusThreadManager::set_session_manager_client_for_testing(
+ SessionManagerClient* session_manager_client) {
+ session_manager_client_.reset(session_manager_client);
+}
+
} // namespace chromeos
« no previous file with comments | « chrome/browser/chromeos/dbus/dbus_thread_manager.h ('k') | chrome/browser/chromeos/enterprise_extension_observer.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698