New issue
Advanced search Search tips

Issue 879640 link

Starred by 2 users

Issue metadata

Status: Duplicate
Merged: issue 776800
Owner: ----
Closed: Oct 31
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux , Android , Windows , Chrome , Mac
Pri: 2
Type: Task



Sign in to add a comment

Use OOP raster for canvas 2d.

Project Member Reported by khushals...@chromium.org, Aug 31

Issue description

This is a placeholder bug for all work required to enable OOP raster on specifically for canvas 2d.
 
I thought I filed a bug for this, but I can't find this.

Things that need to be handled:
(1) Add readback into mailbox in raster decoder
(2) OOP-R serialization needs to handle drawImage(mailbox) (or paint image has a mailbox)
(3) There might be some issues with GL or Skia assumptions baked into some of the paint code that would need to be extracted, e.g. PaintCanvasVideoRenderer.
Cc: backer@chromium.org
This also requires using raster decoder for the shared renderer main thread context, that is also used by media.

+backer, is there a list of things remaining that raster decoder needs to be support, specifically for media? I remember we had talked about some additional work here that we could punt on since currently it was only going to be used by the compositor worker context.
FYI, there's already a bug for this: https://bugs.chromium.org/p/chromium/issues/detail?id=776800
Re #2:

I don't really know what is required to make RasterInterface/RasterDecoder work for media. I did briefly start down this path and my recollection is that video is still using GLES2/GrContext for texture allocation, video copying, and color space conversion.

We'll address allocation with shareables. RasterInterface/RasterDecoder will need to be updated for the rest.
Mergedinto: 776800
Status: Duplicate (was: Available)

Sign in to add a comment