Implement GetTextFromRange(), GetTextRange() and GetSelectionRange() for ArcImeService.

components/arc/ime/arc_ime_bridge_impl.h

 // 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.
 
 #ifndef COMPONENTS_ARC_IME_ARC_IME_BRIDGE_IMPL_H_
 #define COMPONENTS_ARC_IME_ARC_IME_BRIDGE_IMPL_H_
 
 #include "base/macros.h"
 #include "base/strings/string16.h"
 #include "components/arc/common/ime.mojom.h"
@@ skipping 28 matching lines @@
   void SendConfirmCompositionText() override;
   void SendInsertText(const base::string16& text) override;
   void SendOnKeyboardBoundsChanging(const gfx::Rect& new_bounds) override;
   void SendExtendSelectionAndDelete(size_t before, size_t after) override;
 
   // mojom::ImeHost overrides:
   void OnTextInputTypeChanged(mojom::TextInputType type) override;
   void OnCursorRectChanged(mojom::CursorRectPtr rect) override;
   void OnCancelComposition() override;
   void ShowImeIfNeeded() override;
+  void OnCursorRectChangedWithSurroundingText(
+      mojom::CursorRectPtr rect,

[Luis Héctor Chávez, 2017/05/15 15:44:27]

Can you add typemappings from mojom::CursorRectPtr -> gfx::Rect and
mojom::TextRangePtr -> gfx::Range so you don't need the (manual) conversions?

[yhanada, 2017/05/16 10:02:12]

Done.

+      mojom::TextRangePtr text_range,
+      const std::string& text_in_range,
+      mojom::TextRangePtr selection_range) override;
 
  private:
   mojo::Binding<mojom::ImeHost> binding_;
   Delegate* const delegate_;
   ArcBridgeService* const bridge_service_;
 
   DISALLOW_COPY_AND_ASSIGN(ArcImeBridgeImpl);
 };
 
 }  // namespace arc
 
 #endif  // COMPONENTS_ARC_IME_ARC_IME_BRIDGE_IMPL_H_