Support decoding opaque to *premul
authorscroggo <scroggo@google.com>
Wed, 3 Feb 2016 17:42:42 +0000 (09:42 -0800)
committerCommit bot <commit-bot@chromium.org>
Wed, 3 Feb 2016 17:42:42 +0000 (09:42 -0800)
commitc5560bef14149f4c945a4536988aeba1a16adedc
treed62cecc289bb7c0ad9083258669179a64fd87d59
parent8ca88e41aa76bc4da568936de9299ec3f8762d9c
Support decoding opaque to *premul

If a client requests unpremul or premul from an opaque SkCodec,
support it. The opaque image can be treated as any of them, though
it will be less efficient to draw than if the client had used
opaque.

Change the filling code (i.e. for incomplete images) to base its color on
the source alpha type. Prior to adding the support to decode opaque to
any, it was fine to use either source or dest (which would have yielded
the same result). If the client requests non-opaque, we do not want this
to switch the fill value from black to transparent. This also allows
simplifying the signatures for getFillValue and onGetFillValue.

In CodexTest, expect the same result when decoding opaque to *premul,
and compare to the opaque version.

BUG=skia:4616
GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1641273003

Review URL: https://codereview.chromium.org/1641273003
16 files changed:
dm/DM.cpp
dm/DMSrcSink.cpp
dm/DMSrcSink.h
include/codec/SkCodec.h
src/codec/SkBmpStandardCodec.cpp
src/codec/SkBmpStandardCodec.h
src/codec/SkCodec.cpp
src/codec/SkCodecPriv.h
src/codec/SkGifCodec.cpp
src/codec/SkGifCodec.h
src/codec/SkJpegCodec.cpp
src/codec/SkMaskSwizzler.cpp
src/codec/SkPngCodec.cpp
src/codec/SkPngCodec.h
src/codec/SkSampledCodec.cpp
tests/CodexTest.cpp