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

Thu Jan 19 16:16:56 PST 2012

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

--- Comment #8 from Noam Rosenthal <noam.rosenthal at nokia.com>  2012-01-19 16:16:56 PST ---
> But what should it return if LayerTreeHostProxy is not allocated yet?
> Allocation is moved to LayerTreeHostProxy::updatePaintNode
OK.

> It's simple.
> In Qt case it will be never call from paintToCurrentGLContext, only from updatePaintNode . It left there with default parameter if others use this function.
> 

How about just removing that parameter, and allowing people to call the sync function directly if they so choose. Otherwise it's a boolean trap.

> > You need to do all that just to call updateViewport on the main thread?
I'm sure we can do the callOnMainThread stuff with less code. This is really elaborate.

> No, it can't. Paint node holds the reference to LayerTreeHostProxy. I've mentioned this in ChangeLog.
OK.

> I don't know, but I was getting division errors in this functions when I was running API tests. The item indeed was 0 size and was resized later.
OK, should be a different patch.

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