Bug 1273784 - Part 3: Implement KeyframeEffect(ReadOnly) copy constructor API. draft
authorBoris Chiou <boris.chiou@gmail.com>
Tue, 01 Nov 2016 18:39:06 +0800
changeset 434760 5ec4769528d590626b59bca4b9bd91fe29883e52
parent 434759 adf91dc4da9dfb6a6df2eb974c5622597d62d4d1
child 434761 6b32d86e10b97973ada448ab993347ca680d1818
push id34821
push userbmo:boris.chiou@gmail.com
push dateMon, 07 Nov 2016 09:50:31 +0000
bugs1273784
milestone52.0a1
Bug 1273784 - Part 3: Implement KeyframeEffect(ReadOnly) copy constructor API. MozReview-Commit-ID: 9ju9jeJ8cvq
dom/animation/KeyframeEffect.cpp
dom/animation/KeyframeEffectReadOnly.cpp
--- a/dom/animation/KeyframeEffect.cpp
+++ b/dom/animation/KeyframeEffect.cpp
@@ -46,18 +46,17 @@ KeyframeEffect::Constructor(
                                                  aOptions, aRv);
 }
 
 /* static */ already_AddRefed<KeyframeEffect>
 KeyframeEffect::Constructor(const GlobalObject& aGlobal,
                             KeyframeEffectReadOnly& aSource,
                             ErrorResult& aRv)
 {
-  // TODO: Implement this in the later patches.
-  return nullptr;
+  return ConstructKeyframeEffect<KeyframeEffect>(aGlobal, aSource, aRv);
 }
 
 /* static */ already_AddRefed<KeyframeEffect>
 KeyframeEffect::Constructor(
     const GlobalObject& aGlobal,
     const Nullable<ElementOrCSSPseudoElement>& aTarget,
     JS::Handle<JSObject*> aKeyframes,
     const UnrestrictedDoubleOrKeyframeAnimationOptions& aOptions,
--- a/dom/animation/KeyframeEffectReadOnly.cpp
+++ b/dom/animation/KeyframeEffectReadOnly.cpp
@@ -752,18 +752,17 @@ KeyframeEffectReadOnly::Constructor(
                                                          aRv);
 }
 
 /* static */ already_AddRefed<KeyframeEffectReadOnly>
 KeyframeEffectReadOnly::Constructor(const GlobalObject& aGlobal,
                                     KeyframeEffectReadOnly& aSource,
                                     ErrorResult& aRv)
 {
-  // TODO: Implement this in the later patches.
-  return nullptr;
+  return ConstructKeyframeEffect<KeyframeEffectReadOnly>(aGlobal, aSource, aRv);
 }
 
 void
 KeyframeEffectReadOnly::GetTarget(
     Nullable<OwningElementOrCSSPseudoElement>& aRv) const
 {
   if (!mTarget) {
     aRv.SetNull();