Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(964)

Unified Diff: Source/core/svg/SVGAnimatedType.cpp

Issue 153883003: [SVG] SVGAnimatedTransform{,List} migration to new SVG property impl. (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: haraken review Created 6 years, 10 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: Source/core/svg/SVGAnimatedType.cpp
diff --git a/Source/core/svg/SVGAnimatedType.cpp b/Source/core/svg/SVGAnimatedType.cpp
index 6c93bab8eaa2e7d14f2340772252b04e0eb0f7d5..5430029d9725adaed8923a43a262c1cac133b7f1 100644
--- a/Source/core/svg/SVGAnimatedType.cpp
+++ b/Source/core/svg/SVGAnimatedType.cpp
@@ -49,9 +49,6 @@ SVGAnimatedType::~SVGAnimatedType()
case AnimatedPath:
delete m_data.path;
break;
- case AnimatedTransformList:
- delete m_data.transformList;
- break;
// Below properties are migrated to new property implementation.
case AnimatedBoolean:
case AnimatedColor:
@@ -65,11 +62,13 @@ SVGAnimatedType::~SVGAnimatedType()
case AnimatedRect:
case AnimatedString:
case AnimatedStringList:
+ case AnimatedTransformList:
// handled by RefPtr
break;
- // There is no SVGAnimatedPoint
+ // There is no SVGAnimated{Point,Transform}
case AnimatedPoint:
+ case AnimatedTransform:
ASSERT_NOT_REACHED();
break;
@@ -119,14 +118,6 @@ PassOwnPtr<SVGAnimatedType> SVGAnimatedType::createPath(PassOwnPtr<SVGPathByteSt
return animatedType.release();
}
-PassOwnPtr<SVGAnimatedType> SVGAnimatedType::createTransformList(SVGTransformList* transformList)
-{
- ASSERT(transformList);
- OwnPtr<SVGAnimatedType> animatedType = adoptPtr(new SVGAnimatedType(AnimatedTransformList));
- animatedType->m_data.transformList = transformList;
- return animatedType.release();
-}
-
PassOwnPtr<SVGAnimatedType> SVGAnimatedType::createNewProperty(PassRefPtr<NewSVGPropertyBase> newProperty)
{
ASSERT(newProperty);
@@ -160,6 +151,7 @@ String SVGAnimatedType::valueAsString()
case AnimatedIntegerOptionalInteger:
case AnimatedPath:
case AnimatedPoint:
+ case AnimatedTransform:
case AnimatedTransformList:
case AnimatedUnknown:
// Only SVG DOM animations use these property types - that means valueAsString() is never used for those.
@@ -196,6 +188,7 @@ bool SVGAnimatedType::setValueAsString(const QualifiedName& attrName, const Stri
case AnimatedIntegerOptionalInteger:
case AnimatedPath:
case AnimatedPoint:
+ case AnimatedTransform:
case AnimatedTransformList:
case AnimatedUnknown:
// Only SVG DOM animations use these property types - that means setValueAsString() is never used for those.

Powered by Google App Engine
This is Rietveld 408576698