Handle newCursorPosition correctly for Android's commitText()

content/renderer/render_view_browsertest.cc

Index: content/renderer/render_view_browsertest.cc
diff --git a/content/renderer/render_view_browsertest.cc b/content/renderer/render_view_browsertest.cc
index 64556e6969084e2caa7e133c395db71bbe7633f2..968b3f9cc2f072e7607a03550a3106255041d192 100644
--- a/content/renderer/render_view_browsertest.cc
+++ b/content/renderer/render_view_browsertest.cc
@@ -1315,8 +1315,7 @@ TEST_F(RenderViewImplTest, ImeComposition) {
       case IME_CONFIRMCOMPOSITION:
         view()->OnImeConfirmComposition(
             base::WideToUTF16(ime_message->ime_string),
-            gfx::Range::InvalidRange(),
-            false);
+            gfx::Range::InvalidRange(), false, 1);
         break;
 
       case IME_CANCELCOMPOSITION:
@@ -1572,8 +1571,8 @@ TEST_F(RenderViewImplTest, GetCompositionCharacterBoundsTest) {
 
   for (size_t i = 0; i < bounds.size(); ++i)
     EXPECT_LT(0, bounds[i].width());
-  view()->OnImeConfirmComposition(
-      empty_string, gfx::Range::InvalidRange(), false);
+  view()->OnImeConfirmComposition(empty_string, gfx::Range::InvalidRange(),
+                                  false, 1);
 
   // Non surrogate pair unicode character.
   const base::string16 unicode_composition = base::UTF8ToUTF16(
@@ -1584,8 +1583,8 @@ TEST_F(RenderViewImplTest, GetCompositionCharacterBoundsTest) {
   ASSERT_EQ(unicode_composition.size(), bounds.size());
   for (size_t i = 0; i < bounds.size(); ++i)
     EXPECT_LT(0, bounds[i].width());
-  view()->OnImeConfirmComposition(
-      empty_string, gfx::Range::InvalidRange(), false);
+  view()->OnImeConfirmComposition(empty_string, gfx::Range::InvalidRange(),
+                                  false, 1);
 
   // Surrogate pair character.
   const base::string16 surrogate_pair_char =
@@ -1599,8 +1598,8 @@ TEST_F(RenderViewImplTest, GetCompositionCharacterBoundsTest) {
   ASSERT_EQ(surrogate_pair_char.size(), bounds.size());
   EXPECT_LT(0, bounds[0].width());
   EXPECT_EQ(0, bounds[1].width());
-  view()->OnImeConfirmComposition(
-      empty_string, gfx::Range::InvalidRange(), false);
+  view()->OnImeConfirmComposition(empty_string, gfx::Range::InvalidRange(),
+                                  false, 1);
 
   // Mixed string.
   const base::string16 surrogate_pair_mixed_composition =
@@ -1623,8 +1622,8 @@ TEST_F(RenderViewImplTest, GetCompositionCharacterBoundsTest) {
       EXPECT_LT(0, bounds[i].width());
     }
   }
-  view()->OnImeConfirmComposition(
-      empty_string, gfx::Range::InvalidRange(), false);
+  view()->OnImeConfirmComposition(empty_string, gfx::Range::InvalidRange(),
+                                  false, 1);
 }
 #endif