Move CanvasGradient and CanvasPattern in the union of CanvasStyle
Created attachment 200975 [details] Patch
Comment on attachment 200975 [details] Patch Clearing flags on attachment: 200975 Committed r149696: <http://trac.webkit.org/changeset/149696>
All reviewed patches have been landed. Closing bug.
Comment on attachment 200975 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=200975&action=review > Source/WebCore/html/canvas/CanvasStyle.cpp:130 > - , m_gradient(gradient) > + , m_gradient(gradient.leakRef()) > { > + m_gradient->ref(); This looks like a storage leak to me. We already have a ref because of the call to leakRef, no need to call ref again. > Source/WebCore/html/canvas/CanvasStyle.cpp:137 > - , m_pattern(pattern) > + , m_pattern(pattern.leakRef()) > { > + m_pattern->ref(); Ditto. Am I right?
Damn! That is right. Follow up: http://trac.webkit.org/changeset/149706