Fixes the implementation of TabContentsViewMac::TakeFocus().

chrome/browser/tab_contents/tab_contents_view_mac.mm

 // Copyright (c) 2006-2008 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 "chrome/browser/tab_contents/tab_contents_view_mac.h"
 
 #include "chrome/browser/browser.h" // TODO(beng): this dependency is awful.
 #include "chrome/browser/cocoa/sad_tab_view.h"
 #include "chrome/browser/renderer_host/render_widget_host.h"
 #include "chrome/browser/renderer_host/render_widget_host_view_mac.h"
@@ skipping 108 matching lines @@
 void TabContentsViewMac::RestoreFocus() {
   // TODO(port)
   // For now just assume we are viewing the tab for the first time.
   SetInitialFocus();
 }
 
 void TabContentsViewMac::UpdateDragCursor(bool is_drop_target) {
   NOTIMPLEMENTED();
 }
 
+// This is called when we the renderer asks us to take focus back (i.e., it has
+// iterated past the last focusable element on the page).
 void TabContentsViewMac::TakeFocus(bool reverse) {
-  [cocoa_view_.get() becomeFirstResponder];
+  if (reverse) {
+    [[cocoa_view_ window] selectPreviousKeyView:cocoa_view_.get()];
+  } else {
+    [[cocoa_view_ window] selectNextKeyView:cocoa_view_.get()];
+  }
 }
 
 void TabContentsViewMac::HandleKeyboardEvent(
     const NativeWebKeyboardEvent& event) {
   [cocoa_view_.get() processKeyboardEvent:event.os_event];
 }
 
 void TabContentsViewMac::ShowContextMenu(const ContextMenuParams& params) {
   RenderViewContextMenuMac menu(web_contents(),
                                 params,
@@ skipping 97 matching lines @@
 }
 
 - (void)paste:(id)sender {
   TabContentsView_->web_contents()->Paste();
 }
 
 // Tons of stuff goes here, where we grab events going on in Cocoaland and send
 // them into the C++ system. TODO(avi): all that jazz
 
 @end