RESOLVED FIXED 48624
[Qt] GraphicsLayer: Don't notifySyncRequired() more than necessary 
https://bugs.webkit.org/show_bug.cgi?id=48624
Summary [Qt] GraphicsLayer: Don't notifySyncRequired() more than necessary
Andreas Kling
Reported 2010-10-29 04:29:05 PDT
If there's already a notifySyncRequired() queued, we should only update the ChangeMask. We currently queue a call (using QMetaMethod::invoke) every time notifyChange() is called. This is expensive, and also makes GraphicsLayerQt expensive to destroy (since all posted method invocation events have to be ripped out of the event queue.)
Attachments
Proposed patch (2.43 KB, patch)
2010-10-29 04:37 PDT, Andreas Kling 		no flags
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Andreas Kling
Comment 1 2010-10-29 04:37:14 PDT
Created attachment 72314 [details] Proposed patch 4% speedup on <http://ariya.github.com/clock.htm>
Andreas Kling
Comment 2 2010-10-29 06:39:43 PDT
Committed r70868: <http://trac.webkit.org/changeset/70868>
Note You need to log in before you can comment on or make changes to this bug.