Bug 26219 - REGRESSION: 92 SVG layout tests failing on Leopard Debug buildbot with wrong fill color
Summary: REGRESSION: 92 SVG layout tests failing on Leopard Debug buildbot with wrong ...
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Tools / Tests (show other bugs)
Version: 528+ (Nightly build)
Hardware: Mac OS X 10.5
: P2 Normal
Assignee: Nobody
URL: http://build.webkit.org/builders/Leop...
Keywords: LayoutTestFailure, NeedsReduction, Regression
Depends on:
Blocks:
 
Reported: 2009-06-05 09:45 PDT by David Kilzer (:ddkilzer)
Modified: 2009-06-09 21:42 PDT (History)
1 user (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description David Kilzer (:ddkilzer) 2009-06-05 09:45:53 PDT
* SUMMARY
Ninety-two SVG layout tests are failing on the Leopard Debug buildbot due to differences in fill color as reported by the dumped render tree.

* STEPS TO REPRODUCE
Unknown at this time.  The tests only seem to be failing on the one buildbot.

* RESULTS
The fill color differs on the dumped render tree for each of the tests:

-      RenderPath {rect} [...] [fill={[type=SOLID] [color=#FF0000] [...]
+      RenderPath {rect} [...] [fill={[type=SOLID] [color=#008000] [...]

* REGRESSION
This is a recent regression.
Comment 1 Eric Seidel (no email) 2009-06-05 09:48:49 PDT
-      RenderPath {rect} [...] [fill={[type=SOLID] [color=#FF0000] [...]
+      RenderPath {rect} [...] [fill={[type=SOLID] [color=#008000] [...]

Moving from Red to Green seems like a good thing? ;)
Comment 3 Eric Seidel (no email) 2009-06-05 09:53:48 PDT
I don't see how a test which is explicitly listing fill="red" could be end up filling 0x008000, unless somehow the CSS color names are off on leopard?

We'll need to figure out when this regression started.
Comment 4 Mark Rowe (bdash) 2009-06-07 01:03:09 PDT
I've been poking around on the buildbot a little and haven't been able to work out what's going on.  When the tests are run by themselves they pass without any problems.  If I run with --nthly=10 then all of the tests pass without problems too.  With --nthly=100 two tests fail.  It seems a lot like one or more tests are doing something that causes later tests to fail.  It's possible that some SVG code is reading uninitialized memory, which happens to be the expected value most of the time but on this machine happens to trigger bogus behavior.  I'll keep digging and see if I can come up with anything useful.
Comment 5 Mark Rowe (bdash) 2009-06-07 01:44:18 PDT
I've been focusing on svg/custom/group-opacity.svg, which is typically one of the first tests to fail.  Valgrind shows CoreGraphics reading from some uninitialized stack memory while doing color-matching on the PNGs that the test loads.  Other than the usual JSCore GC things, there's no other valgrind output.  I don't think CG reading from uninitialized memory could cause this failure.  I'm running out of ideas on how to track this down though.
Comment 6 Mark Rowe (bdash) 2009-06-07 02:06:26 PDT
The first time that this showed up on the buildbot was with r44388.  The last successful run prior to that was r44383.  Nothing in that revision range looks relevant.  The closest change that looks like it could be relevant is r44379.
Comment 7 Mark Rowe (bdash) 2009-06-07 02:06:50 PDT
<http://trac.webkit.org/changeset/44379> for easy access.
Comment 8 David Kilzer (:ddkilzer) 2009-06-07 06:17:05 PDT
(In reply to comment #6)
> The first time that this showed up on the buildbot was with r44388.  The last
> successful run prior to that was r44383.  Nothing in that revision range looks
> relevant.  The closest change that looks like it could be relevant is r44379.

Oddly enough, this seems to be related to Bug 26218 Comment #2, which shows the same revision range causing issues.
Comment 9 David Kilzer (:ddkilzer) 2009-06-09 21:42:15 PDT
(In reply to comment #8)
> (In reply to comment #6)
> Oddly enough, this seems to be related to Bug 26218 Comment #2, which shows the
> same revision range causing issues.

The fix for Bug 26218 also fixed this issue:

http://trac.webkit.org/changeset/44549