Comment 11 for bug 263661

Revision history for this message
In , Vladimir Vukicevic (vvuk) wrote :

(In reply to comment #4)
> (In reply to comment #2)
> > >+ pattern->SetExtend(gfxPattern::EXTEND_PAD);
> >
> > This shouldn't be done on OSX; the quartz surface behaves slightly differently
> > and PAD will cause us to take a slow path.
>
> It's necessary. I haven't tried this exact testcase but I can reproduce visual
> artifacts on OSX without it. Also, looking at cairo-quartz-surface.c, it
> doesn't seem to take a fallback for EXTEND_PAD, only EXTEND_REPEAT/REFLECT.

I can't reproduce with this testcase; no black or anything but green shows up. As you say, PAD is ignored anyway (incorrectly, I guess, but correctly for perf). So there's a separate bug there, I think.