[Webkit-unassigned] [Bug 76661] [Qt] [WK2] Support threaded renderer in WK2

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=76661





--- Comment #15 from Viatcheslav Ostapenko <ostapenko.viatcheslav at nokia.com>  2012-01-20 09:19:27 PST ---
(In reply to comment #14)
> (From update of attachment 123234 [details])
> View in context: https://bugs.webkit.org/attachment.cgi?id=123234&action=review
> 
> The clean up of QQuickWebPage looks very nice, some other comments:
> 
> > Source/WebKit2/UIProcess/API/qt/qquickwebpage.cpp:36
> > +    setClip(true);
> 
> Why is this needed, isn't the texture mapper overriding the clip anyway?

1. It doesn't work if display doesn't have stencil buffer.
2. I think we shouldn't have clipping it texturemapper, because Qt does it more effectively choosing between stencil/scissor clipping.
3. It seems there is problems with clipping when we have overshoot panning. At least I've seen painting artefacts and switching to item clipping solves it.

> >> Source/WebKit2/UIProcess/API/qt/qquickwebpage.cpp:93
> >> +        return drawingArea->layerTreeHostProxy()->updatePaintNode(oldNode);
> > 
> > Add a layerTreeHostProxy() function to QQuickWebPagePrivate instead of doing this twice
> 
> I don't like the idea of the LayerTreeHost being referenced both on the scene graph and in WebKit.
> The problem is when the WebView gets destroyed on the UI thread, the rendering thread must be able to continue. And delaying the destruction of the LayerTreeHostProxy alone until the rendering thread is done destroying the scene graph nodes seems wrong to me.
> 
> Isn't it possible to instead move all the texture mapper related objects/nodes ownership to the scene graph node that we attach on the scene graph, and leave the LayerTreeHostProxy referenced by the UI thread only? To put it in other words, all the texture mapper node tree would be owned only by the object we return from updatePaintNode.

Just tried to keep patch minimal. Agreed with Noam on irc that we should do real move.

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.