diff options
author | glen@chromium.org <glen@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-07-15 20:35:13 +0000 |
---|---|---|
committer | glen@chromium.org <glen@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-07-15 20:35:13 +0000 |
commit | fb23b9b370b689a954fb42b0beea2e61eee9e159 (patch) | |
tree | eec1d826f89a34d48de7868ca34fe05b8d2d9fc4 /skia/ext/skia_utils_unittest.cc | |
parent | 0aad67b662f0ba306eeea71117d77bccefc3533f (diff) | |
download | chromium_src-fb23b9b370b689a954fb42b0beea2e61eee9e159.zip chromium_src-fb23b9b370b689a954fb42b0beea2e61eee9e159.tar.gz chromium_src-fb23b9b370b689a954fb42b0beea2e61eee9e159.tar.bz2 |
Make our HSL shifting match Photoshop's.
Also clean up a bunch of PMColor code - skia_utils should operate on SkColors and PMColors should only ever be used by SkBitmaps and ImageOperations.
BUG=16687
Review URL: http://codereview.chromium.org/149663
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@20788 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'skia/ext/skia_utils_unittest.cc')
-rw-r--r-- | skia/ext/skia_utils_unittest.cc | 5 |
1 files changed, 2 insertions, 3 deletions
diff --git a/skia/ext/skia_utils_unittest.cc b/skia/ext/skia_utils_unittest.cc index 21a6c8d..3b590b4 100644 --- a/skia/ext/skia_utils_unittest.cc +++ b/skia/ext/skia_utils_unittest.cc @@ -29,12 +29,11 @@ TEST(SkiaUtils, SkColorToHSLGrey) { } TEST(SkiaUtils, HSLToSkColorWithAlpha) { - // Premultiplied alpha - this is full red. - SkColor red = SkColorSetARGB(128, 128, 0, 0); + SkColor red = SkColorSetARGB(128, 255, 0, 0); skia::HSL hsl = { 0, 1, 0.5 }; - SkColor result = skia::HSLToSKColor(128, hsl); + SkColor result = skia::HSLToSkColor(128, hsl); EXPECT_EQ(SkColorGetA(red), SkColorGetA(result)); EXPECT_EQ(SkColorGetR(red), SkColorGetR(result)); EXPECT_EQ(SkColorGetG(red), SkColorGetG(result)); |