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

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: remove m_zoomAndPan 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
« no previous file with comments | « Source/core/svg/SVGAnimatedType.h ('k') | Source/core/svg/SVGAnimatorFactory.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: Source/core/svg/SVGAnimatedType.cpp
diff --git a/Source/core/svg/SVGAnimatedType.cpp b/Source/core/svg/SVGAnimatedType.cpp
index 4e478ff6323ed8c59a4c1f5fa94173e1204cdbc7..4f892d99642139c0726b5f88015a2b5955888fa2 100644
--- a/Source/core/svg/SVGAnimatedType.cpp
+++ b/Source/core/svg/SVGAnimatedType.cpp
@@ -37,9 +37,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 AnimatedAngle:
case AnimatedBoolean:
@@ -57,11 +54,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;
@@ -79,14 +78,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);
@@ -120,6 +111,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.
@@ -156,6 +148,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.
« no previous file with comments | « Source/core/svg/SVGAnimatedType.h ('k') | Source/core/svg/SVGAnimatorFactory.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698