New issue
Advanced search Search tips

Issue 810531 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner: ----
Closed: Apr 2018
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 2
Type: Bug

Blocking:
issue 737629



Sign in to add a comment

oopr: fix serialization of PaintRecord-backed PaintImages

Project Member Reported by enne@chromium.org, Feb 8 2018

Issue description

PaintImage can be backed by a PaintRecord, via PaintImageBuilder::set_paint_record.  However, PaintRecord-backed PaintImages are also not serialized at the moment.

One option might be to store these PaintRecords in the transfer cache, but PaintRecords themselves might have PaintImages that need to be in the cache, so that dependency tree might get messy.

Another option is to change the code to remove PaintRecord-backed PaintImages and only use PaintRecord-backed PaintShaders in these cases, which should be equivalent.


See also:   issue 806313 , where non-lazy SkImage-backed PaintImages are also not serialized.
 
They do work right now but not optimally. We software rasterize them and transfer the bitmap and serializing the PaintRecord will be better.

Comment 2 by enne@chromium.org, Feb 8 2018

Description: Show this description

Comment 3 by enne@chromium.org, Feb 8 2018

Description: Show this description

Comment 4 by enne@chromium.org, Apr 5 2018

Status: Fixed (was: Available)
I think this should be fixed now.

Sign in to add a comment