Plumb composition character bounds for Android 5.0

content/browser/renderer_host/render_widget_host_view_android.cc

Index: content/browser/renderer_host/render_widget_host_view_android.cc
diff --git a/content/browser/renderer_host/render_widget_host_view_android.cc b/content/browser/renderer_host/render_widget_host_view_android.cc
index 0343568ec2c0dc9894f4907a15c3e6a0a17e1b25..3cf9d89cda3f977bc3de256a0c3d2c7a6bc2410a 100644
--- a/content/browser/renderer_host/render_widget_host_view_android.cc
+++ b/content/browser/renderer_host/render_widget_host_view_android.cc
@@ -760,6 +760,12 @@ void RenderWidgetHostViewAndroid::ImeCancelComposition() {
   ime_adapter_android_.CancelComposition();
 }
 
+void RenderWidgetHostViewAndroid::ImeCompositionRangeChanged(
+    const gfx::Range& range,
+    const std::vector<gfx::Rect>& character_bounds) {
+  // TODO(yukawa): Implement this.

[Mike West, 2014/10/25 05:59:53]

It doesn't look like the existing implementation is doing any sanity checking
for the size of the passed vector. It would be worth adding some reasonable
upper bound, but that certainly doesn't have to happen in this CL.

[sky, 2014/10/27 15:28:32]

NOTIMPLEMENTED?

[yukawa, 2014/10/27 16:10:29]

Acknowledged.

[yukawa, 2014/10/27 16:10:29]

Thanks but let me keep this as a comment. I'd like to not introduce a possible
log spam here because this callback actually starts being called in Android 5.0
with this CL.  Hopefully this method will be implemented shortly with subsequent
CLs.

+}
+
 void RenderWidgetHostViewAndroid::FocusedNodeChanged(bool is_editable_node) {
   ime_adapter_android_.FocusedNodeChanged(is_editable_node);
   if (selection_controller_)