Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(345)

Unified Diff: ui/views/animation/ink_drop_animation_unittest.cc

Issue 1945743003: Added a views::test::TestInkDropAnimationObserverHelper (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@2704
Patch Set: Created 4 years, 8 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
Index: ui/views/animation/ink_drop_animation_unittest.cc
diff --git a/ui/views/animation/ink_drop_animation_unittest.cc b/ui/views/animation/ink_drop_animation_unittest.cc
index 84d608ea367a6f8934bad56ec334bee1a435de87..e1ff62c456ca6e487d46c147fec7c9f34ce00516 100644
--- a/ui/views/animation/ink_drop_animation_unittest.cc
+++ b/ui/views/animation/ink_drop_animation_unittest.cc
@@ -179,8 +179,8 @@ TEST_P(InkDropAnimationTest, AnimationsAbortedDuringDeletion) {
EXPECT_EQ(1, observer_.last_animation_started_ordinal());
EXPECT_EQ(2, observer_.last_animation_ended_ordinal());
EXPECT_EQ(views::InkDropState::ACTION_PENDING,
- observer_.last_animation_state_ended());
- EXPECT_EQ(InkDropAnimationObserver::InkDropAnimationEndedReason::PRE_EMPTED,
+ observer_.last_animation_ended_context());
+ EXPECT_EQ(InkDropAnimationEndedReason::PRE_EMPTED,
observer_.last_animation_ended_reason());
}
@@ -191,7 +191,7 @@ TEST_P(InkDropAnimationTest, VerifyObserversAreNotified) {
EXPECT_EQ(1, observer_.last_animation_started_ordinal());
EXPECT_TRUE(observer_.AnimationHasNotEnded());
EXPECT_EQ(InkDropState::ACTION_PENDING,
- observer_.last_animation_state_started());
+ observer_.last_animation_started_context());
test_api_->CompleteAnimations();
@@ -199,7 +199,7 @@ TEST_P(InkDropAnimationTest, VerifyObserversAreNotified) {
EXPECT_EQ(1, observer_.last_animation_started_ordinal());
EXPECT_EQ(2, observer_.last_animation_ended_ordinal());
EXPECT_EQ(InkDropState::ACTION_PENDING,
- observer_.last_animation_state_ended());
+ observer_.last_animation_ended_context());
}
TEST_P(InkDropAnimationTest, VerifyObserversAreNotifiedOfSuccessfulAnimations) {
@@ -207,7 +207,7 @@ TEST_P(InkDropAnimationTest, VerifyObserversAreNotifiedOfSuccessfulAnimations) {
test_api_->CompleteAnimations();
EXPECT_EQ(2, observer_.last_animation_ended_ordinal());
- EXPECT_EQ(InkDropAnimationObserver::InkDropAnimationEndedReason::SUCCESS,
+ EXPECT_EQ(InkDropAnimationEndedReason::SUCCESS,
observer_.last_animation_ended_reason());
}
@@ -216,7 +216,7 @@ TEST_P(InkDropAnimationTest, VerifyObserversAreNotifiedOfPreemptedAnimations) {
ink_drop_animation_->AnimateToState(InkDropState::ALTERNATE_ACTION_PENDING);
EXPECT_EQ(2, observer_.last_animation_ended_ordinal());
- EXPECT_EQ(InkDropAnimationObserver::InkDropAnimationEndedReason::PRE_EMPTED,
+ EXPECT_EQ(InkDropAnimationEndedReason::PRE_EMPTED,
observer_.last_animation_ended_reason());
}
@@ -264,8 +264,8 @@ TEST_P(InkDropAnimationTest, HideImmediatelyWithActiveAnimations) {
EXPECT_EQ(1, observer_.last_animation_started_ordinal());
EXPECT_EQ(2, observer_.last_animation_ended_ordinal());
EXPECT_EQ(InkDropState::ACTION_PENDING,
- observer_.last_animation_state_ended());
- EXPECT_EQ(InkDropAnimationObserver::InkDropAnimationEndedReason::PRE_EMPTED,
+ observer_.last_animation_ended_context());
+ EXPECT_EQ(InkDropAnimationEndedReason::PRE_EMPTED,
observer_.last_animation_ended_reason());
EXPECT_EQ(InkDropAnimation::kHiddenOpacity, test_api_->GetCurrentOpacity());
@@ -310,8 +310,8 @@ TEST_P(InkDropAnimationTest, SnapToActivatedWithActiveAnimations) {
ink_drop_animation_->target_ink_drop_state());
EXPECT_EQ(3, observer_.last_animation_started_ordinal());
EXPECT_EQ(4, observer_.last_animation_ended_ordinal());
- EXPECT_EQ(InkDropState::ACTIVATED, observer_.last_animation_state_ended());
- EXPECT_EQ(InkDropAnimationObserver::InkDropAnimationEndedReason::SUCCESS,
+ EXPECT_EQ(InkDropState::ACTIVATED, observer_.last_animation_ended_context());
+ EXPECT_EQ(InkDropAnimationEndedReason::SUCCESS,
observer_.last_animation_ended_reason());
EXPECT_EQ(InkDropAnimation::kVisibleOpacity, test_api_->GetCurrentOpacity());
« no previous file with comments | « ui/views/animation/ink_drop_animation_observer.cc ('k') | ui/views/animation/test/test_ink_drop_animation_observer.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698