Tracking composition range on the browser side (Aura)

content/browser/renderer_host/render_widget_host_view_base.cc

 // Copyright (c) 2012 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 "content/browser/renderer_host/render_widget_host_view_base.h"
 
 #include "base/logging.h"
 #include "build/build_config.h"
 #include "content/browser/accessibility/browser_accessibility_manager.h"
 #include "content/browser/gpu/gpu_data_manager_impl.h"
@@ skipping 406 matching lines @@
 }
 
 void RenderWidgetHostViewBase::ImeCancelComposition() {
 // TODO(ekaramad): Use TextInputManager code paths for IME on other platforms.
 #if defined(USE_AURA)
   if (GetTextInputManager())
     GetTextInputManager()->ImeCancelComposition(this);
 #endif
 }
 
+void RenderWidgetHostViewBase::ImeCompositionRangeChanged(
+    const gfx::Range& range,
+    const std::vector<gfx::Rect>& character_bounds) {
+#if defined(USE_AURA)

[kenrb, 2016/07/08 18:13:42]

I would add the TODO for removal on this guard, as you do for the other ones.

[EhsanK, 2016/07/11 18:07:21]

Acknowledged.

+  if (GetTextInputManager()) {
+    GetTextInputManager()->ImeCompositionRangeChanged(this, range,
+                                                      character_bounds);
+  }
+#endif
+}
+
 TextInputManager* RenderWidgetHostViewBase::GetTextInputManager() {
   if (text_input_manager_)
     return text_input_manager_;
 
   RenderWidgetHostImpl* host =
       RenderWidgetHostImpl::From(GetRenderWidgetHost());
   if (!host || !host->delegate())
     return nullptr;
 
   // This RWHV needs to be registered with the TextInputManager so that the
@@ skipping 17 matching lines @@
 
 bool RenderWidgetHostViewBase::IsChildFrameForTesting() const {
   return false;
 }
 
 cc::SurfaceId RenderWidgetHostViewBase::SurfaceIdForTesting() const {
   return cc::SurfaceId();
 }
 
 }  // namespace content