CLOSED FIXED 13220
[S60] Browser doesn't support for embedded DRM image content. 
https://bugs.webkit.org/show_bug.cgi?id=13220
Summary [S60] Browser doesn't support for embedded DRM image content.
Hong Zhao
Reported 2007-03-28 14:18:07 PDT
Some images crash when browsing some wap page. These images are converted to DRM content by server. They are not supported in the current Browser. So Browser displays the broken images. TSW# ID: TKWG-6Z59YX
Attachments
Add DRM image decoder support for OSS Browser. (9.42 KB, patch)
2007-03-28 14:54 PDT, Hong Zhao 		yongjun.zhang: review-
DetailsFormatted DiffDiff
Add DRM image decode support for 3.2 (5.95 KB, patch)
2007-03-28 15:09 PDT, Hong Zhao 		no flags
DetailsFormatted DiffDiff
Make some changes based on the review comments. (10.35 KB, patch)
2007-03-29 09:19 PDT, Hong Zhao 		yongjun.zhang: review-
DetailsFormatted DiffDiff
Remove the redundant part in change log block. (9.78 KB, patch)
2007-03-29 09:46 PDT, Hong Zhao 		yongjun.zhang: review-
DetailsFormatted DiffDiff
Add DRM image decode function for 3.1m. (6.34 KB, patch)
2007-03-29 10:41 PDT, Hong Zhao 		yongjun.zhang: review+
DetailsFormatted DiffDiff
Add function to decode DRM image content in 3.2. (6.24 KB, patch)
2007-03-29 11:51 PDT, Hong Zhao 		yongjun.zhang: review+
DetailsFormatted DiffDiff
Show Obsolete (4) View All Add attachment proposed patch, testcase, etc.
Hong Zhao
Comment 1 2007-03-28 14:54:45 PDT
Created attachment 13851 [details] Add DRM image decoder support for OSS Browser.
Hong Zhao
Comment 2 2007-03-28 15:09:27 PDT
Created attachment 13852 [details] Add DRM image decode support for 3.2
Yongjun Zhang
Comment 3 2007-03-29 07:06:53 PDT
Comment on attachment 13851 [details] Add DRM image decoder support for OSS Browser. The fix is incorrect: 1. function name "DisplayDRMImageL" needs to be something like "DecodeDRMImageL" because you just decode DRM image in this function. 2. the condition to trigger DRM encoding is wrong - any error image (no matter is drm or now) will trigger DRM decoding. 3. Performance is very bad because DisplayDRMImageL is called very time when a decoding error occurs. 4. Don't use sychrous decoding, WaitForRequest will block the active object and the whole image deocoder halts. 5. The whole idea of StaticImagedecoder is to reuse the Bufferdecoder, should not create another buffered decoder for drm images.
Hong Zhao
Comment 4 2007-03-29 09:19:23 PDT
Created attachment 13864 [details] Make some changes based on the review comments.
Yongjun Zhang
Comment 5 2007-03-29 09:41:15 PDT
Comment on attachment 13864 [details] Make some changes based on the review comments. remove the redundant change log block.
Hong Zhao
Comment 6 2007-03-29 09:46:09 PDT
Created attachment 13865 [details] Remove the redundant part in change log block.
Yongjun Zhang
Comment 7 2007-03-29 10:04:02 PDT
Comment on attachment 13865 [details] Remove the redundant part in change log block. why webkit.mmp is changed from __GCCE__ to __BROWSER_SDK?
Hong Zhao
Comment 8 2007-03-29 10:41:23 PDT
Created attachment 13867 [details] Add DRM image decode function for 3.1m.
Yongjun Zhang
Comment 9 2007-03-29 10:51:05 PDT
Comment on attachment 13867 [details] Add DRM image decode function for 3.1m. r=me
Yongjun Zhang
Comment 10 2007-03-29 10:51:43 PDT
Committed on revision 20576.
Hong Zhao
Comment 11 2007-03-29 11:51:40 PDT
Created attachment 13872 [details] Add function to decode DRM image content in 3.2.
Yongjun Zhang
Comment 12 2007-03-29 13:21:41 PDT
Comment on attachment 13872 [details] Add function to decode DRM image content in 3.2. r = me
Bradley Morrison
Comment 13 2008-04-09 11:39:46 PDT
Bulk closing of all s60 platform bugs. Sorry for the noise!
Joel Parks
Comment 14 2011-03-21 11:53:18 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.