[webkit-reviews] review granted: [Bug 39985] Several layout tests needlessly use invalid HTML : [Attachment 57525] Patch

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Tue Jun 1 10:29:08 PDT 2010


Adam Barth <abarth at webkit.org> has granted Eric Seidel <eric at webkit.org>'s
request for review:
Bug 39985: Several layout tests needlessly use invalid HTML
https://bugs.webkit.org/show_bug.cgi?id=39985

Attachment 57525: Patch
https://bugs.webkit.org/attachment.cgi?id=57525&action=review

------- Additional Comments from Adam Barth <abarth at webkit.org>
LayoutTests/ChangeLog:12
 +	       for this test to depend on WebKit's old parser behavior.
Can you cite a specific test here?  I want to be sure we don't lose test
coverage.

LayoutTests/ChangeLog:18
 +	       no need to depend on the old title behavior here.
This is also tested in fast/tokenizer.	I worry that some of these changes will
cause web compat problems.  We need to keep close tabs on this sort of thing.


More information about the webkit-reviews mailing list