Changeset 62992 in webkit

Timestamp:

Jul 9, 2010 2:04:46 PM (14 years ago)

Author:

aestes@apple.com

Message:

2010-07-09 Andy Estes <​aestes@apple.com>

Reviewed by Adele Peterson.

Remove the workaround for a Core Animation bug on platforms where the
bug has been fixed.
​https://bugs.webkit.org/show_bug.cgi?id=41927
<rdar://problem/7920153>

• platform/graphics/mac/GraphicsLayerCA.mm: (WebCore::GraphicsLayerCA::createTransformAnimationsFromKeyframes): Add a compile-time check for platforms that have a Core Animation bug that needs working around.

Location:

trunk/WebCore

Files:

• ChangeLog (modified) (1 diff)
• platform/graphics/mac/GraphicsLayerCA.mm (modified) (1 diff)

trunk/WebCore/ChangeLog

@@ +1 @@
+2010-07-09  Andy Estes  <aestes@apple.com>
+
+        Reviewed by Adele Peterson.
+
+        Remove the workaround for a Core Animation bug on platforms where the
+        bug has been fixed.
+        https://bugs.webkit.org/show_bug.cgi?id=41927
+        <rdar://problem/7920153>
+
+        * platform/graphics/mac/GraphicsLayerCA.mm:
+        (WebCore::GraphicsLayerCA::createTransformAnimationsFromKeyframes): Add
+        a compile-time check for platforms that have a Core Animation bug that
+        needs working around.
+
 2010-07-08  Tony Gentilcore  <tonyg@chromium.org>
 

trunk/WebCore/platform/graphics/mac/GraphicsLayerCA.mm

@@ -1821 +1821 @@
         CAPropertyAnimation* caAnimation;
 
+#if defined(BUILDING_ON_LEOPARD) || defined(BUILDING_ON_SNOW_LEOPARD)
         // CA applies animations in reverse order (<rdar://problem/7095638>) so we need the last one we add (per property)
         // to be non-additive.
         bool additive = animationIndex < (numAnimations - 1);
+#else
+        bool additive = animationIndex > 0;
+#endif
         if (isKeyframe) {
             CAKeyframeAnimation* keyframeAnim = createKeyframeAnimation(animation, valueList.property(), additive);