<html>
    <head>
      <base href="https://bugs.webkit.org/">
    </head>
    <body>
      <p>
        <div>
            <b><a class="bz_bug_link 
          bz_status_NEW "
   title="NEW - JPEGImageDecoder: use libjpeg-turbo RGBA output path even for Adobe transform=0 JPEGs"
   href="https://bugs.webkit.org/show_bug.cgi?id=200163#c18">Comment # 18</a>
              on <a class="bz_bug_link 
          bz_status_NEW "
   title="NEW - JPEGImageDecoder: use libjpeg-turbo RGBA output path even for Adobe transform=0 JPEGs"
   href="https://bugs.webkit.org/show_bug.cgi?id=200163">bug 200163</a>
              from <span class="vcard"><a class="email" href="mailto:loic.yhuel@softathome.com" title="Loïc Yhuel <loic.yhuel@softathome.com>"> <span class="fn">Loïc Yhuel</span></a>
</span></b>
        <pre>(In reply to Fujii Hironori from <a href="show_bug.cgi?id=200163#c16">comment #16</a>)
<span class="quote">> Comment on <span class=""><a href="attachment.cgi?id=374964&action=diff" name="attach_374964" title="Patch">attachment 374964</a> <a href="attachment.cgi?id=374964&action=edit" title="Patch">[details]</a></span>
> Patch

> View in context:
> <a href="https://bugs.webkit.org/attachment.cgi?id=374964&action=review">https://bugs.webkit.org/attachment.cgi?id=374964&action=review</a>

> > Source/WebCore/platform/image-decoders/jpeg/JPEGImageDecoder.cpp:302
> >                  m_info.out_color_space = rgbOutputColorSpace();

> It seems that JPEGImageDecoder::outputScanlines doesn't support the case
> out_color_space is BGRA and m_scaled is false.
> I think we should set out_color_space JCS_RGB if m_scaled.

>   m_info.out_color_space = m_scaled ? JCS_RGB : rgbOutputColorSpace();</span >

Yes, <a href="https://trac.webkit.org/r225091">r225091</a> left dead code which wasn't under the flag, but depended on the flag to work properly.

Btw, the generic downsampling code doesn't really makes sense for JPEG, since the library support fractional M/8 scaling (which is faster than full size decoding, let alone the additional manual scaling loop here).</pre>
        </div>
      </p>


      <hr>
      <span>You are receiving this mail because:</span>

      <ul>
          <li>You are the assignee for the bug.</li>
      </ul>
    </body>
</html>