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

Unified Diff: ui/views/controls/tree/tree_view.cc

Issue 2715813002: Change focus indicator for TableView and TreeView. (Closed)
Patch Set: Created 3 years, 10 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: ui/views/controls/tree/tree_view.cc
diff --git a/ui/views/controls/tree/tree_view.cc b/ui/views/controls/tree/tree_view.cc
index 7e964f96c66ea799acd3547c60ee57475be07fbd..bd93dddedb0cb8754b224b0fbe74e81a370f9814 100644
--- a/ui/views/controls/tree/tree_view.cc
+++ b/ui/views/controls/tree/tree_view.cc
@@ -807,8 +807,6 @@ void TreeView::PaintRow(gfx::Canvas* canvas,
if (!PlatformStyle::kTreeViewSelectionPaintsEntireRow &&
node == selected_node_) {
canvas->FillRect(text_bounds, selected_row_bg_color);
- if (HasFocus())
- canvas->DrawFocusRect(text_bounds);
}
// Paint the text.
@@ -1058,10 +1056,6 @@ bool TreeView::IsPointInExpandControl(InternalNode* node,
}
void TreeView::SetHasFocusRing(bool shows) {
- if (!ui::MaterialDesignController::IsSecondaryUiMaterial() ||
- !PlatformStyle::kTreeViewHasFocusRing) {
- return;
- }
// If this View is the grandchild of a ScrollView, use the grandparent
// ScrollView for the focus ring instead of this View so that the focus ring
// won't be scrolled. Since ScrollViews have a single content view, which they

Powered by Google App Engine
This is Rietveld 408576698