484076 - Update pixman and cairo

Status: RESOLVED FIXED

(Core :: Graphics, defect)

Description

[Jeff Muizelaar [:jrmuizel]]

Attachment: update pixman

The following patches update pixman and cairo to upstream trunk.

Combined they pass everything on the tryserver and bring a tp win on linux

Comment 1

[Vladimir Vukicevic [:vlad] [:vladv] (needinfo me, slow to respond)]

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.

Comment 2

[Jeff Muizelaar [:jrmuizel]]

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

Comment 3

[Joe Drew (not getting mail)]

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

Comment 4

[Joe Drew (not getting mail)]

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.

Comment 5

[Jeff Muizelaar [:jrmuizel]]

I've updated the patch at http://people.mozilla.org/~jmuizelaar/cairo.patch
It updates us to cairo revision c932a809d6484503d7ee267d934bbc87c8d44092

Comment 6

[Joe Drew (not getting mail)]

http://hg.mozilla.org/mozilla-central/rev/1effb72d30cf

I accidentally pushed this as myself instead of changing the user to Jeff. Sorry, Jeff!

Comment 7

[Jeff Muizelaar [:jrmuizel]]

Attachment: Fix Windows CE build

Comment 8

[Joe Drew (not getting mail)]

http://hg.mozilla.org/mozilla-central/rev/3aba63a26a48

Comment 9

[Brad Lassey [:blassey] (use needinfo?)]

Attachment: hypot depricated on windows and non-existant on windows ce

Comment 10

[Vladimir Vukicevic [:vlad] [:vladv] (needinfo me, slow to respond)]

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.

Comment 11

[Brad Lassey [:blassey] (use needinfo?)]

pushed http://hg.mozilla.org/mozilla-central/rev/c268d93ef781

Comment 12

[Jeff Muizelaar [:jrmuizel]]

Attachment: Shrink the glyph cache to fix tp_Rss regression

Comment 13

[Joe Drew (not getting mail)]

http://hg.mozilla.org/mozilla-central/rev/2617fe64e693

Comment 14

[philippe (part-time)]

Did this possibly cause bug 484954 (seems to affect Windows 7 only) ?

Comment 15

[Jeff Muizelaar [:jrmuizel]]

It's definitely possible, though I'm not sure why or how it would have... I'll try to take a look.