Handle newCursorPosition correctly for Android's commitText()

content/renderer/browser_plugin/browser_plugin.cc

Index: content/renderer/browser_plugin/browser_plugin.cc
diff --git a/content/renderer/browser_plugin/browser_plugin.cc b/content/renderer/browser_plugin/browser_plugin.cc
index 93947fdeeb898165dccdcdfaf2d70e7648deadf8..afd5a9a6859c02c3bb9da769017cc72078557509 100644
--- a/content/renderer/browser_plugin/browser_plugin.cc
+++ b/content/renderer/browser_plugin/browser_plugin.cc
@@ -541,15 +541,15 @@ bool BrowserPlugin::setComposition(
 
 bool BrowserPlugin::confirmComposition(
     const blink::WebString& text,
-    blink::WebWidget::ConfirmCompositionBehavior selectionBehavior) {
+    blink::WebWidget::ConfirmCompositionBehavior selectionBehavior,

[Changwan Ryu, 2016/08/03 07:37:29]

selection_behavior

[yabinh, 2016/08/08 07:33:43]

Done.

+    int newCursorPosition) {

[Changwan Ryu, 2016/08/03 07:37:29]

new_cursor_position

[yabinh, 2016/08/08 07:33:43]

Done.

   if (!attached())
     return false;
   bool keep_selection = (selectionBehavior == blink::WebWidget::KeepSelection);
   BrowserPluginManager::Get()->Send(
       new BrowserPluginHostMsg_ImeConfirmComposition(
-          browser_plugin_instance_id_,
-          text.utf8(),
-          keep_selection));
+          browser_plugin_instance_id_, text.utf8(), keep_selection,
+          newCursorPosition));
   // TODO(kochi): This assumes the IPC handling always succeeds.
   return true;
 }