Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(2151)

Unified Diff: chrome/browser/ui/views/accessible_pane_view.cc

Issue 7057014: Variety of tweaks to View API. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src/
Patch Set: Created 9 years, 7 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/ui/views/accessible_pane_view.cc
===================================================================
--- chrome/browser/ui/views/accessible_pane_view.cc (revision 86083)
+++ chrome/browser/ui/views/accessible_pane_view.cc (working copy)
@@ -36,7 +36,7 @@
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
@@ -50,7 +50,7 @@
// use the first focusable child.
if (!initial_focus ||
!Contains(initial_focus) ||
- !initial_focus->IsVisible() ||
+ !initial_focus->visible() ||
!initial_focus->IsEnabled()) {
initial_focus = GetFirstFocusableChild();
}
@@ -190,7 +190,7 @@
}
void AccessiblePaneView::SetVisible(bool flag) {
- if (IsVisible() && !flag && pane_has_focus_) {
+ if (visible() && !flag && pane_has_focus_) {
RemovePaneFocus();
RestoreLastFocusedView();
}

Powered by Google App Engine
This is Rietveld 408576698