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

Unified Diff: chrome/browser/chromeos/chrome_browser_main_chromeos.cc

Issue 2742343004: Revert of Split LivenessService out from LibCrosService. (Closed)
Patch Set: Created 3 years, 9 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 | chromeos/dbus/services/liveness_service_provider.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/chromeos/chrome_browser_main_chromeos.cc
diff --git a/chrome/browser/chromeos/chrome_browser_main_chromeos.cc b/chrome/browser/chromeos/chrome_browser_main_chromeos.cc
index 7f859b95637d3b84d84a1fc54d9a1a1397534b36..1ee08dfe0278ba9a54298574e5e001efd9f3d5eb 100644
--- a/chrome/browser/chromeos/chrome_browser_main_chromeos.cc
+++ b/chrome/browser/chromeos/chrome_browser_main_chromeos.cc
@@ -238,10 +238,7 @@
service_providers.push_back(base::MakeUnique<DisplayPowerServiceProvider>(
base::MakeUnique<ChromeDisplayPowerServiceProviderDelegate>()));
}
- // TODO(teravest): Remove this service provider once clients use
- // LivenessService instead of LibCrosService.
- service_providers.push_back(
- base::MakeUnique<LivenessServiceProvider>(kLibCrosServiceInterface));
+ service_providers.push_back(base::MakeUnique<LivenessServiceProvider>());
service_providers.push_back(base::MakeUnique<ScreenLockServiceProvider>());
if (ash_util::IsRunningInMash()) {
service_providers.push_back(base::MakeUnique<ConsoleServiceProvider>(
@@ -255,13 +252,6 @@
kLibCrosServiceName, dbus::ObjectPath(kLibCrosServicePath),
std::move(service_providers));
- CrosDBusService::ServiceProviderList liveness_service_providers;
- liveness_service_providers.push_back(
- base::MakeUnique<LivenessServiceProvider>(kLivenessServiceInterface));
- liveness_service_ = CrosDBusService::Create(
- kLivenessServiceName, dbus::ObjectPath(kLivenessServicePath),
- std::move(liveness_service_providers));
-
// Initialize PowerDataCollector after DBusThreadManager is initialized.
PowerDataCollector::Initialize();
@@ -311,7 +301,6 @@
CertLoader::Shutdown();
TPMTokenLoader::Shutdown();
cros_dbus_service_.reset();
- liveness_service_.reset();
PowerDataCollector::Shutdown();
PowerPolicyController::Shutdown();
device::BluetoothAdapterFactory::Shutdown();
@@ -328,8 +317,6 @@
// TODO(derat): Move these providers into more-specific services that are
// split between different processes: http://crbug.com/692246
std::unique_ptr<CrosDBusService> cros_dbus_service_;
-
- std::unique_ptr<CrosDBusService> liveness_service_;
std::unique_ptr<NetworkConnectDelegateChromeOS> network_connect_delegate_;
« no previous file with comments | « no previous file | chromeos/dbus/services/liveness_service_provider.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698