Bug 39980 - Add Chromium baselines for svg/filters/parent-children-with-same-filter-expected
Summary: Add Chromium baselines for svg/filters/parent-children-with-same-filter-expected
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Platform (show other bugs)
Version: 528+ (Nightly build)
Hardware: PC All
: P2 Normal
Assignee: Stephen White
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-05-31 21:22 PDT by Roland Steiner
Modified: 2010-06-09 05:22 PDT (History)
4 users (show)

See Also:

Attachments
Patch (4.57 KB, patch)
2010-05-31 22:51 PDT, Roland Steiner 		no flags Details | Formatted Diff | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Roland Steiner 2010-05-31 21:22:40 PDT 
WebKit r60439 added a new layout test svg/filters/parent-children-with-same-filter-expected.

This test has differing pixel results on Chromium.
Comment 1 Roland Steiner 2010-05-31 22:51:08 PDT 
Created attachment 57519 [details]
Patch
Comment 2 Roland Steiner 2010-05-31 22:57:36 PDT 
Comment on attachment 57519 [details]
Patch

Results are similar to svg/filters/big-sized-filter2, using the same filters.

To unblock the WebKit roll I therefore committed the new baselines, but please review them.
Comment 3 Roland Steiner 2010-05-31 22:58:02 PDT 
Committed new baselines as WebKit r60462
Comment 4 Stephen White 2010-06-01 07:50:02 PDT 
Seems a little strange that the Mac baseline is so much darker.  Not your problem, but perhaps has something to do with colour space issues.  Other than that, LGTM.
Comment 5 Peter Kasting 2010-06-01 10:26:38 PDT 
Should this bug be closed, or should the Mac baseline concern in comment 4 be investigated by someone?
Comment 6 Dirk Schulze 2010-06-09 02:20:45 PDT 
Any news on this bug report?
Comment 7 Stephen White 2010-06-09 05:22:45 PDT 
(In reply to comment #5)
> Should this bug be closed, or should the Mac baseline concern in comment 4 be investigated by someone?

Since this test is basically a crash test, and the colourspace issues are covered by a number of other layout tests, I'm closing this bug.