Bug 13151

Summary:

[S60] Some images crash when browsing some wap page - TKWG-6Z59YX

Product:

WebKit

Reporter:

Hong Zhao <hong.a.zhao>

Component:

Images

Assignee:

Nobody <webkit-unassigned>

Status:

CLOSED FIXED

Severity:

Normal

Keywords:

PlatformOnly

Priority:

Version:

523.x (Safari 3)

Hardware:

S60 Emulator

OS:

S60 3rd edition

Attachments:

Description	Flags
Remove extra content-type strings that server added before passing to image decoder.	zalan: review-
Remove the extra content-type strings that server added before passing the image to decoder.	zalan: review-
Remove some extra image content-type strings added by the server before passing it to the image decoder	Sachin.Padma: review+
Remove the extra image content-type strings added by the server before passing it to the image decoder	Sachin.Padma: review+

Hong Zhao

Reported 2007-03-21 17:10:38 PDT

TSW#: TKWG-6Z59YX Some servers will add feature "cfm: forward = no" for all images from this folder . Apart from this folder, all images could be displayed normally. It seems the OSS browser cannot display such image(forward=no). However, image could be saved and displayed when access the image link directly. The reason is that the server adds extra strings in reponse of header.

Attachments
Remove extra content-type strings that server added before passing to image decoder. (2.91 KB, patch) 2007-03-21 17:34 PDT, Hong Zhao	zalan: review-	Details Formatted Diff Diff
Remove the extra content-type strings that server added before passing the image to decoder. (5.45 KB, patch) 2007-03-21 17:46 PDT, Hong Zhao	zalan: review-	Details Formatted Diff Diff
Remove some extra image content-type strings added by the server before passing it to the image decoder (5.40 KB, patch) 2007-03-21 21:59 PDT, Hong Zhao	Sachin.Padma: review+	Details Formatted Diff Diff
Remove the extra image content-type strings added by the server before passing it to the image decoder (2.02 KB, patch) 2007-03-21 22:07 PDT, Hong Zhao	Sachin.Padma: review+	Details Formatted Diff Diff
Show Obsolete (2) View All Add attachment proposed patch, testcase, etc.

Hong Zhao

Comment 1 2007-03-21 17:34:32 PDT

Created attachment 13753 [details] Remove extra content-type strings that server added before passing to image decoder.

Hong Zhao

Comment 2 2007-03-21 17:46:18 PDT

Created attachment 13755 [details] Remove the extra content-type strings that server added before passing the image to decoder.

zalan

Comment 3 2007-03-21 17:48:13 PDT

Comment on attachment 13753 [details] Remove extra content-type strings that server added before passing to image decoder. i can't find KExtraAdded in the patch. Could you rename it to something like KSemicolonDelimiter and bfExtraStrLen to bfExtraStrPos. Thanks. btw the changelog file needs fixing.

Hong Zhao

Comment 4 2007-03-21 21:59:49 PDT

Created attachment 13758 [details] Remove some extra image content-type strings added by the server before passing it to the image decoder

Hong Zhao

Comment 5 2007-03-21 22:07:09 PDT

Created attachment 13759 [details] Remove the extra image content-type strings added by the server before passing it to the image decoder

Sachin Padma

Comment 6 2007-03-23 07:41:30 PDT

Comment on attachment 13758 [details] Remove some extra image content-type strings added by the server before passing it to the image decoder r=me

Sachin Padma

Comment 7 2007-03-23 07:42:47 PDT

Comment on attachment 13759 [details] Remove the extra image content-type strings added by the server before passing it to the image decoder r=me

Sachin Padma

Comment 8 2007-03-23 08:13:32 PDT

commited in r20431 in 3.1m and r20432 in ccb.

Bradley Morrison

Comment 9 2008-04-09 11:39:47 PDT

Bulk closing of all s60 platform bugs. Sorry for the noise!

Joel Parks

Comment 10 2011-03-21 11:53:16 PDT

re-purposing InTSW keyword for use by QtWebkit team

Note You need to log in before you can comment on or make changes to this bug.