CLOSED FIXED 36365
[Qt] [Performance] GraphicsLayer: constructing the layers takes too long 
https://bugs.webkit.org/show_bug.cgi?id=36365
Summary [Qt] [Performance] GraphicsLayer: constructing the layers takes too long
Noam Rosenthal
Reported 2010-03-19 07:08:56 PDT
There's a wait of ~350ms on some machines between clicking on the animation and until it starts. That seems to be related to overhead created when constructing the GraphicsLayer scene.
Attachments
make GraphicsLayerQt manage the cache itself instead of using QGraphicsItem::cache (7.53 KB, patch)
2010-04-21 13:10 PDT, Noam Rosenthal 		no flags
DetailsFormatted DiffDiff
Fixed a problem in the first patch. Tested with all relevant LayoutTests (7.75 KB, patch)
2010-04-21 15:00 PDT, Noam Rosenthal 		no flags
DetailsFormatted DiffDiff
style/comments (8.08 KB, patch)
2010-04-21 15:15 PDT, Noam Rosenthal 		no flags
DetailsFormatted DiffDiff
Show Obsolete (2) View All Add attachment proposed patch, testcase, etc.
Noam Rosenthal
Comment 1 2010-04-21 13:10:51 PDT
Created attachment 53981 [details] make GraphicsLayerQt manage the cache itself instead of using QGraphicsItem::cache
Noam Rosenthal
Comment 2 2010-04-21 15:00:53 PDT
Created attachment 53995 [details] Fixed a problem in the first patch. Tested with all relevant LayoutTests
Noam Rosenthal
Comment 3 2010-04-21 15:15:04 PDT
Created attachment 53996 [details] style/comments
WebKit Commit Bot
Comment 4 2010-04-23 15:02:26 PDT
Comment on attachment 53996 [details] style/comments Clearing flags on attachment: 53996 Committed r58190: <http://trac.webkit.org/changeset/58190>
WebKit Commit Bot
Comment 5 2010-04-23 15:02:31 PDT
All reviewed patches have been landed. Closing bug.
Simon Hausmann
Comment 6 2010-04-26 03:23:55 PDT
Revision r58190 cherry-picked into qtwebkit-2.0 with commit 991f4f6002271f4f98cf6a9057184bf3954a1d0c
Note You need to log in before you can comment on or make changes to this bug.