Closed Bug 484076 Opened 16 years ago Closed 16 years ago

Update pixman and cairo

Categories

(Core :: Graphics, defect)

x86
macOS
defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla1.9.2a1

People

(Reporter: jrmuizel, Assigned: jrmuizel)

References

(Depends on 1 open bug)

Details

Attachments

(4 files)

Attached patch update pixman (deleted) — Splinter Review
The following patches update pixman and cairo to upstream trunk. Combined they pass everything on the tryserver and bring a tp win on linux
Attachment #368130 - Flags: review?(vladimir)
Comment on attachment 368130 [details] [diff] [review] update pixman Woo! Though please edit the bug summary here to say which git revs we're updating each to (or add a comment) to have that info somewhere.
Attachment #368130 - Flags: review?(vladimir) → review+
Update to cairo trunk. There's a pile of changes here (too big to attach) http://people.mozilla.org/~jmuizelaar/cairo.patch Highlights: scanline-raster, fast in_fill, new glyph cache This also lets us fix a todo in test_2d.path.isPointInPath.edge.html The pixman revision is 072d848f592530973f1f0a0066a320ec5965625c and the cairo revision is 3ec94f9b5917b8f71304c982bd153e5a810f3b12
Whiteboard: checkin-needed
In future, you'll want to set the _keyword_ checkin-needed, not whiteboard. Luckily I watch GFX: Thebes. :) http://hg.mozilla.org/mozilla-central/rev/d641ee2bf812
Status: NEW → RESOLVED
Closed: 16 years ago
Resolution: --- → FIXED
Whiteboard: checkin-needed
Turns out I only pushed the pixman patch because I missed the fact that the Cairo patch wasn't attached. Jeff wants to ensure we have everything we need with the Cairo update before it gets pushed, though.
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
I've updated the patch at http://people.mozilla.org/~jmuizelaar/cairo.patch It updates us to cairo revision c932a809d6484503d7ee267d934bbc87c8d44092
Keywords: checkin-needed
http://hg.mozilla.org/mozilla-central/rev/1effb72d30cf I accidentally pushed this as myself instead of changing the user to Jeff. Sorry, Jeff!
Status: REOPENED → RESOLVED
Closed: 16 years ago16 years ago
Resolution: --- → FIXED
Attached patch Fix Windows CE build (deleted) — Splinter Review
Comment on attachment 368700 [details] [diff] [review] hypot depricated on windows and non-existant on windows ce I'll check this into cairo upstream as well.
Attachment #368700 - Flags: review?(vladimir) → review+
Attachment #368897 - Flags: review?(joe)
Attachment #368897 - Flags: review?(joe) → review+
Keywords: checkin-needed
Did this possibly cause bug 484954 (seems to affect Windows 7 only) ?
Depends on: 484954
It's definitely possible, though I'm not sure why or how it would have... I'll try to take a look.
Depends on: 487137
Depends on: 488250
Depends on: 494743
Depends on: 526976
Assignee: nobody → jmuizelaar
Target Milestone: --- → mozilla1.9.2a1
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: