[webkit-reviews] review denied: [Bug 23041] Add new API for visibleContents() to QWebFrame : [Attachment 26338] Revised patch which adds it as a read only property too
bugzilla-daemon at webkit.org
bugzilla-daemon at webkit.org
Fri Jan 2 10:10:57 PST 2009
George Staikos <staikos at kde.org> has denied Adam Treat <treat at kde.org>'s
request for review:
Bug 23041: Add new API for visibleContents() to QWebFrame
https://bugs.webkit.org/show_bug.cgi?id=23041
Attachment 26338: Revised patch which adds it as a read only property too
https://bugs.webkit.org/attachment.cgi?id=26338&action=review
------- Additional Comments from George Staikos <staikos at kde.org>
I think the API name is very un-Qt-like. Maybe visibleContentSize()?
Otherwise the patch is correct.
More information about the webkit-reviews
mailing list