Bug 13151

Summary: [S60] Some images crash when browsing some wap page - TKWG-6Z59YX
Product: WebKit Reporter: Hong Zhao <hong.a.zhao>
Component: ImagesAssignee: Nobody <webkit-unassigned>
Status: CLOSED FIXED    
Severity: Normal Keywords: PlatformOnly
Priority: P2    
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+

Description Hong Zhao 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.
Comment 1 Hong Zhao 2007-03-21 17:34:32 PDT
Created attachment 13753 [details]
Remove extra content-type strings that server added before passing to image decoder.
Comment 2 Hong Zhao 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.
Comment 3 zalan 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.
Comment 4 Hong Zhao 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
Comment 5 Hong Zhao 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
Comment 6 Sachin Padma 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
Comment 7 Sachin Padma 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
Comment 8 Sachin Padma 2007-03-23 08:13:32 PDT
commited in r20431 in 3.1m and r20432 in ccb.
Comment 9 Bradley Morrison 2008-04-09 11:39:47 PDT
Bulk closing of all s60 platform bugs. 

Sorry for the noise!
Comment 10 Joel Parks 2011-03-21 11:53:16 PDT
re-purposing InTSW keyword for use by QtWebkit team