Only create instances of WebTransformOperations using cc::TransformOperations::Create

webkit/compositor_bindings/web_transform_animation_curve_impl.cc

 // Copyright 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "web_transform_animation_curve_impl.h"
 
 #include "cc/keyframed_animation_curve.h"
 #include "cc/timing_function.h"
 #include "web_animation_curve_common.h"
 
@@ skipping 13 matching lines @@
     return WebAnimationCurve::AnimationCurveTypeTransform;
 }
 
 void WebTransformAnimationCurveImpl::add(const WebTransformKeyframe& keyframe)
 {
     add(keyframe, TimingFunctionTypeEase);
 }
 
 void WebTransformAnimationCurveImpl::add(const WebTransformKeyframe& keyframe, TimingFunctionType type)
 {
+#ifdef WEBKIT_WEB_TRANSFORM_OPERATIONS_USE_CREATE
+    m_curve->addKeyframe(cc::TransformKeyframe::create(keyframe.time(), keyframe.value(), createTimingFunction(type)));

[ajuma, 2012/12/20 16:48:11]

WebTransformKeyframe will no longer be able to directly expose an instance of
WebTransformOperations, so we'll need to use an accessor method.

+#else
     m_curve->addKeyframe(cc::TransformKeyframe::create(keyframe.time, keyframe.value, createTimingFunction(type)));
+#endif
 }
 
 void WebTransformAnimationCurveImpl::add(const WebTransformKeyframe& keyframe, double x1, double y1, double x2, double y2)
 {
+#ifdef WEBKIT_WEB_TRANSFORM_OPERATIONS_USE_CREATE
+    m_curve->addKeyframe(cc::TransformKeyframe::create(keyframe.time(), keyframe.value(), cc::CubicBezierTimingFunction::create(x1, y1, x2, y2).PassAs<cc::TimingFunction>()));
+#else
     m_curve->addKeyframe(cc::TransformKeyframe::create(keyframe.time, keyframe.value, cc::CubicBezierTimingFunction::create(x1, y1, x2, y2).PassAs<cc::TimingFunction>()));
+#endif
 }
 
 WebTransformationMatrix WebTransformAnimationCurveImpl::getValue(double time) const
 {
     return m_curve->getValue(time);
 }
 
 scoped_ptr<cc::AnimationCurve> WebTransformAnimationCurveImpl::cloneToAnimationCurve() const
 {
     return m_curve->clone();
 }
 
 } // namespace WebKit