color_utils cleanup:

ash/frame/default_header_painter.cc

Index: ash/frame/default_header_painter.cc
diff --git a/ash/frame/default_header_painter.cc b/ash/frame/default_header_painter.cc
index 432764b27f9a96290d53c7b5c784907b067787a5..18086ac62e0c8971a0d07bf6cd1e30f974299ede 100644
--- a/ash/frame/default_header_painter.cc
+++ b/ash/frame/default_header_painter.cc
@@ -42,8 +42,6 @@ const SkColor kHeaderContentSeparatorInactiveColor =
 const SkColor kDefaultFrameColor = SkColorSetRGB(242, 242, 242);
 // Duration of crossfade animation for activating and deactivating frame.
 const int kActivationCrossfadeDurationMs = 200;
-// Luminance below which to use white caption buttons.
-const int kMaxLuminanceForLightButtons = 125;
 
 // Tiles an image into an area, rounding the top corners.
 void TileRoundRect(gfx::Canvas* canvas,
@@ -279,9 +277,8 @@ void DefaultHeaderPainter::PaintHeaderContentSeparator(gfx::Canvas* canvas) {
 }
 
 bool DefaultHeaderPainter::ShouldUseLightImages() {
-  int luminance = color_utils::GetLuminanceForColor(
-      mode_ == MODE_INACTIVE ? inactive_frame_color_ : active_frame_color_);
-  return luminance < kMaxLuminanceForLightButtons;
+  return color_utils::IsDark(mode_ == MODE_INACTIVE ? inactive_frame_color_

[Peter Kasting, 2016/03/04 01:39:26]

This is not quite the same as the old code, but almost; it's basically "< 128"
instead of "< 125".  This difference is so small I have a hard time believing it
matters; using the standard function seems better.

+                                                    : active_frame_color_);
 }
 
 void DefaultHeaderPainter::UpdateAllButtonImages() {