[Webkit-unassigned] [Bug 70407] [TexMap][QT] PageClientQt should skip redundant sync requests during Accelerated Composition.

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Mon Nov 14 20:41:11 PST 2011


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





--- Comment #6 from Noam Rosenthal <noam.rosenthal at nokia.com>  2011-11-14 20:41:11 PST ---
(In reply to comment #5)
> You're welcome.
> 
> I understood "shouldSync" is for makeForSync, but I did not understand yet why there is not something like this code.
> if (shouldSync)
>     doSomething();
> 
> Anyway, I am not sure whether the situation related to 600 sync as I mentioned is bug or not.
> If it is bug, is it proper to add PageClientQGraphicsWidget's member variable like shouldSync in order to reduce redundant markForSync calls?

Hmmm... looking at it again some of that logic doesn't make sense, and might be leftovers from a previous iteration.
But if we fix it let's do it right!
Maybe the right fix should be to eliminate shouldSync, and to simply start the timer on markForSync?

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



More information about the webkit-unassigned mailing list