| Index: chromeos/dbus/dbus_clients_common.cc
|
| diff --git a/chromeos/dbus/dbus_clients_common.cc b/chromeos/dbus/dbus_clients_common.cc
|
| index 244185220070d0de1448471646e645f2940926b9..77b8f7ce32c2489d56193763b2877292e78caeda 100644
|
| --- a/chromeos/dbus/dbus_clients_common.cc
|
| +++ b/chromeos/dbus/dbus_clients_common.cc
|
| @@ -8,6 +8,7 @@
|
| #include "chromeos/chromeos_switches.h"
|
| #include "chromeos/dbus/cras_audio_client.h"
|
| #include "chromeos/dbus/cryptohome_client.h"
|
| +#include "chromeos/dbus/dbus_client_implementation_type.h"
|
| #include "chromeos/dbus/dbus_thread_manager.h"
|
| #include "chromeos/dbus/fake_cras_audio_client.h"
|
| #include "chromeos/dbus/fake_cryptohome_client.h"
|
| @@ -40,19 +41,18 @@
|
|
|
| namespace chromeos {
|
|
|
| -DBusClientsCommon::DBusClientsCommon(DBusClientTypeMask real_client_mask)
|
| - : real_client_mask_(real_client_mask) {
|
| - if (IsUsingReal(DBusClientType::CRAS))
|
| +DBusClientsCommon::DBusClientsCommon(bool use_real_clients) {
|
| + if (use_real_clients)
|
| cras_audio_client_.reset(CrasAudioClient::Create());
|
| else
|
| cras_audio_client_.reset(new FakeCrasAudioClient);
|
|
|
| - if (IsUsingReal(DBusClientType::CRYPTOHOME))
|
| + if (use_real_clients)
|
| cryptohome_client_.reset(CryptohomeClient::Create());
|
| else
|
| cryptohome_client_.reset(new FakeCryptohomeClient);
|
|
|
| - if (IsUsingReal(DBusClientType::SHILL)) {
|
| + if (use_real_clients) {
|
| shill_manager_client_.reset(ShillManagerClient::Create());
|
| shill_device_client_.reset(ShillDeviceClient::Create());
|
| shill_ipconfig_client_.reset(ShillIPConfigClient::Create());
|
| @@ -70,7 +70,7 @@ DBusClientsCommon::DBusClientsCommon(DBusClientTypeMask real_client_mask)
|
| new FakeShillThirdPartyVpnDriverClient);
|
| }
|
|
|
| - if (IsUsingReal(DBusClientType::GSM_SMS)) {
|
| + if (use_real_clients) {
|
| gsm_sms_client_.reset(GsmSMSClient::Create());
|
| } else {
|
| FakeGsmSMSClient* gsm_sms_client = new FakeGsmSMSClient();
|
| @@ -80,48 +80,41 @@ DBusClientsCommon::DBusClientsCommon(DBusClientTypeMask real_client_mask)
|
| gsm_sms_client_.reset(gsm_sms_client);
|
| }
|
|
|
| - if (IsUsingReal(DBusClientType::MODEM_MESSAGING))
|
| + if (use_real_clients)
|
| modem_messaging_client_.reset(ModemMessagingClient::Create());
|
| else
|
| modem_messaging_client_.reset(new FakeModemMessagingClient);
|
|
|
| - if (IsUsingReal(DBusClientType::PERMISSION_BROKER))
|
| + if (use_real_clients)
|
| permission_broker_client_.reset(PermissionBrokerClient::Create());
|
| else
|
| permission_broker_client_.reset(new FakePermissionBrokerClient);
|
|
|
| - power_manager_client_.reset(
|
| - PowerManagerClient::Create(IsUsingReal(DBusClientType::POWER_MANAGER)
|
| - ? REAL_DBUS_CLIENT_IMPLEMENTATION
|
| - : FAKE_DBUS_CLIENT_IMPLEMENTATION));
|
| + power_manager_client_.reset(PowerManagerClient::Create(
|
| + use_real_clients ? REAL_DBUS_CLIENT_IMPLEMENTATION
|
| + : FAKE_DBUS_CLIENT_IMPLEMENTATION));
|
|
|
| - session_manager_client_.reset(
|
| - SessionManagerClient::Create(IsUsingReal(DBusClientType::SESSION_MANAGER)
|
| - ? REAL_DBUS_CLIENT_IMPLEMENTATION
|
| - : FAKE_DBUS_CLIENT_IMPLEMENTATION));
|
| + session_manager_client_.reset(SessionManagerClient::Create(
|
| + use_real_clients ? REAL_DBUS_CLIENT_IMPLEMENTATION
|
| + : FAKE_DBUS_CLIENT_IMPLEMENTATION));
|
|
|
| - if (IsUsingReal(DBusClientType::SMS))
|
| + if (use_real_clients)
|
| sms_client_.reset(SMSClient::Create());
|
| else
|
| sms_client_.reset(new FakeSMSClient);
|
|
|
| - if (IsUsingReal(DBusClientType::SYSTEM_CLOCK))
|
| + if (use_real_clients)
|
| system_clock_client_.reset(SystemClockClient::Create());
|
| else
|
| system_clock_client_.reset(new FakeSystemClockClient);
|
|
|
| - update_engine_client_.reset(
|
| - UpdateEngineClient::Create(IsUsingReal(DBusClientType::UPDATE_ENGINE)
|
| - ? REAL_DBUS_CLIENT_IMPLEMENTATION
|
| - : FAKE_DBUS_CLIENT_IMPLEMENTATION));
|
| + update_engine_client_.reset(UpdateEngineClient::Create(
|
| + use_real_clients ? REAL_DBUS_CLIENT_IMPLEMENTATION
|
| + : FAKE_DBUS_CLIENT_IMPLEMENTATION));
|
| }
|
|
|
| DBusClientsCommon::~DBusClientsCommon() {}
|
|
|
| -bool DBusClientsCommon::IsUsingReal(DBusClientType client) const {
|
| - return real_client_mask_ & static_cast<DBusClientTypeMask>(client);
|
| -}
|
| -
|
| void DBusClientsCommon::Initialize(dbus::Bus* system_bus) {
|
| DCHECK(DBusThreadManager::IsInitialized());
|
|
|
|
|