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

Issue 632350 link

Starred by 2 users

Issue metadata

Status: WontFix
Owner:
Last visit > 30 days ago
Closed: Jan 2018
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: All
Pri: 3
Type: Bug



Sign in to add a comment

Try to avoid swizzle in createImageBitmap(ImageData)

Project Member Reported by xidac...@chromium.org, Jul 28 2016

Issue description

In the case of premultiplyAlpha = false, creaetImageBitmap(ImageData) creates a SkImage from ImageData directly. Now because ImageData is RBGA format, and in some platform SkImage supports BGRA format, so we have to swizzle the data. We should try to avoid that cost.
 
Cc: junov@chromium.org
Owner: ----
Status: Available (was: Assigned)
Project Member

Comment 2 by sheriffbot@chromium.org, Nov 29 2017

Labels: Hotlist-Recharge-Cold
Status: Untriaged (was: Available)
This issue has been Available for over a year. If it's no longer important or seems unlikely to be fixed, please consider closing it out. If it is important, please re-triage the issue.

Sorry for the inconvenience if the bug really should have been left as Available. If you change it back, also remove the "Hotlist-Recharge-Cold" label.

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
Components: Blink>Canvas

Comment 4 by junov@chromium.org, Dec 6 2017

Owner: zakerinasab@chromium.org
Status: Assigned (was: Untriaged)
IIUC there is no way that we can avoid swizzling. Even if we avoid swizzling in createImageBitmap(ImageData) and create the SkImage in RGBA mode, assuming that the system uses BGRA order, the image will get converted to BGRA inside Skia whenever it is drawn on to a BGRA surface/canvas. In fact by postponing the conversion to what matches the system expectation, we might end up with multiple swizzlings if the user uses/draws the image multiple times.

I think this should be a WontFix.

Comment 6 by junov@chromium.org, Jan 10 2018

Status: WontFix (was: Assigned)
Makes Sense. -> WontFix

Sign in to add a comment