focus reverse traversal was not working for TextfieldViews.

views/controls/textfield/textfield.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 "views/controls/textfield/textfield.h"
 
 #if defined(OS_LINUX)
 #include <gdk/gdkkeysyms.h>
 #endif
 
@@ skipping 243 matching lines @@
 gfx::Size Textfield::GetPreferredSize() {
   gfx::Insets insets;
   if (draw_border_ && native_wrapper_)
     insets = native_wrapper_->CalculateInsets();
   return gfx::Size(font_.GetExpectedTextWidth(default_width_in_chars_) +
                        insets.width(),
                    num_lines_ * font_.GetHeight() + insets.height());
 }
 
 bool Textfield::IsFocusable() const {
-  return IsEnabled() && !read_only_;
+  return View::IsFocusable() && !read_only_;
 }
 
 void Textfield::AboutToRequestFocusFromTabTraversal(bool reverse) {
   SelectAll();
 }
 
 bool Textfield::SkipDefaultKeyEventProcessing(const KeyEvent& e) {
   // TODO(hamaji): Figure out which keyboard combinations we need to add here,
   //               similar to LocationBarView::SkipDefaultKeyEventProcessing.
   app::KeyboardCode key = e.GetKeyCode();
   if (key == app::VKEY_BACK)
     return true;  // We'll handle BackSpace ourselves.
 
 #if defined(OS_WIN)
   // We don't translate accelerators for ALT + NumPad digit on Windows, they are
   // used for entering special characters.  We do translate alt-home.
   if (e.IsAltDown() && (key != app::VKEY_HOME) &&
       app::win::IsNumPadDigit(key, e.IsExtendedKey()))
     return true;
 #endif
   return false;
 }
 
 void Textfield::PaintFocusBorder(gfx::Canvas* canvas) {
   if (NativeViewHost::kRenderNativeControlFocus)
     View::PaintFocusBorder(canvas);
 }
 
+bool Textfield::OnKeyPressed(const views::KeyEvent& e) {
+  return native_wrapper_ && native_wrapper_->HandleKeyPressed(e);
+}
+
+bool Textfield::OnKeyReleased(const views::KeyEvent& e) {
+  return native_wrapper_ && native_wrapper_->HandleKeyReleased(e);
+}
+
+void Textfield::WillGainFocus() {
+  if (native_wrapper_)
+    native_wrapper_->HandleWillGainFocus();
+}
+
+void Textfield::DidGainFocus() {
+  if (native_wrapper_)
+    native_wrapper_->HandleDidGainFocus();
+}
+
+void Textfield::WillLoseFocus() {
+  if (native_wrapper_)
+    native_wrapper_->HandleWillLoseFocus();
+}
+
 AccessibilityTypes::Role Textfield::GetAccessibleRole() {
   return AccessibilityTypes::ROLE_TEXT;
 }
 
 AccessibilityTypes::State Textfield::GetAccessibleState() {
   int state = 0;
   if (read_only())
     state |= AccessibilityTypes::STATE_READONLY;
   if (IsPassword())
     state |= AccessibilityTypes::STATE_PROTECTED;
   return state;
 }
 
 std::wstring Textfield::GetAccessibleValue() {
   if (!text_.empty())
     return UTF16ToWide(text_);
   return std::wstring();
 }
 
 void Textfield::SetEnabled(bool enabled) {
   View::SetEnabled(enabled);
   if (native_wrapper_)
     native_wrapper_->UpdateEnabled();
 }
 
 void Textfield::Focus() {
-  if (native_wrapper_) {
-    // Forward the focus to the wrapper if it exists.
-    native_wrapper_->SetFocus();
-  } else {
-    // If there is no wrapper, cause the RootView to be focused so that we still
-    // get keyboard messages.
+  // Forward the focus to the wrapper if it exists.
+  if (!native_wrapper_ || !native_wrapper_->SetFocus()) {
+    // If there is no wrapper or the wrapper din't take focus, call
+    // View::Focus to clear the native focus so that we still get
+    // keyboard messages.
     View::Focus();
   }
 }
 
 void Textfield::ViewHierarchyChanged(bool is_add, View* parent, View* child) {
   if (is_add && !native_wrapper_ && GetWidget() && !initialized_) {
     initialized_ = true;
 
     // The native wrapper's lifetime will be managed by the view hierarchy after
     // we call AddChildView.
@@ skipping 13 matching lines @@
     static_cast<NativeTextfieldWin*>(native_wrapper_)->AttachHack();
 #endif
   }
 }
 
 std::string Textfield::GetClassName() const {
   return kViewClassName;
 }
 
 }  // namespace views