Add ime.ExtendSelectionAndDelete Mojo RPC

components/arc/common/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.
 
-// Next MinVersion: 4
+// Next MinVersion: 5
 
 module arc.mojom;
 
 import "screen_rect.mojom";
 
 // Represents the type of text input field currently focused.
 [Extensible]
 enum TextInputType {
   NONE,
   TEXT,
@@ skipping 19 matching lines @@
 // Represents a single segment of text currently composed by IME.
 struct CompositionSegment {
   // Start offset of the segment in UTF-16 index.
   uint32 start_offset;
   // End offset of the segment in UTF-16 index.
   uint32 end_offset;
   // Indicates that this segment should be emphasized.
   bool emphasized;
 };
 
+// Next method ID: 4
 interface ImeHost {
   // Notifies Chrome that the text input focus is changed.
   OnTextInputTypeChanged@0(TextInputType type);
 
   // Notifies Chrome that the cursor poisition has changed.
   OnCursorRectChanged@1(CursorRect rect);
 
   // Notifies Chrome that the current composition is canceled.
   [MinVersion=1] OnCancelComposition@2();
 
   // Show virtual keyboard of Chrome OS if needed.
   [MinVersion=2] ShowImeIfNeeded@3();
 };
 
+// Next method ID: 6
 interface ImeInstance {
   Init@0(ImeHost host_ptr);
 
   // Sets composition text and attributes requested by the host IME.
   SetCompositionText@1(string text, array<CompositionSegment> segments);
 
   // Commits the last set composition text and clears the composition.
   ConfirmCompositionText@2();
 
   // Commits the specified text and clears the composition.
   InsertText@3(string text);
 
   // Informs the virtual keyboard bounds on screen is changing.
   // |new_bounds| Represents a ChromeOS virtual keyboard bounds in ChromeOS
   // screen coordinate, physical pixel as a unit.
   // When all members are zero value, virtual keyboard is being hidden.
   [MinVersion=3] OnKeyboardBoundsChanging@4(ScreenRect new_bounds);
+

[kinaba, 2016/07/11 07:43:56]

nit: 1 blank line (not 2)

[takaoka, 2016/07/11 08:34:55]

Done.

+
+  // Deletes current selection plus the specified number of characters before

[kinaba, 2016/07/11 07:43:56]

Could you elaborate more on the "number of characters"?
(Is it the number of codepoints, or the number of char16 values, or ...?)

[takaoka, 2016/07/11 08:34:55]

Confirmed that "characters" means
char16.https://cs.chromium.org/chromium/src/ui/views/controls/textfield/textfield.cc?q=Textfield::ExtendSelectionAndDelete&sq=package:chromium

+  // and after selection or caret.
+  [MinVersion=4] ExtendSelectionAndDelete@5(uint32 before, uint32 after);
 };