GraphicsContext::getCTM() does not need to make a copy of the QTransform returned by QPainter::combinedTransform(), it could instead use a const reference. This will avoid a copy for compilers that don't implement (or enable by default) the NRVO optimization.
Created attachment 77575 [details] Patch
Comment on attachment 77575 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=77575&action=review > WebCore/platform/graphics/qt/GraphicsContextQt.cpp:292 > return AffineTransform(matrix.m11(), matrix.m12(), matrix.m21(), Why don't we just have a AffineTransform(const QTransform&) constructor which is compiled on Qt?
I guess that's not the pattern. No other platfrom seems to have custom constructors anymore. Strange, since that's how we (at least used to) do IntPoint, IntRect, etc.
Comment on attachment 77575 [details] Patch Clearing flags on attachment: 77575 Committed r74728: <http://trac.webkit.org/changeset/74728>
All reviewed patches have been landed. Closing bug.
http://trac.webkit.org/changeset/74728 might have broken Leopard Intel Debug (Tests)