[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