Changeset 76358 in webkit

Timestamp:

Jan 21, 2011 10:31:02 AM (13 years ago)

Author:

Adam Roben

Message:

Clean up PlatformCAAnimationWin

Fixes <​http://webkit.org/b/52904> PlatformCAAnimationWin is leaky and inefficient

Reviewed by Simon Fraser.

• platform/graphics/ca/win/PlatformCAAnimationWin.cpp:

(toCACFFillModeType):
(fromCACFFillModeType):
(toCACFValueFunctionType):
(fromCACFValueFunctionType):
Changed to take and return CFStringRefs. There's no need to convert to WebCore::String just
so we can later convert back to CFStringRef.

(toCACFTimingFunction): Fixed leaks by changing this to return a RetainPtr and adopting the
results of CACFTimingFunctionCreate.
(PlatformCAAnimation::PlatformCAAnimation): Changed not to needlessly roundtrip through
WebCore::String. Also changed an ASSERT(0) to ASSERT_NOT_REACHED().

(PlatformCAAnimation::setFillMode):
(PlatformCAAnimation::setTimingFunction):
(PlatformCAAnimation::setValueFunction):
(PlatformCAAnimation::setTimingFunctions):
Updated for changes to the above conversion functions.

Location:

trunk/Source/WebCore

Files:

• ChangeLog (modified) (1 diff)
• platform/graphics/ca/win/PlatformCAAnimationWin.cpp (modified) (9 diffs)

trunk/Source/WebCore/ChangeLog

@@ +1 @@
+2011-01-21  Adam Roben  <aroben@apple.com>
+
+        Clean up PlatformCAAnimationWin
+
+        Fixes <http://webkit.org/b/52904> PlatformCAAnimationWin is leaky and inefficient
+
+        Reviewed by Simon Fraser.
+
+        * platform/graphics/ca/win/PlatformCAAnimationWin.cpp:
+        (toCACFFillModeType):
+        (fromCACFFillModeType):
+        (toCACFValueFunctionType):
+        (fromCACFValueFunctionType):
+        Changed to take and return CFStringRefs. There's no need to convert to WebCore::String just
+        so we can later convert back to CFStringRef.
+
+        (toCACFTimingFunction): Fixed leaks by changing this to return a RetainPtr and adopting the
+        results of CACFTimingFunctionCreate.
+        (PlatformCAAnimation::PlatformCAAnimation): Changed not to needlessly roundtrip through
+        WebCore::String. Also changed an ASSERT(0) to ASSERT_NOT_REACHED().
+
+        (PlatformCAAnimation::setFillMode):
+        (PlatformCAAnimation::setTimingFunction):
+        (PlatformCAAnimation::setValueFunction):
+        (PlatformCAAnimation::setTimingFunctions):
+        Updated for changes to the above conversion functions.
+
 2011-01-21  Charlie Reis  <creis@chromium.org>
 

trunk/Source/WebCore/platform/graphics/ca/win/PlatformCAAnimationWin.cpp

@@ -42 +42 @@
 using namespace WebCore;
 
-static String toCACFFillModeType(PlatformCAAnimation::FillModeType type)
+static CFStringRef toCACFFillModeType(PlatformCAAnimation::FillModeType type)
 {
     switch (type) {
@@ -50 +50 @@
     case PlatformCAAnimation::Both: return kCACFFillModeBoth;
     }
-    return "";
-}
-
-static PlatformCAAnimation::FillModeType fromCACFFillModeType(const String& string)
+    ASSERT_NOT_REACHED();
+    return 0;
+}
+
+static PlatformCAAnimation::FillModeType fromCACFFillModeType(CFStringRef string)
 {
     if (string == kCACFFillModeBackwards)
@@ -64 +65 @@
 }
 
-static String toCACFValueFunctionType(PlatformCAAnimation::ValueFunctionType type)
+static CFStringRef toCACFValueFunctionType(PlatformCAAnimation::ValueFunctionType type)
 {
     switch (type) {
-    case PlatformCAAnimation::NoValueFunction: return "";
+    case PlatformCAAnimation::NoValueFunction: return 0;
     case PlatformCAAnimation::RotateX: return kCACFValueFunctionRotateX;
     case PlatformCAAnimation::RotateY: return kCACFValueFunctionRotateY;
@@ -80 +81 @@
     case PlatformCAAnimation::Translate: return kCACFValueFunctionTranslate;
     }
-    return "";
-}
-
-static PlatformCAAnimation::ValueFunctionType fromCACFValueFunctionType(const String string)
+    ASSERT_NOT_REACHED();
+    return 0;
+}
+
+static PlatformCAAnimation::ValueFunctionType fromCACFValueFunctionType(CFStringRef string)
 {
     if (string == kCACFValueFunctionRotateX)
@@ -121 +123 @@
 }
 
-static CACFTimingFunctionRef toCACFTimingFunction(const TimingFunction* timingFunction)
+static RetainPtr<CACFTimingFunctionRef> toCACFTimingFunction(const TimingFunction* timingFunction)
 {
     if (!timingFunction)
-        return CACFTimingFunctionCreate(0.25f, 0.1f, 0.25f, 0.1f);
+        return RetainPtr<CACFTimingFunctionRef>(AdoptCF, CACFTimingFunctionCreate(0.25f, 0.1f, 0.25f, 0.1f));
             
     if (timingFunction->isCubicBezierTimingFunction()) {
         const CubicBezierTimingFunction* ctf = static_cast<const CubicBezierTimingFunction*>(timingFunction);
-        return CACFTimingFunctionCreate(static_cast<float>(ctf->x1()), static_cast<float>(ctf->y1()), static_cast<float>(ctf->x2()), static_cast<float>(ctf->y2()));
+        return RetainPtr<CACFTimingFunctionRef>(AdoptCF, CACFTimingFunctionCreate(static_cast<float>(ctf->x1()), static_cast<float>(ctf->y1()), static_cast<float>(ctf->x2()), static_cast<float>(ctf->y2())));
     }
     
@@ -163 +165 @@
 PlatformCAAnimation::PlatformCAAnimation(PlatformAnimationRef animation)
 {
-    if (String(CACFAnimationGetClass(animation)) == kCACFBasicAnimation)
+    if (CACFAnimationGetClass(animation) == kCACFBasicAnimation)
         m_type = Basic;
-    else if (String(CACFAnimationGetClass(animation)) == kCACFKeyframeAnimation)
+    else if (CACFAnimationGetClass(animation) == kCACFKeyframeAnimation)
         m_type = Keyframe;
     else {
-        ASSERT(0);
+        ASSERT_NOT_REACHED();
         return;
     }
@@ -290 +292 @@
 void PlatformCAAnimation::setFillMode(FillModeType value)
 {
-    RetainPtr<CFStringRef> keyPath(AdoptCF, toCACFFillModeType(value).createCFString());
-    CACFAnimationSetFillMode(m_animation.get(), keyPath.get());
+    CACFAnimationSetFillMode(m_animation.get(), toCACFFillModeType(value));
 }
 
 void PlatformCAAnimation::setTimingFunction(const TimingFunction* value)
 {
-    CACFAnimationSetTimingFunction(m_animation.get(), toCACFTimingFunction(value));
+    CACFAnimationSetTimingFunction(m_animation.get(), toCACFTimingFunction(value).get());
 }
 
@@ -331 +332 @@
 void PlatformCAAnimation::setValueFunction(ValueFunctionType value)
 {
-    RetainPtr<CFStringRef> keyPath(AdoptCF, toCACFValueFunctionType(value).createCFString());
-    CACFAnimationSetValueFunction(m_animation.get(), CACFValueFunctionGetFunctionWithName(keyPath.get()));
+    CACFAnimationSetValueFunction(m_animation.get(), CACFValueFunctionGetFunctionWithName(toCACFValueFunctionType(value)));
 }
 
@@ -525 +525 @@
     for (size_t i = 0; i < value.size(); ++i) {
         RetainPtr<CFNumberRef> v(AdoptCF, CFNumberCreate(0, kCFNumberFloatType, &value[i]));
-        CFArrayAppendValue(array.get(), toCACFTimingFunction(value[i]));
+        CFArrayAppendValue(array.get(), toCACFTimingFunction(value[i]).get());
     }