Summary: | [Regression] PNG-in-ICO decoding in public image decoders broken | ||||||||
---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Peter Kasting <pkasting> | ||||||
Component: | Images | Assignee: | Peter Kasting <pkasting> | ||||||
Status: | RESOLVED FIXED | ||||||||
Severity: | Normal | CC: | abarth, eric, webkit.review.bot, yong.li.webkit | ||||||
Priority: | P2 | ||||||||
Version: | 528+ (Nightly build) | ||||||||
Hardware: | PC | ||||||||
OS: | All | ||||||||
Attachments: |
|
Description
Peter Kasting
2010-01-06 10:49:21 PST
sorry it's my bug. see line 121 in PNGImageDecoder.cpp: if ((sizeOnly && decoder->isSizeAvailable()) || m_hasFinishedDecoding) break; just chaning "break" to "return" should fix it. -Yong Created attachment 45973 [details]
the candidate patch
Attachment 45973 [details] did not pass style-queue:
Failed to run "WebKitTools/Scripts/check-webkit-style" exit_code: 1
Traceback (most recent call last):
File "WebKitTools/Scripts/check-webkit-style", line 98, in <module>
main()
File "WebKitTools/Scripts/check-webkit-style", line 62, in main
defaults = style.ArgumentDefaults(style.DEFAULT_OUTPUT_FORMAT,
AttributeError: 'module' object has no attribute 'ArgumentDefaults'
Sorry for the spam. Stylebot is sick. Looks like this caused crashes: http://build.webkit.org/builders/Leopard%20Intel%20Debug%20%28Tests%29/builds/8928 http://build.webkit.org/results/Leopard%20Intel%20Debug%20(Tests)/r52862%20(8928)/results.html svg/W3C-SVG-1.1/linking-a-01-b.svg I think we'll need to roll this out. :( http://trac.webkit.org/changeset/52862 was the original commit. Do we not have a way to test image decoders? I'm surprised this landed w/o a test case? Yong doesn't seem to be around on IRC, so I'm rolling this out in hopes it will fix the crashes on the bot. Rolled out in http://trac.webkit.org/changeset/52871. Peter Kasting notes that we may need to roll out r52831 as well, otherwise this breaks chromium. This change should have been unrelated. If it's found to be unrelated, I'll roll it back in. I'm rolling this back in after further discussions with Peter and Yong. I'm testing it on Leopard Debug first to make sure. The leopard build is broken at the moment due to http://trac.webkit.org/changeset/52872, so I can't land the re-land yet. Committed r52880: <http://trac.webkit.org/changeset/52880> |