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

services/ui/public/interfaces/ime.mojom

 // 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.
 
 module ui.mojom;
 
 import "ui/events/mojo/event.mojom";
 import "ui/gfx/geometry/mojo/geometry.mojom";
 
 struct CompositionEvent {
@@ skipping 84 matching lines @@
 
   ProcessKeyEvent(ui.mojom.Event key_event);
   CancelComposition();
 };
 
 // IME drivers send updates to clients using the TextInputClient interface.
 interface TextInputClient {
   // Corresponds to "input method result" functions of ui::TextInputClient.
   OnCompositionEvent(CompositionEvent event);
 
+  // Called when IME driver didn't handle the event.

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

Please better document when this is called. Maybe the documentation should be in
InputMethod, e.g.:

// Called to process a key event. If the event is not handled by
// InputMethod OnUnhandledEvent is called on TextInputClient.

and then you could refer to ProcessKeyEvent here.

Also, not for this cl, but remind me why we went with the name of
TextInputClient instead of InputMethodClient? Wouldn't that better match who the
client is paired to?

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

Done.
I agree that InputMethodClient is a better name. I chose to go with
TextInputClient so it matches with the corresponding ui::TextInputClient, which
has a similar functionality.

+  OnUnhandledEvent(ui.mojom.Event key_event);
+
   // TODO(moshayedi): Add functions corresponding to ui::TextInputClient for:
   // - Input text confirmation
   // - Document content operations
   // - Miscellaneous functions
   // crbug.com/631527.
 };