Remove opaque alpha channel special case

third_party/WebKit/Source/platform/image-decoders/jpeg/JPEGImageDecoder.cpp

Index: third_party/WebKit/Source/platform/image-decoders/jpeg/JPEGImageDecoder.cpp
diff --git a/third_party/WebKit/Source/platform/image-decoders/jpeg/JPEGImageDecoder.cpp b/third_party/WebKit/Source/platform/image-decoders/jpeg/JPEGImageDecoder.cpp
index 7e2aea53f3404b413f3a986dbc46d62d5995a37d..fc984920145d9758cd14ddf7e35ebae3f74ee9b3 100644
--- a/third_party/WebKit/Source/platform/image-decoders/jpeg/JPEGImageDecoder.cpp
+++ b/third_party/WebKit/Source/platform/image-decoders/jpeg/JPEGImageDecoder.cpp
@@ -943,7 +943,6 @@ bool JPEGImageDecoder::OutputScanlines() {
     // The buffer is transparent outside the decoded area while the image is
     // loading. The image will be marked fully opaque in Complete().
     buffer.SetStatus(ImageFrame::kFramePartial);
-    buffer.SetHasAlpha(true);

[scroggo_chromium, 2017/05/31 14:27:50]

This removal is related, but isn't called out in the change description. It
won't change anything (has_alpha_ is initialized to false, and the ImageFrame is
considered to have alpha until it's FrameComplete, regardless of has_alpha). Do
we already have a test that verifies this?

[cblume, 2017/08/10 20:52:12]

I just ran it before & after on the bots. It looks like this is not tested.

Aside from that, you are right that it is unrelated. It makes more sense to
leave it and correctly mark a partial image as having transparency.

I've added it back in. Since that was the only change to this file, the file was
removed from the list of changes.

 
     // For JPEGs, the frame always fills the entire image.
     buffer.SetOriginalFrameRect(IntRect(IntPoint(), Size()));