| Index: Source/core/platform/image-decoders/png/PNGImageDecoder.cpp
|
| diff --git a/Source/core/platform/image-decoders/png/PNGImageDecoder.cpp b/Source/core/platform/image-decoders/png/PNGImageDecoder.cpp
|
| index 828328b956c2bb0c8d8d8de534228c2abee60223..b039e3793e09f161d43a9dcadcf149d39a8e8b03 100644
|
| --- a/Source/core/platform/image-decoders/png/PNGImageDecoder.cpp
|
| +++ b/Source/core/platform/image-decoders/png/PNGImageDecoder.cpp
|
| @@ -489,18 +489,30 @@ void PNGImageDecoder::rowAvailable(unsigned char* rowBuffer, unsigned rowIndex,
|
|
|
| // Write the decoded row pixels to the frame buffer.
|
| ImageFrame::PixelData* address = buffer.getAddr(0, y);
|
| - bool nonTrivialAlpha = false;
|
| + unsigned alphaMask = 255;
|
| int width = size().width();
|
|
|
| + // Do not merge the loops below. They are hand rolled for each case for maximal performance.
|
| png_bytep pixel = row;
|
| - for (int x = 0; x < width; ++x, pixel += colorChannels) {
|
| - unsigned alpha = hasAlpha ? pixel[3] : 255;
|
| - buffer.setRGBA(address++, pixel[0], pixel[1], pixel[2], alpha);
|
| - nonTrivialAlpha |= alpha < 255;
|
| + if (hasAlpha) {
|
| + if (buffer.premultiplyAlpha()) {
|
| + for (int x = 0; x < width; ++x, pixel += 4) {
|
| + buffer.setRGBAPremultiply(address + x, pixel[3], pixel[0], pixel[1], pixel[2]);
|
| + alphaMask &= pixel[3];
|
| + }
|
| + } else {
|
| + for (int x = 0; x < width; ++x, pixel += 4) {
|
| + buffer.setRGBANoPremultiply(address + x, pixel[3], pixel[0], pixel[1], pixel[2]);
|
| + alphaMask &= pixel[3];
|
| + }
|
| + }
|
| + } else {
|
| + for (int x = 0; x < width; ++x, pixel += 3)
|
| + buffer.setRGB(address + x, pixel[0], pixel[1], pixel[2]);
|
| }
|
|
|
| - if (nonTrivialAlpha && !buffer.hasAlpha())
|
| - buffer.setHasAlpha(nonTrivialAlpha);
|
| + bool hasNonTrivialAlpha = alphaMask != 255;
|
| + buffer.setHasAlpha(hasNonTrivialAlpha);
|
| }
|
|
|
| void PNGImageDecoder::pngComplete()
|
|
|