Closed Bug 935923 Opened 11 years ago Closed 11 years ago

Rename AlphaMode helper function and stop using it

Categories

(Core :: Graphics, defect)

All
Windows 7
defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla28

People

(Reporter: mstange, Assigned: mstange)

References

Details

(Whiteboard: [qa-])

Attachments

(1 file, 1 obsolete file)

The AlphaMode function in D2DHelpers.h conflicts with the AlphaMode enum I want to add in Filters.h. And in most places we can just use D2DPixelFormat instead.
Attached patch rename-alphamode (obsolete) — Splinter Review
Attachment #828579 - Flags: review?(bas)
Attached patch rename-alphamodeSplinter Review
missed one
Attachment #828579 - Attachment is obsolete: true
Attachment #828579 - Flags: review?(bas)
Attachment #828589 - Flags: review?(bas)
Comment on attachment 828589 [details] [diff] [review]
rename-alphamode

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

Looks good to me! Please push to both Moz2D and m-c.
Attachment #828589 - Flags: review?(bas) → review+
https://hg.mozilla.org/mozilla-central/rev/ee8d7b270b8f
Status: ASSIGNED → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla28
Whiteboard: [qa-]
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: