OnKeyPress was not returning true when keypress is handled.

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 53 matching lines @@
         Widget::DeleteOnDestroy,
         Widget::DontMirrorOriginInRTL);
     widget_->Init(NULL, gfx::Rect());
     widget_->SetContentsView(textfield_);
     textfield_view_
         = static_cast<NativeTextfieldViews*>(textfield_->native_wrapper());
     DCHECK(textfield_view_);
     model_ = textfield_view_->model_.get();
   }
 
-  void SendKeyEventToTextfieldViews(app::KeyboardCode key_code,
+  bool SendKeyEventToTextfieldViews(app::KeyboardCode key_code,
                                     bool shift,
                                     bool control) {
     int flags = (shift ? KeyEvent::EF_SHIFT_DOWN : 0) |
         (control ? KeyEvent::EF_CONTROL_DOWN : 0);
     KeyEvent event(KeyEvent::ET_KEY_PRESSED, key_code, flags, 1, 0);
-    textfield_view_->OnKeyPressed(event);
+    return textfield_view_->OnKeyPressed(event);
   }
 
-  void SendKeyEventToTextfieldViews(app::KeyboardCode key_code) {
-    SendKeyEventToTextfieldViews(key_code, false, false);
+  bool SendKeyEventToTextfieldViews(app::KeyboardCode key_code) {
+    return SendKeyEventToTextfieldViews(key_code, false, false);
   }
 
  protected:
   // We need widget to populate wrapper class.
   Widget* widget_;
 
   Textfield* textfield_;
   NativeTextfieldViews* textfield_view_;
   TextfieldViewsModel* model_;
 
@@ skipping 121 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, TestOnKeyPressReturnValue) {
+  InitTextfield(Textfield::STYLE_DEFAULT);
+  EXPECT_TRUE(SendKeyEventToTextfieldViews(app::VKEY_A));
+  // F24, up/down key won't be handled.
+  EXPECT_FALSE(SendKeyEventToTextfieldViews(app::VKEY_F24));
+  EXPECT_FALSE(SendKeyEventToTextfieldViews(app::VKEY_UP));
+  EXPECT_FALSE(SendKeyEventToTextfieldViews(app::VKEY_DOWN));
+}
+
 }  // namespace views