RESOLVED FIXED 42456
[Qt] Path: Avoid creating a new GraphicsContext in strokeContains() and strokeBoundingRect() 
https://bugs.webkit.org/show_bug.cgi?id=42456
Summary [Qt] Path: Avoid creating a new GraphicsContext in strokeContains() and strok...
Andreas Kling
Reported 2010-07-16 08:22:49 PDT
From PathQt.cpp: // FIXME: We should try to use a 'shared Context' instead of creating a new ImageBuffer // on each call. We should use the same "scratch context" pattern Skia uses.
Attachments
Proposed patch (2.24 KB, patch)
2010-07-16 08:26 PDT, Andreas Kling 		no flags
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Andreas Kling
Comment 1 2010-07-16 08:26:14 PDT
Created attachment 61808 [details] Proposed patch
Andreas Kling
Comment 2 2010-07-16 08:32:34 PDT
This change yields a 7% speedup on http://www.openrise.com/lab/FlowerPower/
Antonio Gomes
Comment 3 2010-07-16 08:34:24 PDT
Comment on attachment 61808 [details] Proposed patch Looks sane. r=me
Andreas Kling
Comment 4 2010-07-16 08:50:16 PDT
Comment on attachment 61808 [details] Proposed patch Clearing flags on attachment: 61808 Committed r63552: <http://trac.webkit.org/changeset/63552>
Andreas Kling
Comment 5 2010-07-16 08:50:26 PDT
All reviewed patches have been landed. Closing bug.
Andreas Kling
Comment 6 2010-07-16 08:59:38 PDT
Marking for cherry-pick into 2.0 since it's a very nice improvement for a very tiny change.
Note You need to log in before you can comment on or make changes to this bug.