[Webkit-unassigned] [Bug 13220] [S60] Browser doesn't support for embedded DRM image content.

[bugzilla-daemon at webkit.org]

http://bugs.webkit.org/show_bug.cgi?id=13220


yongjun.zhang at nokia.com changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
  Attachment #13851|review?(yongjun.zhang at nokia.|review-
               Flag|com)                        |




------- Comment #3 from yongjun.zhang at nokia.com  2007-03-29 07:06 PDT -------
(From update of attachment 13851)
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.


-- 
Configure bugmail: http://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.