Bug 19589 – Patch to re-enable some layout tests

• Status: UNCONFIRMED
• Component: Layout and Rendering

[Jean-Charles Verdié]

according to https://bugs.webkit.org/show_bug.cgi?id=19569
there are 51 layout disabled
47 of them are not crashing anymore and should be re-enabled.

The patch is pretty huge but I'm not sure how to split it correctly without it being a hassle :(

[Jean-Charles Verdié]

Created attachment 21738 [details]
re-enable layout tests which do not crash or hang

[Robert Blaut]

(In reply to comment #0)
> according to https://bugs.webkit.org/show_bug.cgi?id=19569
> there are 51 layout disabled
> 47 of them are not crashing anymore and should be re-enabled.

Thank you for your contribution. Jean-Charles, you should consider that many of disabled tests were disabled because of other problems than crash. For example:bug 11662, http://trac.webkit.org/changeset/28763

I think the best way for finding a real purpose of disabling test is checking revision logs of these tests: http://trac.webkit.org/browser

[Maciej Stachowiak]

What Robert suggests (reviewing the revision history of each test, or the ChangeLog) is a good idea. I forgot that some tests were off because they fail depending on system configuration. For those, we'd like to change them so they don't depend on the particular fonts the user has installed, and we could use a bug to represent that.

[Geoffrey Garen]

Comment on attachment 21738 [details]
re-enable layout tests which do not crash or hang

I guess I'll r- this patch, since it sounds like not all of these tests should be re-enabled right away.