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

Issue 776808 link

Starred by 1 user

Issue metadata

Status: WontFix
Owner:
Last visit > 30 days ago
Closed: Nov 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Task

Blocking:
issue 776800



Sign in to add a comment

Replace SkPicture with PaintOpBuffer in Canvas2DLayerBridge

Project Member Reported by xlai@chromium.org, Oct 20 2017

Issue description

Replace SkPicture with PaintOpBuffer in Canvas2DLayerBridge
 

Comment 1 by xlai@chromium.org, Oct 20 2017

Blocking: 776800

Comment 2 by xlai@chromium.org, Nov 14 2017

Cc: enne@chromium.org
enne@ is working in replacing Blink and ui’s use of SkPicture with a new custom display list backing to store recorded paint commands while keeping the painting API mostly unchanged. In this big CL written by her (https://chromium.googlesource.com/chromium/src/+/34f6084cd35016a633942588724e6dd7dae428a3), it looks like Canvas2DLayerBridge is just part of the many classes that had SkPicture replaced with PaintOpBuffer already (via an alias "PaintRecord" as seen in https://cs.chromium.org/chromium/src/cc/paint/paint_record.h?l=17).

I guess there is not much we need to do for this CL then.

Comment 3 by junov@chromium.org, Nov 14 2017

Status: WontFix (was: Assigned)
Yeah, it looks like Canvas2DLayerBridge is already using PaintOpBuffers, via PaintRecord.  Let's just close this.

Sign in to add a comment