Closed Bug 1301579 Opened 8 years ago Closed 8 years ago

Add assertion in D2D1 backend while handling A8 format in DrawSurface

Categories

(Core :: Graphics, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla52
Tracking Status
firefox52 --- fixed

People

(Reporter: ethlin, Assigned: ethlin)

Details

Attachments

(1 file)

Per Bug 1275478 comment 8, we should add assertion so others don't have to discover the behaviour difference themselves.
Attached patch add assertionSplinter Review
Add assertion to prevent the usage of DrawTargetD2D1::DrawSurface with A8 format.
Attachment #8789659 - Flags: review?(mstange)
Comment on attachment 8789659 [details] [diff] [review]
add assertion

Review of attachment 8789659 [details] [diff] [review]:
-----------------------------------------------------------------

Oops, somehow I missed this review request.
Attachment #8789659 - Flags: review?(mstange) → review+
Pushed by cbook@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/07276d294e17
Add assertion to prevent the use of DrawTargetD2D1::DrawSurface with A8 format. r=mstange
Keywords: checkin-needed
https://hg.mozilla.org/mozilla-central/rev/07276d294e17
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: