Bug 13220

Summary:

[S60] Browser doesn't support for embedded DRM image content.

Product:

WebKit

Reporter:

Hong Zhao <hong.a.zhao>

Component:

Images

Assignee:

Nobody <webkit-unassigned>

Status:

CLOSED FIXED

Severity:

Critical

Keywords:

PlatformOnly

Priority:

Version:

523.x (Safari 3)

Hardware:

S60 Emulator

OS:

S60 3rd edition

Attachments:

Description	Flags
Add DRM image decoder support for OSS Browser.	yongjun.zhang: review-
Add DRM image decode support for 3.2	none
Make some changes based on the review comments.	yongjun.zhang: review-
Remove the redundant part in change log block.	yongjun.zhang: review-
Add DRM image decode function for 3.1m.	yongjun.zhang: review+
Add function to decode DRM image content in 3.2.	yongjun.zhang: review+

Description Hong Zhao 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

Comment 1 Hong Zhao 2007-03-28 14:54:45 PDT

Created attachment 13851 [details]
Add DRM image decoder support for OSS Browser.

Comment 2 Hong Zhao 2007-03-28 15:09:27 PDT

Created attachment 13852 [details]
Add DRM image decode support for 3.2

Comment 3 Yongjun Zhang 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.

Comment 4 Hong Zhao 2007-03-29 09:19:23 PDT

Created attachment 13864 [details]
Make some changes based on the review comments.

Comment 5 Yongjun Zhang 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.

Comment 6 Hong Zhao 2007-03-29 09:46:09 PDT

Created attachment 13865 [details]
Remove the redundant part in change log block.

Comment 7 Yongjun Zhang 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?

Comment 8 Hong Zhao 2007-03-29 10:41:23 PDT

Created attachment 13867 [details]
Add DRM image decode function for 3.1m.

Comment 9 Yongjun Zhang 2007-03-29 10:51:05 PDT

Comment on attachment 13867 [details]
Add DRM image decode function for 3.1m.

r=me

Comment 10 Yongjun Zhang 2007-03-29 10:51:43 PDT

Committed on revision 20576.

Comment 11 Hong Zhao 2007-03-29 11:51:40 PDT

Created attachment 13872 [details]
Add function to decode DRM image content in 3.2.

Comment 12 Yongjun Zhang 2007-03-29 13:21:41 PDT

Comment on attachment 13872 [details]
Add function to decode DRM image content in 3.2.

r = me

Comment 13 Bradley Morrison 2008-04-09 11:39:46 PDT

Bulk closing of all s60 platform bugs. 

Sorry for the noise!

Comment 14 Joel Parks 2011-03-21 11:53:18 PDT

re-purposing InTSW keyword for use by QtWebkit team