Bug 37028

Summary: Update image results for SVG as CSS border image tests
Product: WebKit Reporter: Simon Fraser (smfr) <simon.fraser>
Component: SVGAssignee: Simon Fraser (smfr) <simon.fraser>
Status: RESOLVED FIXED    
Severity: Normal CC: bdakin, eric, jparent, krit, xan.lopez, yael, zimmermann
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: PC   
OS: OS X 10.5   
Attachments:
Description Flags
Better tests (zip)
none
Patch bdakin: review+

Description Simon Fraser (smfr) 2010-04-02 10:16:01 PDT
The pixel results of LayoutTests/fast/borders/svg-as-border-image-2-diffs.html show that SVG as border image is broken.
Comment 1 Simon Fraser (smfr) 2010-04-02 10:17:09 PDT
fast/borders/svg-as-border-image.html also broken
Comment 2 Simon Fraser (smfr) 2010-04-02 10:51:03 PDT
This was broken by http://trac.webkit.org/changeset/54865
Comment 3 Yael 2010-04-02 11:42:49 PDT
I think the expected result is wrong, and the same is for fast/images/svg-as-tiled-background.html.

The expected result of fast/borders/border-image-01.html shows that the image covers the whole div, so I don;t understand why this is not the case for SVG images.
Comment 4 Simon Fraser (smfr) 2010-04-28 22:58:12 PDT
Yes, I think the expected image are wrong. I have some better tests to land.
Comment 5 Simon Fraser (smfr) 2010-04-28 22:58:46 PDT
Created attachment 54678 [details]
Better tests (zip)
Comment 6 Nikolas Zimmermann 2010-04-29 00:57:15 PDT
Nice tests! If you upload a patch, I'm happy to r+ them.
Comment 7 Nikolas Zimmermann 2010-04-29 00:58:21 PDT
*** Bug 35323 has been marked as a duplicate of this bug. ***
Comment 8 Simon Fraser (smfr) 2010-04-29 20:33:23 PDT
Created attachment 54777 [details]
Patch
Comment 9 Beth Dakin 2010-04-29 20:57:09 PDT
Comment on attachment 54777 [details]
Patch

r=me!
Comment 10 Eric Seidel (no email) 2010-04-30 11:01:52 PDT
It makes no sense to me, but it seems the Gtk bot seems more likely to crash in worker tests after this change (which seems impossible to be related).  We'll have to monitor and see if it's just a bot issue.
Comment 12 Eric Seidel (no email) 2010-04-30 12:08:07 PDT
The Gtk bot seems to have righted itself and stopped being mad at the world.  I'll follow up with Xan to see if we've seen this before/how we can avoid false-positives like this in the future.
Comment 13 Eric Seidel (no email) 2010-05-02 19:31:07 PDT
Attachment 54777 [details] was posted by a committer and has review+, assigning to Simon Fraser for commit.
Comment 14 Simon Fraser (smfr) 2010-05-02 21:14:51 PDT
This landed in http://trac.webkit.org/changeset/58580