Add ability to constrain dominant color selection to a HSL range.

ui/gfx/color_utils_unittest.cc

Index: ui/gfx/color_utils_unittest.cc
diff --git a/ui/gfx/color_utils_unittest.cc b/ui/gfx/color_utils_unittest.cc
index 59eaeba1e3e8c779b3a475cc768003e54113a1f7..96730a1053f751eb09fe0e7201e1ffc38ff1623c 100644
--- a/ui/gfx/color_utils_unittest.cc
+++ b/ui/gfx/color_utils_unittest.cc
@@ -36,7 +36,6 @@ TEST(ColorUtils, HSLToSkColorWithAlpha) {
   EXPECT_EQ(SkColorGetB(red), SkColorGetB(result));
 }
 
-
 TEST(ColorUtils, RGBtoHSLRoundTrip) {
   // Just spot check values near the edges.
   for (int r = 0; r < 10; ++r) {
@@ -67,6 +66,38 @@ TEST(ColorUtils, RGBtoHSLRoundTrip) {
   }
 }
 
+TEST(ColorUtils, IsWithinHSLRange) {
+  color_utils::HSL hsl = {0.3, 0.4, 0.5};

[Alexei Svitkine (slow), 2014/05/30 14:56:39]

Nit: Make the test file have namespace color_utils, so you don't need to prefix
things within it.

[calamity, 2014/06/02 03:53:17]

Done.

+  color_utils::HSL lower = {0.2, 0.3, 0.4};
+  color_utils::HSL upper = {0.4, 0.5, 0.6};
+  EXPECT_TRUE(color_utils::IsWithinHSLRange(hsl, lower, upper));
+  // Bounds are inclusive.
+  hsl.h = 0.2;
+  EXPECT_TRUE(color_utils::IsWithinHSLRange(hsl, lower, upper));
+  hsl.h = 0.4;
+  EXPECT_TRUE(color_utils::IsWithinHSLRange(hsl, lower, upper));
+  hsl.s = 0.3;
+  EXPECT_TRUE(color_utils::IsWithinHSLRange(hsl, lower, upper));
+  hsl.s = 0.5;
+  EXPECT_TRUE(color_utils::IsWithinHSLRange(hsl, lower, upper));
+  hsl.l = 0.4;
+  EXPECT_TRUE(color_utils::IsWithinHSLRange(hsl, lower, upper));
+  hsl.l = 0.6;
+  EXPECT_TRUE(color_utils::IsWithinHSLRange(hsl, lower, upper));
+}
+
+TEST(ColorUtils, IsWithinHSLRangeHueWrapAround) {
+  color_utils::HSL hsl = {0.3, 0.4, 0.5};
+  color_utils::HSL lower = {0.8, -1, -1};
+  color_utils::HSL upper = {1.2, -1, -1};
+  hsl.h = 0.1;
+  EXPECT_TRUE(color_utils::IsWithinHSLRange(hsl, lower, upper));
+  hsl.h = 0.9;
+  EXPECT_TRUE(color_utils::IsWithinHSLRange(hsl, lower, upper));
+  hsl.h = 0.3;
+  EXPECT_FALSE(color_utils::IsWithinHSLRange(hsl, lower, upper));
+}
+
 TEST(ColorUtils, ColorToHSLRegisterSpill) {
   // In a opt build on Linux, this was causing a register spill on my laptop
   // (Pentium M) when converting from SkColor to HSL.