581083 - REFTEST TEST-UNEXPECTED-FAIL | file:///c:/talos-slave/mozilla-central-win7-opt-u-reftest-d2d/build/reftest/tests/layout/reftests/svg/dynamic-small-object-scaled-up-02.svg |

Status: RESOLVED FIXED

(Core :: Graphics, defect)

Description

[Jeff Muizelaar [:jrmuizel]]

This one has a pretty substantial difference.

Comment 1

[Bas Schouten (:bas.schouten)]

Attachment: Fix transformed stroke drawing

The problem here is that we're inverse transforming the stroke in order to set the transform on the rasterizer. This is needed because the miter angles and stroke width and such are untransformed. Because this path was really small when inverse transformed back to its original, the cairo fixed point precision of 8 fractional bits was insufficient to store the stroke properly and rounding errors occurred. I know inverse the transformation in D2D which uses floating point, avoiding the issue. This fixes the reftest.

Comment 2

[Jeff Muizelaar [:jrmuizel]]

Comment on attachment 460186 [details] [diff] [review]
Fix transformed stroke drawing

Why do the box tests go away?

Comment 3

[Bas Schouten (:bas.schouten)]

(In reply to comment #2)
> Comment on attachment 460186 [details] [diff] [review]
> Fix transformed stroke drawing
> 
> Why do the box tests go away?

Because I'm drawing a transformed geometry anyway. I could transform a rectangle geometry but I doubt it would perform that much fast, I could bring it back though.

Comment 4

[Jeff Muizelaar [:jrmuizel]]

Comment on attachment 460186 [details] [diff] [review]
Fix transformed stroke drawing

Make sure you get rid of the DEBUG_LEVEL changes.

Comment 5

[Bas Schouten (:bas.schouten)]

This needs blocking to land.

Comment 6

[Mike Shaver (:shaver -- probably not reading bugmail closely)]

Blocks a blocker, let's go.

Comment 7

[Bas Schouten (:bas.schouten)]

Pushed http://hg.mozilla.org/mozilla-central/rev/d29e7fff79b6.

Comment 8

[Jeff Muizelaar [:jrmuizel]]

This or the gradient patch seems to have caused a bunch of new reftest failures. Bas, can you try to figure out which and why?

Comment 9

[Bas Schouten (:bas.schouten)]

I believe these are intermittent failures sadly, I'm not sure why, when I tested locally I looked very briefly at a few, but they weren't really rendering problems. It was like two textboxes showing completely different text and such things. But I didn't look at what seemed to cause it yet.