Bug 42402 - [chromium] r63422 caused svg/custom/svg-fonts-without-missing-glyph.xhtml to fail on Linux and Windows
Summary: [chromium] r63422 caused svg/custom/svg-fonts-without-missing-glyph.xhtml to ...
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:
 
Reported: 2010-07-15 13:55 PDT by Ojan Vafai
Modified: 2010-07-15 23:21 PDT (History)
4 users (show)

See Also:


Attachments
Added missing expectations. (24.70 KB, patch)
2010-07-15 22:23 PDT, Hajime Morrita
tkent: review-
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Ojan Vafai 2010-07-15 13:55:46 PDT
It looks like it might just be antialiasing differences, but I don't understand enough about our SVG implementation to be sure.
Comment 1 Hajime Morrita 2010-07-15 22:23:20 PDT
Created attachment 61766 [details]
Added missing expectations.
Comment 2 Hajime Morrita 2010-07-15 22:26:11 PDT
(In reply to comment #0)
> It looks like it might just be antialiasing differences, but I don't understand enough about our SVG implementation to be sure.
That's right. The test is for pixel-testing, so it's natural to fail if platform expectations are missing.
Attached patch is the expectations created by rebaseline-chromium-webkit-tests.
Thank you for letting me know.
Comment 3 Kent Tamura 2010-07-15 22:43:07 PDT
Comment on attachment 61766 [details]
Added missing expectations.

You may commit test expectation update without review.
Anyway, the diff for ChangeLog is malformed. So r-.
Comment 4 Hajime Morrita 2010-07-15 23:19:51 PDT
Committed r63520: <http://trac.webkit.org/changeset/63520>
Comment 5 Hajime Morrita 2010-07-15 23:21:44 PDT
> (From update of attachment 61766 [details])
> You may commit test expectation update without review.
> Anyway, the diff for ChangeLog is malformed. So r-.
Thank you for advice. I landed the patch fixing the changelog.