IME for Mus: Make InputMethodMus use the IME Mojo API.

mash/session/session.cc

 // Copyright 2015 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/session/session.h"
 
 #include "base/bind.h"
 #include "base/command_line.h"
 #include "base/message_loop/message_loop.h"
 #include "mash/login/public/interfaces/login.mojom.h"
@@ skipping 13 matching lines @@
 namespace mash {
 namespace session {
 
 Session::Session() : screen_locked_(false) {}
 Session::~Session() {}
 
 void Session::OnStart(const shell::Identity& identity) {
   StartAppDriver();
   StartWindowManager();
   StartQuickLaunch();
+  StartIMEDriver();
   // Launch a chrome window for dev convience; don't do this in the long term.
   connector()->Connect("exe:chrome");
 }
 
 bool Session::OnConnect(const shell::Identity& remote_identity,
                         shell::InterfaceRegistry* registry) {
   registry->AddInterface<mojom::Session>(this);
   return true;
 }
 
@@ skipping 58 matching lines @@
       base::Bind(&Session::StartAppDriver, base::Unretained(this)));
 }
 
 void Session::StartQuickLaunch() {
   StartRestartableService(
       "mojo:quick_launch",
       base::Bind(&Session::StartQuickLaunch,
                  base::Unretained(this)));
 }
 
+void Session::StartIMEDriver() {
+  StartRestartableService(

[sky, 2016/08/18 23:43:07]

Do we really need the Ime service to be restartable? It seems like the IME
service should be created the first time someone uses it and be allowed to close
when no one is connected to it.

[Hadi, 2016/08/25 18:06:06]

Based on our internal discussion, moved starting of IME driver to IMEServerImpl.

+      "mojo:test_ime_driver",

[sky, 2016/08/18 23:43:07]

Why are we starting a test driver?

[Hadi, 2016/08/25 18:06:06]

Based on our internal discussion, InputMethodMus always assume that there is a
running driver. We only have the test driver for now, and we will switch to the
proper driver once we have it.

+      base::Bind(&Session::StartIMEDriver, base::Unretained(this)));
+}
+
 void Session::StartScreenlock() {
   StartRestartableService(
       "mojo:screenlock",
       base::Bind(&Session::StartScreenlock,
                  base::Unretained(this)));
 }
 
 void Session::StopScreenlock() {
   auto connection = connections_.find("mojo:screenlock");
   DCHECK(connections_.end() != connection);
@@ skipping 10 matching lines @@
   // Note: |connection| may be null if we've lost our connection to the shell.
   if (connection) {
     connection->SetConnectionLostClosure(
         base::Bind(&LogAndCallServiceRestartCallback, url, restart_callback));
     connections_[url] = std::move(connection);
   }
 }
 
 }  // namespace session
 }  // namespace main