Index: src/codec/SkCodecImageGenerator.cpp |
diff --git a/src/codec/SkCodecImageGenerator.cpp b/src/codec/SkCodecImageGenerator.cpp |
index 2548a006299f71efb612fbb78b8d2583ec89bffa..e579da92f6f435c8a34ba21aae5fdfb02f2cbe23 100644 |
--- a/src/codec/SkCodecImageGenerator.cpp |
+++ b/src/codec/SkCodecImageGenerator.cpp |
@@ -6,7 +6,6 @@ |
*/ |
#include "SkCodecImageGenerator.h" |
-#include "SkPM4fPriv.h" |
SkImageGenerator* SkCodecImageGenerator::NewFromEncodedCodec(SkData* data) { |
SkCodec* codec = SkCodec::NewFromData(data); |
@@ -17,27 +16,16 @@ SkImageGenerator* SkCodecImageGenerator::NewFromEncodedCodec(SkData* data) { |
return new SkCodecImageGenerator(codec, data); |
} |
-// FIXME: We should expose information about the encoded format on the |
-// SkImageGenerator, so the client can interpret the encoded |
-// format and request an output format. For now, as a workaround, |
-// we guess what output format the client wants. |
-static SkImageInfo fix_info(const SkCodec& codec) { |
- const SkImageInfo& info = codec.getInfo(); |
- SkAlphaType alphaType = (kUnpremul_SkAlphaType == info.alphaType()) ? kPremul_SkAlphaType : |
- info.alphaType(); |
- |
- SkColorProfileType profileType = kLinear_SkColorProfileType; |
- // Crudely guess that the presence of a color space means sRGB, or obey the global sRGB |
- // selector. |
- if (gTreatSkColorAsSRGB || codec.getColorSpace()) { |
- profileType = kSRGB_SkColorProfileType; |
+static SkImageInfo make_premul(const SkImageInfo& info) { |
+ if (kUnpremul_SkAlphaType == info.alphaType()) { |
+ return info.makeAlphaType(kPremul_SkAlphaType); |
} |
- return SkImageInfo::Make(info.width(), info.height(), info.colorType(), alphaType, profileType); |
+ return info; |
} |
SkCodecImageGenerator::SkCodecImageGenerator(SkCodec* codec, SkData* data) |
- : INHERITED(fix_info(*codec)) |
+ : INHERITED(make_premul(codec->getInfo())) |
, fCodec(codec) |
, fData(SkRef(data)) |
{} |