[webkit-reviews] review denied: [Bug 24900] Render empty optgroup elements. : [Attachment 29559] patch

[bugzilla-daemon at webkit.org]

Eric Seidel <eric at webkit.org> has denied Adam Langley <agl at chromium.org>'s
request for review:
Bug 24900: Render empty optgroup elements.
https://bugs.webkit.org/show_bug.cgi?id=24900

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

------- Additional Comments from Eric Seidel <eric at webkit.org>
+	 // Odd tags like <div> will already have been removed from the tree.
+	 current = current->traverseNextSibling(this);

This needs a "so what?" explanation. 

// We traverse nextSibling instead of nextNode because... why?

Otherwise this looks fine.  If you update that comment you're good to land IMO.


If you post an updated comment I'm happy to land this for you.