Fix a recent regression in IME inside OOPIFs

content/public/test/text_input_test_utils.cc

 // 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.
 
 #include "content/public/test/text_input_test_utils.h"
 
 #include <unordered_set>
 
 #include "content/browser/frame_host/render_frame_host_impl.h"
 #include "content/browser/renderer_host/render_widget_host_impl.h"
@@ skipping 244 matching lines @@
   manager->GetActiveWidget()->Send(new InputMsg_RequestCompositionUpdate(
       manager->GetActiveWidget()->GetRoutingID(), true, false));
   return true;
 }
 
 bool DoesFrameHaveFocusedEditableElement(RenderFrameHost* frame) {
   return static_cast<RenderFrameHostImpl*>(frame)
       ->has_focused_editable_element();
 }
 
+void SendImeCommitTextToWidget(
+    RenderWidgetHost* rwh,
+    const base::string16& text,
+    const std::vector<ui::CompositionUnderline>& underlines,
+    const gfx::Range& replacement_range,
+    int relative_cursor_pos) {
+  std::vector<blink::WebCompositionUnderline> web_composition_underlines;
+  for (auto underline : underlines) {
+    web_composition_underlines.push_back(blink::WebCompositionUnderline(

[alexmos, 2017/02/08 02:16:23]

optional: emplace_back would let you avoid explicitly writing
blink::WebCompositionUnderline().

[EhsanK, 2017/02/09 00:50:34]

Sorry I just saw this suggestion. Thanks (And Done).

+        static_cast<int>(underline.start_offset),
+        static_cast<int>(underline.end_offset), underline.color,
+        underline.thick, underline.background_color));
+  }
+  RenderWidgetHostImpl::From(rwh)->ImeCommitText(
+      text, web_composition_underlines, replacement_range, relative_cursor_pos);
+}
+
 size_t GetRegisteredViewsCountFromTextInputManager(WebContents* web_contents) {
   std::unordered_set<RenderWidgetHostView*> views;
   TextInputManager* manager =
       static_cast<WebContentsImpl*>(web_contents)->GetTextInputManager();
   return !!manager ? manager->GetRegisteredViewsCountForTesting() : 0;
 }
 
 RenderWidgetHostView* GetActiveViewFromWebContents(WebContents* web_contents) {
   return static_cast<WebContentsImpl*>(web_contents)
       ->GetTextInputManager()
@@ skipping 132 matching lines @@
 
 #ifdef USE_AURA
   RenderWidgetHostViewAura* view = static_cast<RenderWidgetHostViewAura*>(
       web_contents->GetRenderWidgetHostView());
   observer.reset(new InputMethodObserverAura(view->GetInputMethod()));
 #endif
   return observer;
 }
 
 }  // namespace content