Mark webps as sRGB
This may cause some diffs in Gold.
BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1838603002
Review URL: https://codereview.chromium.org/1838603002
diff --git a/src/codec/SkWebpCodec.cpp b/src/codec/SkWebpCodec.cpp
index a795fdd..77af8e5 100644
--- a/src/codec/SkWebpCodec.cpp
+++ b/src/codec/SkWebpCodec.cpp
@@ -85,9 +85,10 @@
// This version is slightly different from SkCodecPriv's version of conversion_possible. It
// supports both byte orders for 8888.
static bool webp_conversion_possible(const SkImageInfo& dst, const SkImageInfo& src) {
- if (dst.profileType() != src.profileType()) {
- return false;
- }
+ // FIXME: skbug.com/4895
+ // Currently, we ignore the SkColorProfileType on the SkImageInfo. We
+ // will treat the encoded data as linear regardless of what the client
+ // requests.
if (!valid_alpha(dst.alphaType(), src.alphaType())) {
return false;
@@ -252,4 +253,6 @@
}
SkWebpCodec::SkWebpCodec(const SkImageInfo& info, SkStream* stream)
- : INHERITED(info, stream) {}
+ // The spec says an unmarked image is sRGB, so we return that space here.
+ // TODO: Add support for parsing ICC profiles from webps.
+ : INHERITED(info, stream, SkColorSpace::NewNamed(SkColorSpace::kSRGB_Named)) {}