[TexMap] Avoid unnecessary TransformationMatrix copies in GraphicsLayerTransform
Created attachment 245075 [details] Patch
Comment on attachment 245075 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=245075&action=review r=me % nits. > Source/WebCore/platform/graphics/GraphicsLayerTransform.h:39 > + const TransformationMatrix& combined(); Why isn't this method const? > Source/WebCore/platform/graphics/GraphicsLayerTransform.h:40 > + const TransformationMatrix& combinedForChildren(); Ditto.
Comment on attachment 245075 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=245075&action=review >> Source/WebCore/platform/graphics/GraphicsLayerTransform.h:40 >> + const TransformationMatrix& combinedForChildren(); > > Ditto. This calls combineTransformsForChildren(), which modifies m_childrenDirty and m_combinedForChildren. I prefer combinedForChildren() and combineTransformsForChildren() to be const and have m_childrenDirty and m_combinedForChildren changed to mutable.
Landed in r179462. http://trac.webkit.org/changeset/179462