[webkit-reviews] review denied: [Bug 6402] Implement frameElement.width and frameElement.height : [Attachment 5518] proposed patch

[bugzilla-request-daemon at opendarwin.org]

Geoffrey Garen <ggaren at apple.com> has denied Alexey Proskuryakov
<ap at nypop.com>'s request for review:
Bug 6402: Implement frameElement.width and frameElement.height
http://bugzilla.opendarwin.org/show_bug.cgi?id=6402

Attachment 5518: proposed patch
http://bugzilla.opendarwin.org/attachment.cgi?id=5518&action=edit

------- Additional Comments from Geoffrey Garen <ggaren at apple.com>
I second Eric's question: why virtual?

The alert will work in the layout test, but it can be kinda annoying when you
open that page manually. An explanation of what you're testing and a reference
to the related bug are also important and missing. Check out
LayoutTests/fast/js/string-split-ignore-case.html for an example. It also
demonstrates how to produce output without an alert.