Bug 70666 – BitmapImage::dataChanged() does not reset enough incomplete frames

• Status: RESOLVED
• Resolution: FIXED
• Component: Images

[Peter Kasting]

Created attachment 112060 [details]
Manual testcase

BitmapData::dataChanged() assumes (unconditionally!) that the last frame it currently has in |m_frames| is affected when more data arrives, and none of the others are.  This is not true for ICO files, where we may have a variety of different frames (due to different parts of the browser asking for different sizes) but any or all of them may be incompletely decoded and may need to have their cached metadata reset.  Therefore, this needs to be fixed to iterate through |m_frames| and clear all incomplete ones.

Technically, my fix will also save us from unnecessarily clearing the last frame in the list (for any image type) if it was already "complete", although since the underlying decoder often caches this, I doubt that will have any noticeable perf impact.

Attached is a manual testcase that repros the problem for me if opened in an incognito Chrome window -- pressing "play" shows a partial Ars Technica logo.  I specify an incognito window because this way you can close the window and then open a new one to flush the cache, without which you won't reproduce the bug after one try (unless you manually clear the cache).

[Peter Kasting]

Created attachment 112061 [details]
patch v1

[WebKit Review Bot]

Attachment 112061 [details] did not pass style-queue:

Failed to run "['Tools/Scripts/check-webkit-style', '--diff-files', u'Source/WebCore/ChangeLog', u'Source/WebCor..." exit_code: 1

Source/WebCore/platform/graphics/BitmapImage.cpp:198:  Should have only a single space after a punctuation in a comment.  [whitespace/comments] [5]
Source/WebCore/platform/graphics/BitmapImage.cpp:199:  Should have only a single space after a punctuation in a comment.  [whitespace/comments] [5]
Source/WebCore/platform/graphics/BitmapImage.cpp:201:  Should have only a single space after a punctuation in a comment.  [whitespace/comments] [5]
Source/WebCore/platform/graphics/BitmapImage.cpp:213:  Should have only a single space after a punctuation in a comment.  [whitespace/comments] [5]
Total errors found: 4 in 2 files


If any of these errors are false positives, please file a bug against check-webkit-style.

[Peter Kasting]

Created attachment 112062 [details]
patch v2

Hate that style rule

[James Robinson]

Comment on attachment 112062 [details]
patch v2

R=me

[WebKit Review Bot]

Comment on attachment 112062 [details]
patch v2

Clearing flags on attachment: 112062

Committed r98930: <http://trac.webkit.org/changeset/98930>

[WebKit Review Bot]

All reviewed patches have been landed.  Closing bug.