Remove opaque alpha channel special case

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

 /*
  * Copyright (C) 2006 Apple Computer, Inc.
  *
  * Portions are Copyright (C) 2001-6 mozilla.org
  *
  * Other contributors:
  *   Stuart Parmenter <stuart@mozilla.com>
  *
  * This library is free software; you can redistribute it and/or
  * modify it under the terms of the GNU Lesser General Public
@@ skipping 927 matching lines @@
               static_cast<JDIMENSION>(m_decodedSize.height()));
 
     if (!buffer.allocatePixelData(info->output_width, info->output_height,
                                   colorSpaceForSkImages()))
       return setFailed();
 
     buffer.zeroFillPixelData();
     // 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::FramePartial);
-    buffer.setHasAlpha(true);

[Peter Kasting, 2017/03/22 23:37:47]

I think this already got removed in another CL?

[cblume, 2017/05/08 10:28:02]

I don't see it in any of my other CLs after a quick look.
The one that I think might have been closest is
https://codereview.chromium.org/2762643004/

 
     // For JPEGs, the frame always fills the entire image.
     buffer.setOriginalFrameRect(IntRect(IntPoint(), size()));
   }
 
 #if defined(TURBO_JPEG_RGB_SWIZZLE)
   if (turboSwizzled(info->out_color_space)) {
     while (info->output_scanline < info->output_height) {
       unsigned char* row = reinterpret_cast_ptr<unsigned char*>(
           buffer.getAddr(0, info->output_scanline));
@@ skipping 51 matching lines @@
   // has failed.
   if (!m_reader->decode(onlySize) && isAllDataReceived())
     setFailed();
 
   // If decoding is done or failed, we don't need the JPEGImageReader anymore.
   if (isComplete(this, onlySize) || failed())
     m_reader.reset();
 }
 
 }  // namespace blink