Closed Bug 1301459 Opened 8 years ago Closed 8 years ago

In nsSVGMaskFrame::GetMaskForMaskedFrame, create surface by Factory::CreateDataSourceSurface

Categories

(Core :: Layout, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla52
Tracking Status
firefox52 --- fixed

People

(Reporter: u459114, Assigned: vliu)

Details

Attachments

(1 file)

In Bug 1301353 comment 3, mstange said:
why are we obtaining the surface by snapshotting an unused DrawTarget? We should create it through Factory::CreateDataSourceSurface.
Assignee: cku → ethlin
I will take this issue for the following.
Assignee: ethlin → vliu
Hi Markus,
Could you please have a review for the patch? Thanks.
Attachment #8796447 - Flags: review?(mstange)
Attachment #8796447 - Flags: review?(mstange) → review+
Pushed by vliu@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/1137d7187e52
In nsSVGMaskFrame::GetMaskForMaskedFrame, create surface by Factory::CreateDataSourceSurface. r=mstange
https://hg.mozilla.org/mozilla-central/rev/1137d7187e52
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla52
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: