Bug 91111 – Move TextureMapperAnimation and texmap/LayerTransform to platform/graphics

• Status: RESOLVED
• Resolution: FIXED
• Component: New Bugs

[Noam Rosenthal]

Move TextureMapperAnimation and texmap/LayerTransform to platform/graphics

[Noam Rosenthal]

Created attachment 151979 [details]
Patch

[Kenneth Rohde Christiansen]

Comment on attachment 151979 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=151979&action=review

> Source/WebCore/ChangeLog:10
> +        TextureMapperAnimation and LayerTransform are not specific to TextureMapper, and we want
> +        to use them for other purposes as well. Moving them to platform/graphics would make that
> +        more explicit.

What about a namechange then to the former?

> Source/WebCore/GNUmakefile.list.am:6128
> +	Source/WebCore/platform/graphics/GraphicsLayerTransform.cpp \
> +	Source/WebCore/platform/graphics/GraphicsLayerTransform.h \
> +	Source/WebCore/platform/graphics/GraphicsLayerAnimation.cpp \
> +	Source/WebCore/platform/graphics/GraphicsLayerAnimation.h \

ah you did that :-)

> Source/WebCore/platform/graphics/GraphicsLayerAnimation.cpp:27
> +#if USE(TEXTURE_MAPPER)
> +namespace WebCore {

You still want this guard?

[Noam Rosenthal]

Created attachment 151987 [details]
Patch for landing

[WebKit Review Bot]

Comment on attachment 151987 [details]
Patch for landing

Clearing flags on attachment: 151987

Committed r122484: <http://trac.webkit.org/changeset/122484>

[WebKit Review Bot]

All reviewed patches have been landed.  Closing bug.