Bug 49521 - REGRESSION svg/W3C-SVG-1.1/fonts-desc-02-t.svg broken by r71979
Summary: REGRESSION svg/W3C-SVG-1.1/fonts-desc-02-t.svg broken by r71979
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Tools / Tests (show other bugs)
Version: 528+ (Nightly build)
Hardware: All All
: P2 Normal
Assignee: Dave Hyatt
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-11-14 16:49 PST by Mihai Parparita
Modified: 2010-11-14 21:43 PST (History)
5 users (show)

See Also:


Attachments
Patch (3.17 KB, patch)
2010-11-14 20:45 PST, Dave Hyatt
mitz: review+
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Mihai Parparita 2010-11-14 16:49:15 PST
This may just need a rebaseline, but the pixel ouput of the test now fails:

Expected: http://trac.webkit.org/export/71987/trunk/LayoutTests/platform/mac/svg/W3C-SVG-1.1/fonts-desc-02-t-expected.png
Actual: http://build.chromium.org/f/chromium/layout_test_results/webkit-rel-mac-webkit-org/results/layout-test-results/svg/W3C-SVG-1.1/fonts-desc-02-t-actual.png
Diff: http://build.chromium.org/f/chromium/layout_test_results/webkit-rel-mac-webkit-org/results/layout-test-results/svg/W3C-SVG-1.1/fonts-desc-02-t-diff.png

(actual result from the chromium-mac bot, but run-webkit-tests -p --tolerance=0 LayoutTests/svg/W3C-SVG-1.1/fonts-desc-02-t.svg produces the same result for the mac port)
Comment 1 Mihai Parparita 2010-11-14 16:50:44 PST
Not sure why hyatt@apple.com was excluded from the mail, adding Dave as an explicit cc.
Comment 2 Dave Hyatt 2010-11-14 18:57:19 PST
I'll look at it tomorrow.  Thanks.
Comment 3 Dave Hyatt 2010-11-14 20:45:13 PST
Created attachment 73872 [details]
Patch
Comment 4 Dave Hyatt 2010-11-14 20:53:27 PST
Fixed in r71993.
Comment 5 WebKit Review Bot 2010-11-14 21:43:00 PST
http://trac.webkit.org/changeset/71993 might have broken Leopard Intel Release (Tests)
The following tests are not passing:
fast/css/font-face-default-font.html