New issue
Advanced search Search tips
Note: Color blocks (like █ or █) mean that a user may not be available. Tooltip shows the reason.

Issue 886999 link

Starred by 1 user
Status: Duplicate
Merged: issue 887015
Owner:
█ zakerinasab@chromium.org
Last visit > 30 days ago
Closed: Sep 19
Cc:
fs...@chromium.org
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug

Blocking:
issue 634542



Sign in to add a comment

ImageBitmap color space conversion option when source is ImageData

Project Member Reported by zakerinasab@chromium.org, Sep 19 
ImageData as a source does not carry any color space information. It's not very clear to me how color space conversion option should be applied when creating an ImageBitmap from an ImageData.

We also need to clarify what to do with the sRGB transfer function if we go with sRGB as the default color space for ImageData (at least when it is used as the source for ImageBitmap creation).

Comment 1 by zakerinasab@chromium.org, Sep 19 

As far as ImageData is concerned, color space can be either sRGB or linear-RGB. Since ImageData is unpremul, both of these assumptions should work. For example, in either case, drawing the result ImageData on a P3 surface should give the same result.

This raises another question though. Since ImageData always is assumed to be in canvas color space, how we should keep the color space of the ImageData attched to it? Do we need a new ImageData object with color space information?
I move this to a new bug.

Comment 2 by zakerinasab@chromium.org, Sep 19

Status: Fixed (was: Assigned)
For now the color space of the image data when used as the source of the is internally set to sRGB. This also matches the default color space value in ImageDataColorSettings.

Comment 3 by zakerinasab@chromium.org, Sep 19 

* as the source of ImageBitmap *

Comment 4 by zakerinasab@chromium.org, Sep 19

Status: Assigned (was: Fixed)

Comment 5 by zakerinasab@chromium.org, Sep 19 

Setting the default color space to sRGB fails a couple of gpu layout tests. Keeping this open for now.

Comment 6 by zakerinasab@chromium.org, Sep 19

Mergedinto: 887015
Status: Duplicate (was: Assigned)
►

Sign in to add a comment