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

Unified Diff: chromeos/dbus/dbus_thread_manager.cc

Issue 472953002: Merged MixedDBusThreadManager functionality into DBusClientBundle. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 6 years, 4 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
« no previous file with comments | « chromeos/dbus/dbus_client_bundle.cc ('k') | chromeos/dbus/mixed_dbus_thread_manager.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chromeos/dbus/dbus_thread_manager.cc
diff --git a/chromeos/dbus/dbus_thread_manager.cc b/chromeos/dbus/dbus_thread_manager.cc
index 08044b591f87b8c6b82577c3d37e58d50446714b..f2edfad992480b24ee8119dc329dc89d5be87192 100644
--- a/chromeos/dbus/dbus_thread_manager.cc
+++ b/chromeos/dbus/dbus_thread_manager.cc
@@ -29,7 +29,6 @@
#include "chromeos/dbus/image_burner_client.h"
#include "chromeos/dbus/introspectable_client.h"
#include "chromeos/dbus/lorgnette_manager_client.h"
-#include "chromeos/dbus/mixed_dbus_thread_manager.h"
#include "chromeos/dbus/modem_messaging_client.h"
#include "chromeos/dbus/nfc_adapter_client.h"
#include "chromeos/dbus/nfc_device_client.h"
@@ -94,6 +93,10 @@ class DBusThreadManagerImpl : public DBusThreadManager {
dbus_thread_->Stop();
}
+ void SetupDefaultEnvironment() {
+ return client_bundle_->SetupDefaultEnvironment();
+ }
+
virtual dbus::Bus* GetSystemBus() OVERRIDE {
return system_bus_.get();
}
@@ -252,8 +255,7 @@ class DBusThreadManagerImpl : public DBusThreadManager {
client_bundle_.reset(new DBusClientBundle());
// TODO(crbug.com/345586): Move PowerPolicyController out of
// DBusThreadManagerImpl.
- if (!DBusThreadManager::IsUsingStub(DBusClientBundle::POWER_MANAGER))
- power_policy_controller_.reset(new PowerPolicyController);
+ power_policy_controller_.reset(new PowerPolicyController);
}
scoped_ptr<base::Thread> dbus_thread_;
@@ -338,14 +340,11 @@ void DBusThreadManager::InitializeWithPartialStub(
<< " cannot be parsed: "
<< unstub_clients;
}
- DBusThreadManager* real_thread_manager = new DBusThreadManagerImpl();
- FakeDBusThreadManager* fake_dbus_thread_manager = new FakeDBusThreadManager;
- fake_dbus_thread_manager->SetFakeClients();
+ DBusThreadManagerImpl* dbus_thread_manager = new DBusThreadManagerImpl();
VLOG(1) << "DBusThreadManager initialized for mixed runtime environment";
- g_dbus_thread_manager = new MixedDBusThreadManager(real_thread_manager,
- fake_dbus_thread_manager);
+ g_dbus_thread_manager = dbus_thread_manager;
InitializeClients();
- fake_dbus_thread_manager->SetupDefaultEnvironment();
+ dbus_thread_manager->SetupDefaultEnvironment();
}
// static
« no previous file with comments | « chromeos/dbus/dbus_client_bundle.cc ('k') | chromeos/dbus/mixed_dbus_thread_manager.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698