Bug 59910 - REGRESSION: SVG pattern clipped with transform on target
Summary: REGRESSION: SVG pattern clipped with transform on target
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: SVG (show other bugs)
Version: 528+ (Nightly build)
Hardware: PC OS X 10.5
: P2 Normal
Assignee: Nobody
URL:
Keywords:
Depends on:
Blocks: 59551
  Show dependency treegraph
 
Reported: 2011-05-01 18:57 PDT by Dirk Schulze
Modified: 2012-01-11 00:10 PST (History)
2 users (show)

See Also:


Attachments
Wrong tile clipping on transformation of target (440 bytes, image/svg+xml)
2011-05-01 18:57 PDT, Dirk Schulze
no flags Details
Another testcase (793 bytes, application/xhtml+xml)
2011-08-02 12:24 PDT, Tim Horton
no flags Details
Broken with accelerated compositing, fine without (634 bytes, application/xhtml+xml)
2011-08-02 12:26 PDT, Tim Horton
no flags Details

Note You need to log in before you can comment on or make changes to this bug.
Description Dirk Schulze 2011-05-01 18:57:22 PDT
Created attachment 91871 [details]
Wrong tile clipping on transformation of target

If a target object references a pattern and the target gets transformed, this might influence the size, position and clipping of the pattern surface in a wrong way. Because we have similar problems on filters and maskers it might be an issue with SVGImageBufferTools. But this needs more investigation. Just open the bug for pattern for now.

I added an example with a pattern. The target object gets rotated by 225 degree at the center of the circles. Without the rotation the tile of the pattern doesn't get clipped.
Comment 1 Dirk Schulze 2011-05-02 09:04:06 PDT
Works on Safari, so more likely introduced with SVGImageBufferSupport.
Comment 2 Tim Horton 2011-08-02 12:20:49 PDT
I think this is the root cause of <rdar://problem/9383222>.
Comment 3 Dirk Schulze 2011-08-02 12:22:21 PDT
(In reply to comment #2)
> I think this is the root cause of <rdar://problem/9383222>.

What is <rdar://problem/9383222> ?
Comment 4 Tim Horton 2011-08-02 12:24:17 PDT
Created attachment 102678 [details]
Another testcase
Comment 5 Tim Horton 2011-08-02 12:25:24 PDT
(In reply to comment #3)
> (In reply to comment #2)
> > I think this is the root cause of <rdar://problem/9383222>.
> 
> What is <rdar://problem/9383222> ?

I've attached the testcase to this bug. The issue goes away when you don't use accelerated compositing, also.
Comment 6 Tim Horton 2011-08-02 12:25:41 PDT
Oh, wait, yours doesn't go away when you turn off accelerated compositing. Maybe they're not related?
Comment 7 Tim Horton 2011-08-02 12:26:35 PDT
Created attachment 102679 [details]
Broken with accelerated compositing, fine without
Comment 8 Tim Horton 2012-01-06 13:52:34 PST
Dirk, can you confirm, it seems like your test case is fixed by http://trac.webkit.org/changeset/94338/trunk

Mine will be fixed by my next patch, but I'll make a separate bug since the issues are clearly unrelated now.
Comment 9 Dirk Schulze 2012-01-08 16:38:25 PST
(In reply to comment #8)
> Dirk, can you confirm, it seems like your test case is fixed by http://trac.webkit.org/changeset/94338/trunk
> 
> Mine will be fixed by my next patch, but I'll make a separate bug since the issues are clearly unrelated now.

Just downloaded the nightly r104403 and I don't see the issue anymore. If your test still fails, can you please create a new bug report? Also can someone verify that it just was an issue with WebKit and CG before closing this bug please? Maybe by testing with Chromium Skia or WebKitGtk?
Comment 10 Tim Horton 2012-01-10 12:30:18 PST
(In reply to comment #9)
> (In reply to comment #8)
> > Dirk, can you confirm, it seems like your test case is fixed by http://trac.webkit.org/changeset/94338/trunk
> > 
> > Mine will be fixed by my next patch, but I'll make a separate bug since the issues are clearly unrelated now.
> 
> Just downloaded the nightly r104403 and I don't see the issue anymore. If your test still fails, can you please create a new bug report? Also can someone verify that it just was an issue with WebKit and CG before closing this bug please? Maybe by testing with Chromium Skia or WebKitGtk?

Chromium Mac nightlies should use Skia, right? The drawing is more aliased than CG, so I'm going to assume it's using Skia. In any case, the problem doesn't reproduce there either.

And, I did file another bug for my issue: https://bugs.webkit.org/show_bug.cgi?id=75741 ... I just have to go land it.
Comment 11 Dirk Schulze 2012-01-11 00:10:07 PST
(In reply to comment #10)
> (In reply to comment #9)
> > (In reply to comment #8)
> > > Dirk, can you confirm, it seems like your test case is fixed by http://trac.webkit.org/changeset/94338/trunk
> > > 
> > > Mine will be fixed by my next patch, but I'll make a separate bug since the issues are clearly unrelated now.
> > 
> > Just downloaded the nightly r104403 and I don't see the issue anymore. If your test still fails, can you please create a new bug report? Also can someone verify that it just was an issue with WebKit and CG before closing this bug please? Maybe by testing with Chromium Skia or WebKitGtk?
> 
> Chromium Mac nightlies should use Skia, right? The drawing is more aliased than CG, so I'm going to assume it's using Skia. In any case, the problem doesn't reproduce there either.
> 
> And, I did file another bug for my issue: https://bugs.webkit.org/show_bug.cgi?id=75741 ... I just have to go land it.

Ok great. I'll close the bug now.