[Webkit-unassigned] [Bug 35957] New: [Qt] Do not use QImageReader::imageCount in QtWebKit

[bugzilla-daemon at webkit.org]

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

           Summary: [Qt] Do not use QImageReader::imageCount in QtWebKit
           Product: WebKit
           Version: 528+ (Nightly build)
          Platform: PC
        OS/Version: Mac OS X 10.5
            Status: NEW
          Severity: Normal
          Priority: P2
         Component: Images
        AssignedTo: webkit-unassigned at lists.webkit.org
        ReportedBy: zecke at selfish.org


In theory decoding animations should work like the following. We determine the
number of frames and then WebCore starts animating and asks us for frame 1, 2,
3, 4, ..., 20 and the throw away some images to save memory 1-8, and when we
are at the end n, we have to go back to image 1.

In Qt terms this would work by calling QImageReader::imageCount and then using
QImageReader::jumpToImage. Qt 4.7 has gained the abilitiy to index the gif and
determine the number of images but jumpToImage is not implemented. This would
mean we need to add error handling to the ImageDecoderQt.cpp in case we can not
jumpToImage. On top of that the jumpToImage will not be implemented for gif.

Besides the error handling we would need to add code to detect if we access the
image in order, and if we don't access it in order recreate the QImageReader.
E.g. this would happen if we are at image n of a large animation and need to
get back to 'n'.

All this will make the ImageDecoderQt.cpp a lot more complicated without any
real gain.

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