ClippedImage should propagate a DrawResult correctly even when it creates an intermediate surface

RESOLVED FIXED in Firefox 44

Status

()

defect
RESOLVED FIXED
4 years ago
4 years ago

People

(Reporter: seth, Assigned: seth)

Tracking

unspecified
mozilla44
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox44 fixed)

Details

Attachments

(1 attachment)

Right now if ClippedImage creates an intermediate surface (by calling GetFrameInternal() in Draw()), it just drops the DrawResult that it generates in DrawSingleTile() on the floor. To support sync decoding for border-image correctly, we need to propagate that DrawResult to the caller.
Here's the patch. This is almost identical to the approach we use in RasterImage
for ImageContainers; we cache the DrawResult that we generate when we create the
cached surface, and then return it to the caller from Draw().
Attachment #8667500 - Flags: review?(tnikkel)
Attachment #8667500 - Flags: review?(tnikkel) → review+
Thanks for the review!
https://hg.mozilla.org/mozilla-central/rev/4d2c675a8830
Status: NEW → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla44
You need to log in before you can comment on or make changes to this bug.