views: Convert IsFocusable() to just focusable() since it's just a simple accessor.

chrome/browser/ui/views/tab_contents/native_tab_contents_container_gtk.cc

 // Copyright (c) 2011 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/ui/views/tab_contents/native_tab_contents_container_gtk.h"
 
 #include "chrome/browser/ui/view_ids.h"
 #include "chrome/browser/ui/views/tab_contents/tab_contents_container.h"
 #include "chrome/browser/ui/views/tab_contents/tab_contents_view_views.h"
 #include "content/browser/renderer_host/render_widget_host_view.h"
 #include "content/browser/tab_contents/interstitial_page.h"
 #include "content/browser/tab_contents/tab_contents.h"
 #include "ui/base/accessibility/accessible_view_state.h"
 #include "ui/views/focus/focus_manager.h"
 
 ////////////////////////////////////////////////////////////////////////////////
 // NativeTabContentsContainerGtk, public:
 
 NativeTabContentsContainerGtk::NativeTabContentsContainerGtk(
     TabContentsContainer* container)
     : container_(container),
       focus_callback_id_(0) {
   set_id(VIEW_ID_TAB_CONTAINER_FOCUS_VIEW);
+  // We need to be focusable when our contents is not a view hierarchy, as
+  // clicking on the contents needs to focus us.
+  set_focusable(container_->tab_contents() != NULL);
 }
 
 NativeTabContentsContainerGtk::~NativeTabContentsContainerGtk() {
 }
 
 ////////////////////////////////////////////////////////////////////////////////
 // NativeTabContentsContainerGtk, NativeTabContentsContainer overrides:
 
 void NativeTabContentsContainerGtk::AttachContents(TabContents* contents) {
   Attach(contents->GetNativeView());
@@ skipping 56 matching lines @@
   // not process them, they'll be returned to us and we'll treat them as
   // accelerators then.
   return container_->tab_contents() &&
          !container_->tab_contents()->is_crashed();
 }
 
 views::FocusTraversable* NativeTabContentsContainerGtk::GetFocusTraversable() {
   return NULL;
 }
 
-bool NativeTabContentsContainerGtk::IsFocusable() const {
-  // We need to be focusable when our contents is not a view hierarchy, as
-  // clicking on the contents needs to focus us.

[Ben Goodger (Google), 2011/12/15 21:21:41]

Same question as the other file.

-  return container_->tab_contents() != NULL;
-}
-
 void NativeTabContentsContainerGtk::OnFocus() {
   if (container_->tab_contents())
     container_->tab_contents()->Focus();
 }
 
 void NativeTabContentsContainerGtk::RequestFocus() {
   // This is a hack to circumvent the fact that a view does not explicitly get
   // a call to set the focus if it already has the focus. This causes a problem
   // with tabs such as the TabContents that instruct the RenderView that it got
   // focus when they actually get the focus. When switching from one TabContents
@@ skipping 27 matching lines @@
 }
 
 ////////////////////////////////////////////////////////////////////////////////
 // NativeTabContentsContainer, public:
 
 // static
 NativeTabContentsContainer* NativeTabContentsContainer::CreateNativeContainer(
     TabContentsContainer* container) {
   return new NativeTabContentsContainerGtk(container);
 }