Bug 40234 - [CHROMIUM] Layout test svg-fonts-in-html is failing
Summary: [CHROMIUM] Layout test svg-fonts-in-html is failing
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Layout and Rendering (show other bugs)
Version: 528+ (Nightly build)
Hardware: PC Windows 7
: P3 Normal
Assignee: Nobody
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-06-07 08:20 PDT by Stephen White
Modified: 2010-06-07 08:47 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 Stephen White 2010-06-07 08:20:22 PDT
The following layout test is failing on chromium-win and chromium-linux:

svg/custom/svg-fonts-in-html.html

Probable cause:

This was caused by something the range 60781:60783.  I'm suspecting 60781 ("WebCore: Space should be allowed between -- and > in comment end"), since the other two are non-code changes.
Comment 1 Julien Chaffraix 2010-06-07 08:35:43 PDT
(In reply to comment #0)
> The following layout test is failing on chromium-win and chromium-linux:
> 
> svg/custom/svg-fonts-in-html.html
> 
> Probable cause:
> 
> This was caused by something the range 60781:60783.  I'm suspecting 60781 ("WebCore: Space should be allowed between -- and > in comment end"), since the other two are non-code changes.

I reverted 60781 as other platforms reported regressions. Btw, I did not see this test failing on the chromium core build bots. Do you have any other bots that caught this?
Comment 2 Stephen White 2010-06-07 08:44:08 PDT
(In reply to comment #1)

> I reverted 60781 as other platforms reported regressions. Btw, I did not see this test failing on the chromium core build bots. Do you have any other bots that caught this?

Yes, we have the "WebKit Canary" bots:

http://build.chromium.org/buildbot/waterfall.fyi/waterfall?builder=Webkit%20(webkit.org)&builder=Webkit%20Mac%20(webkit.org)&builder=Webkit%20Linux%20(webkit.org)&builder=XP%20Perf%20(webkit.org)(1)&builder=XP%20Perf%20(webkit.org)(2)&builder=Linux%20Perf%20(webkit.org)&builder=Webkit%20(purify%20webkit.org)&builder=Webkit%20Linux%20(valgrind%20webkit.org)

These do a full build of tip-of-tree Chromium with tip-of-tree WebKit, and they run the full layout test suite (which I think the EWS and core chromium buildbots don't).  So keeping the canaries green is a "nice to have" at this point, but not really a WebKit committer's responsibility (yet).
Comment 3 Stephen White 2010-06-07 08:47:17 PDT
r60781 was reverted, and this test started passing again.  Closing bug.