[webkit-reviews] review denied: [Bug 47070] [Texmap] [Qt] Texture mapper initial implementation : [Attachment 70377] Patch 4: glue layer (PageClientQt) opt-in for TextureMapper
bugzilla-daemon at webkit.org
bugzilla-daemon at webkit.org
Sat Oct 9 16:47:24 PDT 2010
Kenneth Rohde Christiansen <kenneth at webkit.org> has denied Noam Rosenthal
<noam.rosenthal at nokia.com>'s request for review:
Bug 47070: [Texmap] [Qt] Texture mapper initial implementation
https://bugs.webkit.org/show_bug.cgi?id=47070
Attachment 70377: Patch 4: glue layer (PageClientQt) opt-in for TextureMapper
https://bugs.webkit.org/attachment.cgi?id=70377&action=review
------- Additional Comments from Kenneth Rohde Christiansen
<kenneth at webkit.org>
View in context: https://bugs.webkit.org/attachment.cgi?id=70377&action=review
> WebKit/qt/WebCoreSupport/PageClientQt.cpp:40
> + PlatformLayerProxyQt(QWebFrame* frame, TextureMapperContentLayer* layer,
QWidget* widget = 0, QGraphicsObject* graphicsObject = 0)
> + : QObject(widget ? (QObject*)widget : (QObject*)graphicsObject)
> + , m_widget(widget)
> + , m_graphicsItem(graphicsObject)
I dont like this very much... it is not clear if you can have widget and
graphics object at the same time and in that case, what the widget would be
> WebKit/qt/WebCoreSupport/PageClientQt.cpp:94
> +
> + QRect m_dirtyRect;
> + QWidget* m_widget;
> + QGraphicsItem* m_graphicsItem;
> + QWebFrame* m_frame;
> + TextureMapperContentLayer* m_layer;
Why are these public?
> WebKit/qt/WebCoreSupport/PageClientQt.cpp:428
> - // The sceneRect is a good approximation of the size of the application,
independent of the view.
> + // The sceneRect is a good approximation of the size of the application,
in dependent of the view.
This change doesn't seem right
> WebKit/qt/WebCoreSupport/PageClientQt.h:53
> + PageClientQWidget(QWidget* v, QWebPage *p)
> + : view(v)
> + , page(p)
just write out things
More information about the webkit-reviews
mailing list