Closed Bug 310957 Opened 16 years ago Closed 16 years ago

Switch win32 svg renderer from GDI+ to cairo

Categories

(Firefox Build System :: General, defect)

x86
Windows XP
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: tor, Assigned: tor)

References

Details

(Keywords: fixed1.8)

Attachments

(1 file)

 
Attachment #198332 - Flags: review?(benjamin)
Comment on attachment 198332 [details] [diff] [review]
all platforms default to cairo

This allows SVG to work "out of the box" on pre-XP win32 machines and gives us
consistent behavior over all platforms.  We've had test builds out for a week
and have sorted out all the problems save for a printing font size issue that
is still being tracked down.  Some PR risk due to decreased performance of
cairo compared to GDI+ which was used in 1.5b1.
Attachment #198332 - Flags: approval1.8b5?
Attachment #198332 - Flags: review?(benjamin) → review+
Comment on attachment 198332 [details] [diff] [review]
all platforms default to cairo

Approving r.e. discussion on drivers email about migration to cairo for SVG.
Attachment #198332 - Flags: approval1.8b5? → approval1.8b5+
Checked in on branch.  Leaving open for trunk checkin when we upgrade trunk
cairo to >= 1.0.2.
Keywords: fixed1.8
I assume gksvggdiplus.dll can now be removed from the build (new bug needed ?)
(In reply to comment #5)
> I assume gksvggdiplus.dll can now be removed from the build (new bug needed ?)

I did a quick search on open bugs with GDI, and found 19 FWIW.
Checked in on trunk.
Status: NEW → RESOLVED
Closed: 16 years ago
Resolution: --- → FIXED
Blocks: 314060
Blocks: 314063
*** Bug 299019 has been marked as a duplicate of this bug. ***
Product: Core → Firefox Build System
You need to log in before you can comment on or make changes to this bug.