Closed
Bug 484076
Opened 16 years ago
Closed 16 years ago
Update pixman and cairo
Categories
(Core :: Graphics, defect)
Tracking
()
RESOLVED
FIXED
mozilla1.9.2a1
People
(Reporter: jrmuizel, Assigned: jrmuizel)
References
(Depends on 1 open bug)
Details
Attachments
(4 files)
(deleted),
patch
|
vlad
:
review+
|
Details | Diff | Splinter Review |
(deleted),
patch
|
Details | Diff | Splinter Review | |
(deleted),
patch
|
vlad
:
review+
|
Details | Diff | Splinter Review |
(deleted),
patch
|
joe
:
review+
|
Details | Diff | 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+
Assignee | ||
Comment 2•16 years ago
|
||
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
Assignee | ||
Updated•16 years ago
|
Whiteboard: checkin-needed
Comment 3•16 years ago
|
||
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
Comment 4•16 years ago
|
||
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 → ---
Assignee | ||
Comment 5•16 years ago
|
||
I've updated the patch at http://people.mozilla.org/~jmuizelaar/cairo.patch
It updates us to cairo revision c932a809d6484503d7ee267d934bbc87c8d44092
Keywords: checkin-needed
Comment 6•16 years ago
|
||
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 ago → 16 years ago
Resolution: --- → FIXED
Updated•16 years ago
|
Keywords: checkin-needed
Assignee | ||
Comment 7•16 years ago
|
||
Comment 8•16 years ago
|
||
Comment 9•16 years ago
|
||
Attachment #368700 -
Flags: review?(vladimir)
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+
Comment 11•16 years ago
|
||
Assignee | ||
Comment 12•16 years ago
|
||
Attachment #368897 -
Flags: review?(joe)
Updated•16 years ago
|
Attachment #368897 -
Flags: review?(joe) → review+
Assignee | ||
Updated•16 years ago
|
Keywords: checkin-needed
Comment 13•16 years ago
|
||
Keywords: checkin-needed
Comment 14•16 years ago
|
||
Did this possibly cause bug 484954 (seems to affect Windows 7 only) ?
Assignee | ||
Comment 15•16 years ago
|
||
It's definitely possible, though I'm not sure why or how it would have... I'll try to take a look.
Updated•15 years ago
|
Assignee: nobody → jmuizelaar
Target Milestone: --- → mozilla1.9.2a1
You need to log in
before you can comment on or make changes to this bug.
Description
•