Improve appearance of focus rings at fractional scales.

ui/views/controls/tree/tree_view.cc

 // Copyright (c) 2012 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 "ui/views/controls/tree/tree_view.h"
 
 #include <algorithm>
 
 #include "base/i18n/rtl.h"
 #include "base/memory/ptr_util.h"
@@ skipping 1040 matching lines @@
   int arrow_dx = depth * kIndent + kHorizontalInset;
   gfx::Rect arrow_bounds(bounds().x() + arrow_dx,
                          row * row_height_ + kVerticalInset, kArrowRegionSize,
                          row_height_);
   if (base::i18n::IsRTL())
     arrow_bounds.set_x(bounds().width() - arrow_dx - kArrowRegionSize);
   return arrow_bounds.Contains(point);
 }
 
 void TreeView::SetHasFocusRing(bool shows) {
-  if (!ui::MaterialDesignController::IsSecondaryUiMaterial() ||
+  if (/*!ui::MaterialDesignController::IsSecondaryUiMaterial() ||*/

[msw, 2017/02/23 19:15:13]

Fix this

[Evan Stade, 2017/02/23 20:07:30]

my bad, this is residue from the CL I split off.

       !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
   // wrap in a ScrollView::Viewport, being the grandchild of a ScrollView
   // implies being the sole grandchild, which means it's fine to wrap the focus
   // ring around the grandparent here.
   View* grandparent = parent() ? parent()->parent() : nullptr;
@@ skipping 34 matching lines @@
   if (!is_expanded_)
     return max_width;
   for (int i = 0; i < child_count(); ++i) {
     max_width = std::max(max_width,
                          GetChild(i)->GetMaxWidth(indent, depth + 1));
   }
   return max_width;
 }
 
 }  // namespace views