views: Improve cursor movements on word boundaries.

views/controls/textfield/native_textfield_views_unittest.cc

 // Copyright (c) 2010 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 "app/keyboard_codes.h"
 #include "base/message_loop.h"
 #include "base/utf_string_conversions.h"
 #include "testing/gtest/include/gtest/gtest.h"
 #include "views/controls/textfield/native_textfield_views.h"
 #include "views/controls/textfield/textfield.h"
@@ skipping 205 matching lines @@
 
 TEST_F(NativeTextfieldViewsTest, PasswordTest) {
   InitTextfield(Textfield::STYLE_PASSWORD);
   textfield_->SetText(ASCIIToUTF16("my password"));
   // Just to make sure the text() and callback returns
   // the actual text instead of "*".
   EXPECT_STR_EQ("my password", textfield_->text());
   EXPECT_STR_EQ("my password", last_contents_);
 }
 
+TEST_F(NativeTextfieldViewsTest, CursorMovement) {
+  InitTextfield(Textfield::STYLE_DEFAULT);
+  textfield_->SetText(ASCIIToUTF16("one two hre "));
+
+  // Send the cursor at the end.
+  SendKeyEventToTextfieldViews(app::VKEY_END);
+  // Ctrl+Left should move the cursor just before the last word.
+  SendKeyEventToTextfieldViews(app::VKEY_LEFT, false, true);
+  SendKeyEventToTextfieldViews(app::VKEY_T);
+  EXPECT_STR_EQ("one two thre ", textfield_->text());
+  EXPECT_STR_EQ("one two thre ", last_contents_);
+
+  // Ctrl+Right should move the cursor to the end of the last word.
+  SendKeyEventToTextfieldViews(app::VKEY_RIGHT, false, true);
+  SendKeyEventToTextfieldViews(app::VKEY_E);
+  EXPECT_STR_EQ("one two three ", textfield_->text());
+  EXPECT_STR_EQ("one two three ", last_contents_);
+
+  // Ctrl+Right again should move the cursor to the end.
+  SendKeyEventToTextfieldViews(app::VKEY_RIGHT, false, true);
+  SendKeyEventToTextfieldViews(app::VKEY_BACK);
+  EXPECT_STR_EQ("one two three", textfield_->text());
+  EXPECT_STR_EQ("one two three", last_contents_);

[oshima, 2011/01/04 21:53:24]

can you add another test case with leading white space?

[sadrul, 2011/01/04 23:25:44]

Done.

+}
+
 }  // namespace views