Variety of tweaks to View API.

chrome/browser/ui/views/accessible_pane_view.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 "base/logging.h"
 #include "chrome/browser/ui/view_ids.h"
 #include "chrome/browser/ui/views/accessible_pane_view.h"
 #include "chrome/browser/ui/views/frame/browser_view.h"
 #include "chrome/browser/ui/views/location_bar/location_bar_view.h"
 #include "ui/base/accessibility/accessible_view_state.h"
@@ skipping 18 matching lines @@
 }
 
 AccessiblePaneView::~AccessiblePaneView() {
   if (pane_has_focus_) {
     focus_manager_->RemoveFocusChangeListener(this);
   }
 }
 
 bool AccessiblePaneView::SetPaneFocus(int view_storage_id,
                                       views::View* initial_focus) {
-  if (!IsVisible())
+  if (!visible())
     return false;
 
   // Save the storage id to the last focused view. This would be used to request
   // focus to the view when the traversal is ended.
   last_focused_view_storage_id_ = view_storage_id;
 
   if (!focus_manager_)
     focus_manager_ = GetFocusManager();
 
   // Use the provided initial focus if it's visible and enabled, otherwise
   // use the first focusable child.
   if (!initial_focus ||
       !Contains(initial_focus) ||
-      !initial_focus->IsVisible() ||
+      !initial_focus->visible() ||
       !initial_focus->IsEnabled()) {
     initial_focus = GetFirstFocusableChild();
   }
 
   // Return false if there are no focusable children.
   if (!initial_focus)
     return false;
 
   // Set focus to the initial view. If it's a location bar, use a special
   // method that tells it to select all, also.
@@ skipping 119 matching lines @@
     case ui::VKEY_END:
       focus_manager_->SetFocusedViewWithReason(
           GetLastFocusableChild(), views::FocusManager::kReasonFocusTraversal);
       return true;
     default:
       return false;
   }
 }
 
 void AccessiblePaneView::SetVisible(bool flag) {
-  if (IsVisible() && !flag && pane_has_focus_) {
+  if (visible() && !flag && pane_has_focus_) {
     RemovePaneFocus();
     RestoreLastFocusedView();
   }
   View::SetVisible(flag);
 }
 
 void AccessiblePaneView::GetAccessibleState(ui::AccessibleViewState* state) {
   state->role = ui::AccessibilityTypes::ROLE_PANE;
 }
 
@@ skipping 43 matching lines @@
 
 views::FocusTraversable* AccessiblePaneView::GetFocusTraversableParent() {
   DCHECK(pane_has_focus_);
   return NULL;
 }
 
 views::View* AccessiblePaneView::GetFocusTraversableParentView() {
   DCHECK(pane_has_focus_);
   return NULL;
 }