Create org.chromium.KioskAppService.

chrome/browser/chromeos/chrome_browser_main_chromeos.cc

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 78643442b9f306694e2c3a90a533c4278c61b9f9..260d342ef8b07f40ae07e4a0cc029bdc24ff5aed 100644
--- a/chrome/browser/chromeos/chrome_browser_main_chromeos.cc
+++ b/chrome/browser/chromeos/chrome_browser_main_chromeos.cc
@@ -248,11 +248,21 @@ class DBusServices {
       service_providers.push_back(base::MakeUnique<ConsoleServiceProvider>(
           base::MakeUnique<ChromeConsoleServiceProviderDelegate>()));
     }
-    service_providers.push_back(base::MakeUnique<KioskInfoService>());
+    service_providers.push_back(base::MakeUnique<KioskInfoService>(
+        kLibCrosServiceInterface,
+        kKioskAppServiceGetRequiredPlatformVersionMethod));
     cros_dbus_service_ = CrosDBusService::Create(
         kLibCrosServiceName, dbus::ObjectPath(kLibCrosServicePath),
         std::move(service_providers));
 
+    CrosDBusService::ServiceProviderList kiosk_service_providers;
+    kiosk_service_providers.push_back(base::MakeUnique<KioskInfoService>(
+        kKioskAppServiceInterface,
+        kKioskAppServiceGetRequiredPlatformVersionMethod));
+    kiosk_info_service_ = CrosDBusService::Create(
+        kKioskAppServiceName, dbus::ObjectPath(kKioskAppServicePath),
+        std::move(kiosk_service_providers));

[Daniel Erat, 2017/04/05 02:55:41]

nit: no idea how clang-format will indent it, but how do things look if you
instead create the ServiceProviderList inline in the CrosDBusService::Create()
call? it feels like most/all of these service will just have a single provider,
so it might be nice to get rid of the local list vars.

[teravest, 2017/04/05 15:58:20]

I wrestled with this, but can't figure out how to create the vector inline.
push_back() seems to behave well with unique_ptr; list initialization always
tries to make a copy of the
unique_ptr<CrosDBusService::ServiceProviderInterface>, even when I wrap it in
std::move().

[Daniel Erat, 2017/04/05 18:18:00]

hmm. i take it that just passing e.g.

  {base::MakeUnique<KioskInfoService>(...)}

doesn't work either? i don't really understand the rules for when a
std::unique_ptr can be upcast, so

  {base::WrapUnique<ServiceProviderInterface>(
      new KioskInfoService(...))}

might be worth trying too if you haven't already. sigh.

+
     // Initialize PowerDataCollector after DBusThreadManager is initialized.
     PowerDataCollector::Initialize();
 
@@ -302,6 +312,7 @@ class DBusServices {
     CertLoader::Shutdown();
     TPMTokenLoader::Shutdown();
     cros_dbus_service_.reset();
+    kiosk_info_service_.reset();
     PowerDataCollector::Shutdown();
     PowerPolicyController::Shutdown();
     device::BluetoothAdapterFactory::Shutdown();
@@ -318,6 +329,7 @@ class DBusServices {
   // 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> kiosk_info_service_;

[Daniel Erat, 2017/04/05 02:55:41]

nit: add a blank line above this line to make it clear that the comments above
aren't referring to it

[teravest, 2017/04/05 15:58:20]

Done.

 
   std::unique_ptr<NetworkConnectDelegateChromeOS> network_connect_delegate_;