Add service name constant for UI service.

mash/init/init.cc

 // Copyright 2016 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "mash/init/init.h"
 
 #include "base/bind.h"
 #include "base/command_line.h"
 #include "base/guid.h"
 #include "mash/login/public/interfaces/login.mojom.h"
 #include "services/service_manager/public/cpp/connection.h"
 #include "services/service_manager/public/cpp/connector.h"
 #include "services/service_manager/public/cpp/interface_registry.h"
 #include "services/service_manager/public/cpp/service_context.h"
+#include "services/ui/public/interfaces/constants.mojom.h"
 
 namespace mash {
 namespace init {
 
 Init::Init() {}
 Init::~Init() {}
 
 void Init::OnStart() {
-  context()->connector()->Connect("ui");
+  context()->connector()->Connect(ui::mojom::kServiceName);

[James Cook, 2016/11/16 00:46:02]

I like this. It's clear to me that it's connecting to a mojo service, even if I
don't know what a context or a connector are.

   StartTracing();
   StartLogin();
 }
 
 bool Init::OnConnect(const service_manager::ServiceInfo& remote_info,
                      service_manager::InterfaceRegistry* registry) {
   registry->AddInterface<mojom::Init>(this);
   return true;
 }
 
@@ skipping 32 matching lines @@
 
 void Init::StartLogin() {
   login_connection_ = context()->connector()->Connect("login");
   mash::login::mojom::LoginPtr login;
   login_connection_->GetInterface(&login);
   login->ShowLoginUI();
 }
 
 }  // namespace init
 }  // namespace main