[TexMap] GraphicsLayerTextureMapper::addAnimation() box size parameter should be FloatSize
Created attachment 235426 [details] Patch
Comment on attachment 235426 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=235426&action=review > Source/WebCore/platform/graphics/texmap/GraphicsLayerTextureMapper.h:84 > - virtual bool addAnimation(const KeyframeValueList&, const IntSize&, const Animation*, const String&, double); > + virtual bool addAnimation(const KeyframeValueList&, const FloatSize&, const Animation*, const String&, double); I think adding override to this delcaration is a good idea, since it has the potential to catch these errors in the future.
Comment on attachment 235426 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=235426&action=review >> Source/WebCore/platform/graphics/texmap/GraphicsLayerTextureMapper.h:84 >> + virtual bool addAnimation(const KeyframeValueList&, const FloatSize&, const Animation*, const String&, double); > > I think adding override to this delcaration is a good idea, since it has the potential to catch these errors in the future. That should be done throughout the texmap code. I'll at least run clang-modernize over it, but there are also other things I want to improve.
Comment on attachment 235426 [details] Patch Clearing flags on attachment: 235426 Committed r171725: <http://trac.webkit.org/changeset/171725>
All reviewed patches have been landed. Closing bug.