[webkit-reviews] review denied: [Bug 42431] Update LegacyHTMLTreeBuilder to insert whitespace between </head> and <body> : [Attachment 61754] Patch

Thu Jul 15 18:34:54 PDT 2010

Eric Seidel <eric at webkit.org> has denied Adam Barth <abarth at webkit.org>'s
request for review:
Bug 42431: Update LegacyHTMLTreeBuilder to insert whitespace between </head>
and <body>
https://bugs.webkit.org/show_bug.cgi?id=42431

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

------- Additional Comments from Eric Seidel <eric at webkit.org>
LayoutTests/platform/mac/fast/css/style-parsed-outside-head-expected.txt:5
 +	RenderBlock (anonymous) at (0,0) size 785x18
You some how ended up with text content before the <body>.  That seems wrong. 
I think your change is not quite right. I think you need to check that it's
only whitespace before inserting it before the body.