Source/core/animation/KeyframeEffectTest.cpp - Issue 1120003002: [Oilpan] Migrate most classes under core/animations to Oilpan heap.

Unified Diff: Source/core/animation/KeyframeEffectTest.cpp

Issue 1120003002: [Oilpan] Migrate most classes under core/animations to Oilpan heap. (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@master

Patch Set: Resize expect size of Persistent Created 5 years, 7 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

« Source/core/animation/KeyframeEffect.h ('K') | « Source/core/animation/KeyframeEffectModelTest.cpp ('k') | Source/core/animation/LegacyStyleInterpolation.h » ('j') | Source/core/animation/LengthBoxStyleInterpolation.cpp » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')

Index: Source/core/animation/KeyframeEffectTest.cpp

diff --git a/Source/core/animation/KeyframeEffectTest.cpp b/Source/core/animation/KeyframeEffectTest.cpp

index 5ee2ff3651e3d5a0bd3d71c9e8c1681d9f4e5c1a..7bf4dc825f880705a6b6183e2bf17733a3a6ed1d 100644

--- a/Source/core/animation/KeyframeEffectTest.cpp

+++ b/Source/core/animation/KeyframeEffectTest.cpp

@@ -49,11 +49,11 @@ protected:

}

template<typename T>

- static PassRefPtrWillBeRawPtr<KeyframeEffect> createAnimation(Element* element, Vector<Dictionary> keyframeDictionaryVector, T timingInput, ExceptionState& exceptionState)

+ static KeyframeEffect* createAnimation(Element* element, Vector<Dictionary> keyframeDictionaryVector, T timingInput, ExceptionState& exceptionState)

{

return KeyframeEffect::create(element, keyframeDictionaryVector, timingInput, exceptionState);

}

- static PassRefPtrWillBeRawPtr<KeyframeEffect> createAnimation(Element* element, Vector<Dictionary> keyframeDictionaryVector, ExceptionState& exceptionState)

+ static KeyframeEffect* createAnimation(Element* element, Vector<Dictionary> keyframeDictionaryVector, ExceptionState& exceptionState)

{

return KeyframeEffect::create(element, keyframeDictionaryVector, exceptionState);

}

@@ -88,7 +88,7 @@ TEST_F(AnimationKeyframeEffectV8Test, CanCreateAnAnimation)

ASSERT_TRUE(DictionaryHelper::get(jsKeyframes[1], "width", value2));

ASSERT_EQ("0px", value2);

- RefPtrWillBeRawPtr<KeyframeEffect> animation = createAnimation(element.get(), jsKeyframes, 0, exceptionState);

+ KeyframeEffect* animation = createAnimation(element.get(), jsKeyframes, 0, exceptionState);

Element* target = animation->target();

EXPECT_EQ(*element.get(), *target);

@@ -115,7 +115,7 @@ TEST_F(AnimationKeyframeEffectV8Test, CanSetDuration)

Vector<Dictionary, 0> jsKeyframes;

double duration = 2000;

- RefPtrWillBeRawPtr<KeyframeEffect> animation = createAnimation(element.get(), jsKeyframes, duration, exceptionState);

+ KeyframeEffect* animation = createAnimation(element.get(), jsKeyframes, duration, exceptionState);

EXPECT_EQ(duration / 1000, animation->specifiedTiming().iterationDuration);

}

@@ -123,14 +123,14 @@ TEST_F(AnimationKeyframeEffectV8Test, CanSetDuration)

TEST_F(AnimationKeyframeEffectV8Test, CanOmitSpecifiedDuration)

{

Vector<Dictionary, 0> jsKeyframes;

- RefPtrWillBeRawPtr<KeyframeEffect> animation = createAnimation(element.get(), jsKeyframes, exceptionState);

+ KeyframeEffect* animation = createAnimation(element.get(), jsKeyframes, exceptionState);

EXPECT_TRUE(std::isnan(animation->specifiedTiming().iterationDuration));

}

TEST_F(AnimationKeyframeEffectV8Test, NegativeDurationIsAuto)

{

Vector<Dictionary, 0> jsKeyframes;

- RefPtrWillBeRawPtr<KeyframeEffect> animation = createAnimation(element.get(), jsKeyframes, -2, exceptionState);

+ KeyframeEffect* animation = createAnimation(element.get(), jsKeyframes, -2, exceptionState);

EXPECT_TRUE(std::isnan(animation->specifiedTiming().iterationDuration));

}

@@ -233,9 +233,9 @@ TEST_F(AnimationKeyframeEffectV8Test, SpecifiedGetters)

KeyframeEffectOptions timingInputDictionary;

V8KeyframeEffectOptions::toImpl(m_isolate, timingInput, timingInputDictionary, exceptionState);

- RefPtrWillBeRawPtr<KeyframeEffect> animation = createAnimation(element.get(), jsKeyframes, timingInputDictionary, exceptionState);

+ KeyframeEffect* animation = createAnimation(element.get(), jsKeyframes, timingInputDictionary, exceptionState);

- RefPtrWillBeRawPtr<AnimationEffectTiming> specified = animation->timing();

+ AnimationEffectTiming* specified = animation->timing();

EXPECT_EQ(2, specified->delay());

EXPECT_EQ(0.5, specified->endDelay());

EXPECT_EQ("backwards", specified->fill());

@@ -255,9 +255,9 @@ TEST_F(AnimationKeyframeEffectV8Test, SpecifiedDurationGetter)

KeyframeEffectOptions timingInputDictionaryWithDuration;

V8KeyframeEffectOptions::toImpl(m_isolate, timingInputWithDuration, timingInputDictionaryWithDuration, exceptionState);

- RefPtrWillBeRawPtr<KeyframeEffect> animationWithDuration = createAnimation(element.get(), jsKeyframes, timingInputDictionaryWithDuration, exceptionState);

+ KeyframeEffect* animationWithDuration = createAnimation(element.get(), jsKeyframes, timingInputDictionaryWithDuration, exceptionState);

- RefPtrWillBeRawPtr<AnimationEffectTiming> specifiedWithDuration = animationWithDuration->timing();

+ AnimationEffectTiming* specifiedWithDuration = animationWithDuration->timing();

UnrestrictedDoubleOrString duration;

specifiedWithDuration->duration(duration);

EXPECT_TRUE(duration.isUnrestrictedDouble());

@@ -269,9 +269,9 @@ TEST_F(AnimationKeyframeEffectV8Test, SpecifiedDurationGetter)

KeyframeEffectOptions timingInputDictionaryNoDuration;

V8KeyframeEffectOptions::toImpl(m_isolate, timingInputNoDuration, timingInputDictionaryNoDuration, exceptionState);

- RefPtrWillBeRawPtr<KeyframeEffect> animationNoDuration = createAnimation(element.get(), jsKeyframes, timingInputDictionaryNoDuration, exceptionState);

+ KeyframeEffect* animationNoDuration = createAnimation(element.get(), jsKeyframes, timingInputDictionaryNoDuration, exceptionState);

- RefPtrWillBeRawPtr<AnimationEffectTiming> specifiedNoDuration = animationNoDuration->timing();

+ AnimationEffectTiming* specifiedNoDuration = animationNoDuration->timing();

UnrestrictedDoubleOrString duration2;

specifiedNoDuration->duration(duration2);

EXPECT_FALSE(duration2.isUnrestrictedDouble());

@@ -285,9 +285,9 @@ TEST_F(AnimationKeyframeEffectV8Test, SpecifiedSetters)

v8::Local<v8::Object> timingInput = v8::Object::New(m_isolate);

KeyframeEffectOptions timingInputDictionary;

V8KeyframeEffectOptions::toImpl(m_isolate, timingInput, timingInputDictionary, exceptionState);

- RefPtrWillBeRawPtr<KeyframeEffect> animation = createAnimation(element.get(), jsKeyframes, timingInputDictionary, exceptionState);

+ KeyframeEffect* animation = createAnimation(element.get(), jsKeyframes, timingInputDictionary, exceptionState);

- RefPtrWillBeRawPtr<AnimationEffectTiming> specified = animation->timing();

+ AnimationEffectTiming* specified = animation->timing();

EXPECT_EQ(0, specified->delay());

specified->setDelay(2);

@@ -328,9 +328,9 @@ TEST_F(AnimationKeyframeEffectV8Test, SetSpecifiedDuration)

v8::Local<v8::Object> timingInput = v8::Object::New(m_isolate);

KeyframeEffectOptions timingInputDictionary;

V8KeyframeEffectOptions::toImpl(m_isolate, timingInput, timingInputDictionary, exceptionState);

- RefPtrWillBeRawPtr<KeyframeEffect> animation = createAnimation(element.get(), jsKeyframes, timingInputDictionary, exceptionState);

+ KeyframeEffect* animation = createAnimation(element.get(), jsKeyframes, timingInputDictionary, exceptionState);

- RefPtrWillBeRawPtr<AnimationEffectTiming> specified = animation->timing();

+ AnimationEffectTiming* specified = animation->timing();

UnrestrictedDoubleOrString duration;

specified->duration(duration);

@@ -355,8 +355,8 @@ TEST_F(KeyframeEffectTest, TimeToEffectChange)

timing.startDelay = 100;

timing.endDelay = 100;

timing.fillMode = Timing::FillModeNone;

- RefPtrWillBeRawPtr<KeyframeEffect> animation = KeyframeEffect::create(0, nullptr, timing);

- RefPtrWillBeRawPtr<Animation> player = document.timeline().play(animation.get());

+ KeyframeEffect* animation = KeyframeEffect::create(0, nullptr, timing);

+ Animation* player = document.timeline().play(animation);

double inf = std::numeric_limits<double>::infinity();

EXPECT_EQ(100, animation->timeToForwardsEffectChange());

@@ -388,8 +388,8 @@ TEST_F(KeyframeEffectTest, TimeToEffectChangeWithPlaybackRate)

timing.endDelay = 100;

timing.playbackRate = 2;

timing.fillMode = Timing::FillModeNone;

- RefPtrWillBeRawPtr<KeyframeEffect> animation = KeyframeEffect::create(0, nullptr, timing);

- RefPtrWillBeRawPtr<Animation> player = document.timeline().play(animation.get());

+ KeyframeEffect* animation = KeyframeEffect::create(0, nullptr, timing);

+ Animation* player = document.timeline().play(animation);

double inf = std::numeric_limits<double>::infinity();

EXPECT_EQ(100, animation->timeToForwardsEffectChange());

@@ -421,8 +421,8 @@ TEST_F(KeyframeEffectTest, TimeToEffectChangeWithNegativePlaybackRate)

timing.endDelay = 100;

timing.playbackRate = -2;

timing.fillMode = Timing::FillModeNone;

- RefPtrWillBeRawPtr<KeyframeEffect> animation = KeyframeEffect::create(0, nullptr, timing);

- RefPtrWillBeRawPtr<Animation> player = document.timeline().play(animation.get());

+ KeyframeEffect* animation = KeyframeEffect::create(0, nullptr, timing);

+ Animation* player = document.timeline().play(animation);

double inf = std::numeric_limits<double>::infinity();

EXPECT_EQ(100, animation->timeToForwardsEffectChange());

@@ -451,14 +451,11 @@ TEST_F(KeyframeEffectTest, ElementDestructorClearsAnimationTarget)

// and KeyframeEffect are moved to Oilpan. See crbug.com/362404 for context.

Timing timing;

timing.iterationDuration = 5;

- RefPtrWillBeRawPtr<KeyframeEffect> animation = KeyframeEffect::create(element.get(), nullptr, timing);

+ KeyframeEffect* animation = KeyframeEffect::create(element.get(), nullptr, timing);

EXPECT_EQ(element.get(), animation->target());

- document.timeline().play(animation.get());

+ document.timeline().play(animation);

pageHolder.clear();

element.clear();

-#if !ENABLE(OILPAN)

- EXPECT_EQ(0, animation->target());

-#endif

}

} // namespace blink