Handle newCursorPosition correctly for Android's commitText()

content/common/input_messages.h

Index: content/common/input_messages.h
diff --git a/content/common/input_messages.h b/content/common/input_messages.h
index a602eb56840f62b39a524522538045e4ddd49743..c3bbdc683f62d5c363a9faebd10a5fae3a599ff3 100644
--- a/content/common/input_messages.h
+++ b/content/common/input_messages.h
@@ -167,10 +167,11 @@ IPC_MESSAGE_ROUTED5(
     int /* selection_end */)
 
 // This message confirms an ongoing composition.
-IPC_MESSAGE_ROUTED3(InputMsg_ImeConfirmComposition,
+IPC_MESSAGE_ROUTED4(InputMsg_ImeConfirmComposition,
                     base::string16 /* text */,
                     gfx::Range /* replacement_range */,
-                    bool /* keep_selection */)
+                    bool /* keep_selection */,
+                    int /* new_cursor_pos */)

[aelias_OOO_until_Jul13, 2016/08/24 08:13:07]

I realize that you matched the name used in the Android documentation, but I
think a clearer name would be "relative_cursor_pos".  Could you
search-and-replace in your patch to switch to that name?  I think that would
avoid the mistake of assuming it's an absolute offset from the beginning of the
textbox.

[yabinh, 2016/08/25 02:32:21]

Done.

 
 // This message notifies the renderer that the next key event is bound to one
 // or more pre-defined edit commands. If the next key event is not handled