Bug 13151 - [S60] Some images crash when browsing some wap page - TKWG-6Z59YX
Summary: [S60] Some images crash when browsing some wap page - TKWG-6Z59YX
Status: CLOSED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Images (show other bugs)
Version: 523.x (Safari 3)
Hardware: S60 Emulator S60 3rd edition
: P2 Normal
Assignee: Nobody
URL:
Keywords: PlatformOnly
Depends on:
Blocks:
 
Reported: 2007-03-21 17:10 PDT by Hong Zhao
Modified: 2011-03-21 11:53 PDT (History)
0 users

See Also:


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

Note You need to log in before you can comment on or make changes to this bug.
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