Improve painting of cros tray separators.

ash/common/system/tray/tray_background_view.cc

Index: ash/common/system/tray/tray_background_view.cc
diff --git a/ash/common/system/tray/tray_background_view.cc b/ash/common/system/tray/tray_background_view.cc
index fb988555461b5df0a564ecaa62eaf36c381cc646..82262e8b763d1ec286a2c62382aa7216e878808c 100644
--- a/ash/common/system/tray/tray_background_view.cc
+++ b/ash/common/system/tray/tray_background_view.cc
@@ -7,7 +7,6 @@
 #include <algorithm>
 
 #include "ash/common/ash_constants.h"
-#include "ash/common/material_design/material_design_controller.h"
 #include "ash/common/shelf/shelf_constants.h"
 #include "ash/common/shelf/wm_shelf.h"
 #include "ash/common/shelf/wm_shelf_util.h"
@@ -567,21 +566,15 @@ void TrayBackgroundView::OnPaintFocus(gfx::Canvas* canvas) {
   // sure clicking on the edges brings up the popup. However, the focus border
   // should be only around the container.
   gfx::RectF paint_bounds;
-  if (MaterialDesignController::IsShelfMaterial()) {
-    paint_bounds = gfx::RectF(GetBackgroundBounds());
-    paint_bounds.Inset(gfx::Insets(-kFocusBorderThickness));
-  } else {
-    paint_bounds = gfx::RectF(GetContentsBounds());
-    paint_bounds.Inset(gfx::Insets(1));
-  }
+  paint_bounds = gfx::RectF(GetBackgroundBounds());
+  paint_bounds.Inset(gfx::Insets(-kFocusBorderThickness));
   canvas->DrawSolidFocusRect(paint_bounds, kFocusBorderColor,
                              kFocusBorderThickness);
 }
 
 void TrayBackgroundView::OnPaint(gfx::Canvas* canvas) {
   ActionableView::OnPaint(canvas);
-  if (!MaterialDesignController::IsShelfMaterial() ||
-      shelf()->GetBackgroundType() ==
+  if (shelf()->GetBackgroundType() ==
           ShelfBackgroundType::SHELF_BACKGROUND_DEFAULT ||
       !separator_visible_) {
     return;
@@ -590,33 +583,21 @@ void TrayBackgroundView::OnPaint(gfx::Canvas* canvas) {
   // right or left of the TrayBackgroundView when the system is LTR or RTL
   // aligned, respectively. For a vertical shelf draw the separator line
   // underneath the items instead.
-  const bool horizontal_shelf = IsHorizontalAlignment(shelf_alignment_);
   const gfx::Rect local_bounds = GetLocalBounds();
-  const int height = kTrayItemSize;
-  const int x =
-      horizontal_shelf
-          ? (base::i18n::IsRTL() ? 0 : (local_bounds.width() - kSeparatorWidth))
-          : (local_bounds.height() - kSeparatorWidth);
-  const int y = (GetShelfConstant(SHELF_SIZE) - kTrayItemSize) / 2;
-  gfx::ScopedCanvas scoped_canvas(canvas);
-  const float scale = canvas->UndoDeviceScaleFactor();
-  cc::PaintFlags flags;
-  flags.setColor(kTraySeparatorColor);
-  flags.setAntiAlias(true);
-
-  const gfx::Rect bounds = horizontal_shelf
-                               ? gfx::Rect(x, y, kSeparatorWidth, height)
-                               : gfx::Rect(y, x, height, kSeparatorWidth);
-  gfx::RectF rect(gfx::ScaleRect(gfx::RectF(bounds), scale));
-  gfx::PointF line_start =
-      horizontal_shelf
-          ? (base::i18n::IsRTL() ? rect.origin() : rect.top_right())
-          : rect.bottom_left();
-  gfx::PointF line_end =
-      (horizontal_shelf && base::i18n::IsRTL() ? rect.bottom_left()
-                                               : rect.bottom_right());
-
-  canvas->DrawLine(line_start, line_end, flags);
+  const SkColor color = SkColorSetA(SK_ColorWHITE, 0x4D);

[tdanderson, 2017/02/28 17:18:19]

Why are you changing the alpha from 0x99 to 0x4D?

[Evan Stade, 2017/02/28 18:15:26]

see my comment that I sent out with the review for explanation.

[tdanderson, 2017/02/28 18:41:34]

Acknowledged.

+
+  if (IsHorizontalAlignment(shelf_alignment_)) {
+    const gfx::PointF point(
+        base::i18n::IsRTL() ? 0 : (local_bounds.width() - kSeparatorWidth),
+        (GetShelfConstant(SHELF_SIZE) - kTrayItemSize) / 2);

[tdanderson, 2017/02/28 17:18:19]

looks like a case of a missing )

+    const gfx::Vector2dF vector(0, kTrayItemSize);
+    canvas->Draw1pxLine(point, point + vector, color);
+  } else {
+    const gfx::PointF point((GetShelfConstant(SHELF_SIZE) - kTrayItemSize) / 2,

[tdanderson, 2017/02/28 17:18:19]

I think your x and y should be reversed here

[Evan Stade, 2017/02/28 18:15:26]

I don't think so. This is for vertical shelf.

[tdanderson, 2017/02/28 18:41:34]

My mistake, I was just looking at the current code's definition of x and y,
neglecting to notice that these are swapped by line 207 for a vertical shelf.

+                            local_bounds.height() - kSeparatorWidth);
+    const gfx::Vector2dF vector(kTrayItemSize, 0);
+    canvas->Draw1pxLine(point, point + vector, color);
+  }
 }
 
 gfx::Insets TrayBackgroundView::GetBackgroundInsets() const {