[Webkit-unassigned] [Bug 33213] Make png image decoder work with segmented SharedBuffer

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=33213


Yong Li <yong.li.webkit at gmail.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |darin at apple.com




--- Comment #12 from Yong Li <yong.li.webkit at gmail.com>  2010-01-06 10:48:27 PST ---
Darin, there's a bug in yesterday's commit for png decoder:

I move "if (!m_hasFinishedDecoding && decoder->isAllDataReceived())" out of the
loop, but it hurts another "break".

        while (unsigned segmentLength = data.getSomeData(segment,
m_readOffset)) {
            m_readOffset += segmentLength;
            m_currentBufferSize = m_readOffset;
            png_process_data(m_png, m_info,
reinterpret_cast<png_bytep>(const_cast<char*>(segment)), segmentLength);
            if ((sizeOnly && decoder->isSizeAvailable()) ||
m_hasFinishedDecoding)
                break;
        }
        if (!m_hasFinishedDecoding && decoder->isAllDataReceived())
            decoder->pngComplete();

When "sizeonly" is true, decoder->pngComplete() shouldn't be called.

Should I post a patch in this bug, or raise a new bug?

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.