Bug 93465 - Set m_isComplete to true in BitmapImage(NativeImagePtr, ImageObserver*).
Summary: Set m_isComplete to true in BitmapImage(NativeImagePtr, ImageObserver*).
Status: UNCONFIRMED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Images (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Nobody
URL:
Keywords:
Depends on:
Blocks: 90375
  Show dependency tree / graph
 
Reported: 2012-08-08 05:02 PDT by Dongseong Hwang
Modified: 2012-08-08 22:01 PDT (History)
5 users (show)

See Also:

Attachments
Patch (4.39 KB, patch)
2012-08-08 05:04 PDT, Dongseong Hwang 		no flags Details | Formatted Diff | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Dongseong Hwang 2012-08-08 05:02:25 PDT 
This constructor creates a BitmapImage from a platform specific NativeImagePtr.
It sets m_frame of the first frame to the given NativeImagePtr.
The BitmapImage created does not need decoding because decoding is already done.
So, it is proper to set m_isComplete to true. Currently, only OpenVG
port sets m_isComplete to true.

This change is needed for parallel image decoders. Currently,
BitmapImage checks only m_frame in order to determine whether decoding
is needed or not. But after parallel image decoders are landed,
BitmapImage also needs to check m_isComplete. If m_isComplete is false,
BitmapImage will try to decode again, and it can't actually decode
because the BitmapImage created from a NativeImagePtr does not have
encoded data.
Comment 1 Dongseong Hwang 2012-08-08 05:04:37 PDT 
Created attachment 157185 [details]
Patch
Comment 2 Hin-Chung Lam 2012-08-08 20:15:07 PDT 
I suggest we take down the review flag at the moment. It is unclear why we need this because refactoring for ImageSource and BitmapImage hasn't landed yet. We'll also need test for this change.
Comment 3 Dongseong Hwang 2012-08-08 22:01:52 PDT 
(In reply to comment #2)
> I suggest we take down the review flag at the moment. It is unclear why we need this because refactoring for ImageSource and BitmapImage hasn't landed yet. We'll also need test for this change.

I came close to turn on the review flag of parallel image decoders and RetainedModeBitmapImage. I agreed on you.