Bug 37028 - Update image results for SVG as CSS border image tests
Summary: Update image results for SVG as CSS border image tests
Alias: None
Product: WebKit
Classification: Unclassified
Component: SVG (show other bugs)
Version: 528+ (Nightly build)
Hardware: PC OS X 10.5
: P2 Normal
Assignee: Simon Fraser (smfr)
: 35323 (view as bug list)
Depends on:
Reported: 2010-04-02 10:16 PDT by Simon Fraser (smfr)
Modified: 2010-05-02 21:14 PDT (History)
7 users (show)

See Also:

Better tests (zip) (44.21 KB, application/octet-stream)
2010-04-28 22:58 PDT, Simon Fraser (smfr)
no flags Details
Patch (80.12 KB, patch)
2010-04-29 20:33 PDT, Simon Fraser (smfr)
bdakin: review+
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
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]
Comment 9 Beth Dakin 2010-04-29 20:57:09 PDT
Comment on attachment 54777 [details]

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