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

Unified Diff: ash/mus/window_manager_application.cc

Issue 2319783002: mash: Allow a subset of D-Bus clients to be created in DBusThreadManager (Closed)
Patch Set: cleanup Created 4 years, 3 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 | « no previous file | ash/test/ash_test_helper.cc » ('j') | chromeos/dbus/dbus_client_bundle.h » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: ash/mus/window_manager_application.cc
diff --git a/ash/mus/window_manager_application.cc b/ash/mus/window_manager_application.cc
index eae07e130d2e6929fe5446d29d9e953236268399..bd8cfed75ceab470c9be9391a597dc73e80e88f8 100644
--- a/ash/mus/window_manager_application.cc
+++ b/ash/mus/window_manager_application.cc
@@ -30,6 +30,7 @@
#if defined(OS_CHROMEOS)
#include "ash/common/system/chromeos/power/power_status.h"
#include "chromeos/audio/cras_audio_handler.h"
+#include "chromeos/dbus/dbus_client_bundle.h"
#include "chromeos/dbus/dbus_thread_manager.h"
#include "chromeos/system/fake_statistics_provider.h"
#include "device/bluetooth/dbus/bluez_dbus_manager.h" // nogncheck
@@ -42,14 +43,23 @@ namespace {
void InitializeComponents() {
message_center::MessageCenter::Initialize();
#if defined(OS_CHROMEOS)
+ // TODO(crbug.com/629707): Use mojo API for bluetooth and shill.
+ const chromeos::DBusClientTypeMask clients =
+ chromeos::DBusClientBundle::BLUETOOTH | chromeos::DBusClientBundle::CRAS |
+ chromeos::DBusClientBundle::GSM_SMS |
+ chromeos::DBusClientBundle::POWER_MANAGER |
+ chromeos::DBusClientBundle::SESSION_MANAGER |
+ chromeos::DBusClientBundle::SHILL | chromeos::DBusClientBundle::SMS |
+ chromeos::DBusClientBundle::SYSTEM_CLOCK |
+ chromeos::DBusClientBundle::UPDATE_ENGINE;
// Must occur after mojo::ApplicationRunner has initialized AtExitManager, but
// before WindowManager::Init().
- chromeos::DBusThreadManager::Initialize();
+ chromeos::DBusThreadManager::Initialize(clients);
// See ChromeBrowserMainPartsChromeos for ordering details.
bluez::BluezDBusManager::Initialize(
chromeos::DBusThreadManager::Get()->GetSystemBus(),
- chromeos::DBusThreadManager::Get()->IsUsingStub(
+ chromeos::DBusThreadManager::Get()->IsUsingFake(
chromeos::DBusClientBundle::BLUETOOTH));
// TODO(jamescook): Initialize real audio handler.
chromeos::CrasAudioHandler::InitializeForTesting();
« no previous file with comments | « no previous file | ash/test/ash_test_helper.cc » ('j') | chromeos/dbus/dbus_client_bundle.h » ('J')

Powered by Google App Engine
This is Rietveld 408576698