diff options
author | msarett <msarett@google.com> | 2016-09-22 12:37:04 -0700 |
---|---|---|
committer | Commit bot <commit-bot@chromium.org> | 2016-09-22 12:37:04 -0700 |
commit | cf7b877d62537672b67449bc96858cc1262be5f8 (patch) | |
tree | bf2263dd9fbefb81cf29b4dc1398b9bbdec998e2 /gm/colorspacexform.cpp | |
parent | 5f544345d653900ef30212ea1c243947be8bb074 (diff) |
Add BGRA as input format to SkColorSpaceXform
This is immediately useful for webp and I think it's a fair guess
that BGRA src formats are not uncommon.
BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2353363008
Review-Url: https://codereview.chromium.org/2353363008
Diffstat (limited to 'gm/colorspacexform.cpp')
-rw-r--r-- | gm/colorspacexform.cpp | 6 |
1 files changed, 2 insertions, 4 deletions
diff --git a/gm/colorspacexform.cpp b/gm/colorspacexform.cpp index e1afb1cbfb..7e3285efbd 100644 --- a/gm/colorspacexform.cpp +++ b/gm/colorspacexform.cpp @@ -24,9 +24,6 @@ protected: for (int i = 0; i < kNumColors; i++) { fSRGBColors[i] = SkColor4f::FromColor(colors[i]); - - // FIXME (msarett): SkColorSpaceXform does not support BGRA. - colors[i] = SkSwizzle_RB(colors[i]); } static constexpr float kWideGamutRGB_toXYZD50[]{ @@ -45,7 +42,8 @@ protected: dstSpace.get()); xform->apply(fWideGamutColors, colors, kNumColors, - SkColorSpaceXform::kRGBA_F32_ColorFormat, kOpaque_SkAlphaType); + SkColorSpaceXform::kRGBA_F32_ColorFormat, + SkColorSpaceXform::kBGRA_8888_ColorFormat, kOpaque_SkAlphaType); } SkString onShortName() override { |