Fix appearance of task manager border at fractional scale factors.

ui/views/border.cc

Index: ui/views/border.cc
diff --git a/ui/views/border.cc b/ui/views/border.cc
index 14e6d302e20de5e7955465510d7dfdbc2a6cccb9..a53e4a6b46d3a6e07d2bb867650988f1787e712a 100644
--- a/ui/views/border.cc
+++ b/ui/views/border.cc
@@ -12,6 +12,7 @@
 #include "cc/paint/paint_flags.h"
 #include "ui/gfx/canvas.h"
 #include "ui/gfx/geometry/rect_f.h"
+#include "ui/gfx/scoped_canvas.h"
 #include "ui/views/painter.h"
 #include "ui/views/view.h"
 
@@ -42,18 +43,18 @@ SolidSidedBorder::SolidSidedBorder(const gfx::Insets& insets, SkColor color)
 }
 
 void SolidSidedBorder::Paint(const View& view, gfx::Canvas* canvas) {
-  // Top border.
-  canvas->FillRect(gfx::Rect(0, 0, view.width(), insets_.top()), color_);
-  // Left border.
-  canvas->FillRect(gfx::Rect(0, insets_.top(), insets_.left(),
-                             view.height() - insets_.height()), color_);
-  // Bottom border.
-  canvas->FillRect(gfx::Rect(0, view.height() - insets_.bottom(), view.width(),
-                             insets_.bottom()), color_);
-  // Right border.
-  canvas->FillRect(gfx::Rect(view.width() - insets_.right(), insets_.top(),
-                             insets_.right(), view.height() - insets_.height()),
-                   color_);
+  // Undo DSF so that we can be sure to draw an integral number of pixels for
+  // the border. Integral scale factors should be unaffected by this, but for
+  // fractional scale factors this ensures sharp lines.
+  gfx::ScopedCanvas scoped(canvas);
+  float dsf = canvas->UndoDeviceScaleFactor();
+
+  gfx::RectF bounds(view.GetLocalBounds());

[Peter Kasting, 2017/02/17 02:40:15]

Nit: Prefer = to () for this (see
https://www.chromium.org/developers/coding-style/cpp-dos-and-donts#TOC-Variab...
)

[Evan Stade, 2017/02/17 06:19:55]

For rectf from rect you have to use the constructor don't you?

[Peter Kasting, 2017/02/17 07:13:19]

Oh, I didn't even see that you were converting integral->floating here.  Yeah,
that constructor is explicit.

+  bounds.Scale(dsf);
+  // This scaling operation floors the inset values.
+  bounds.Inset(insets_.Scale(dsf));
+  canvas->ClipRect(bounds, SkClipOp::kDifference);
+  canvas->DrawColor(color_);
 }
 
 gfx::Insets SolidSidedBorder::GetInsets() const {