Remove NativeSurfaceType::CAIRO_SURFACE

RESOLVED FIXED in Firefox 46

Status

()

defect
RESOLVED FIXED
4 years ago
4 years ago

People

(Reporter: njn, Assigned: njn)

Tracking

unspecified
mozilla46
Points:
---

Firefox Tracking Flags

(firefox46 fixed)

Details

Attachments

(1 attachment, 1 obsolete attachment)

Assignee

Description

4 years ago
We have both NativeSurfaceType::CAIRO_CONTEXT and
NativeSurfaceType::CAIRO_SURFACE. They both refer to DrawTargets with Cairo
backends. The only reason for having both is so you can call GetNativeSurface()
and get either the |cairo_t*| or the underlying |cairo_surface_t*|.

I argue that this convenience confuses things more than it helps.
Assignee

Comment 1

4 years ago
This patch removes CAIRO_SURFACE, and adds explicit cairo_get_target() calls
where necessary.
Attachment #8697476 - Flags: review?(bas)
Assignee

Comment 2

4 years ago
(Fix Windows compile error.)
Attachment #8697483 - Flags: review?(bas)
Assignee

Updated

4 years ago
Attachment #8697476 - Attachment is obsolete: true
Attachment #8697476 - Flags: review?(bas)
Comment on attachment 8697483 [details] [diff] [review]
Remove NativeSurfaceType::CAIRO_SURFACE

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

I agree.
Attachment #8697483 - Flags: review?(bas) → review+

Comment 5

4 years ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/0cd53e82df1f
Status: ASSIGNED → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla46
You need to log in before you can comment on or make changes to this bug.