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

Unified Diff: cc/test/animation_test_common.cc

Issue 1973083002: Use element id's for animations (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: rebase Created 4 years, 6 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
« no previous file with comments | « cc/test/animation_test_common.h ('k') | cc/test/animation_timelines_test_common.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: cc/test/animation_test_common.cc
diff --git a/cc/test/animation_test_common.cc b/cc/test/animation_test_common.cc
index 2d5dafb7ed91efbabecd344156ed4790e264fb67..7ab2497e598abee7ecff5a1902354bd4f3cc0fe4 100644
--- a/cc/test/animation_test_common.cc
+++ b/cc/test/animation_test_common.cc
@@ -296,49 +296,49 @@ int AddOpacityStepsToElementAnimations(ElementAnimations* target,
return id;
}
-void AddAnimationToLayerWithPlayer(int layer_id,
- scoped_refptr<AnimationTimeline> timeline,
- std::unique_ptr<Animation> animation) {
+void AddAnimationToElementWithPlayer(ElementId element_id,
+ scoped_refptr<AnimationTimeline> timeline,
+ std::unique_ptr<Animation> animation) {
scoped_refptr<AnimationPlayer> player =
AnimationPlayer::Create(AnimationIdProvider::NextPlayerId());
timeline->AttachPlayer(player);
- player->AttachElement(layer_id);
+ player->AttachElement(element_id);
DCHECK(player->element_animations());
player->AddAnimation(std::move(animation));
}
-void AddAnimationToLayerWithExistingPlayer(
- int layer_id,
+void AddAnimationToElementWithExistingPlayer(
+ ElementId element_id,
scoped_refptr<AnimationTimeline> timeline,
std::unique_ptr<Animation> animation) {
scoped_refptr<ElementAnimations> element_animations =
- timeline->animation_host()->GetElementAnimationsForElementId(layer_id);
+ timeline->animation_host()->GetElementAnimationsForElementId(element_id);
DCHECK(element_animations);
element_animations->AddAnimation(std::move(animation));
}
-void RemoveAnimationFromLayerWithExistingPlayer(
- int layer_id,
+void RemoveAnimationFromElementWithExistingPlayer(
+ ElementId element_id,
scoped_refptr<AnimationTimeline> timeline,
int animation_id) {
scoped_refptr<ElementAnimations> element_animations =
- timeline->animation_host()->GetElementAnimationsForElementId(layer_id);
+ timeline->animation_host()->GetElementAnimationsForElementId(element_id);
DCHECK(element_animations);
element_animations->RemoveAnimation(animation_id);
}
-Animation* GetAnimationFromLayerWithExistingPlayer(
- int layer_id,
+Animation* GetAnimationFromElementWithExistingPlayer(
+ ElementId element_id,
scoped_refptr<AnimationTimeline> timeline,
int animation_id) {
scoped_refptr<ElementAnimations> element_animations =
- timeline->animation_host()->GetElementAnimationsForElementId(layer_id);
+ timeline->animation_host()->GetElementAnimationsForElementId(element_id);
DCHECK(element_animations);
return element_animations->GetAnimationById(animation_id);
}
-int AddAnimatedFilterToLayerWithPlayer(
- int layer_id,
+int AddAnimatedFilterToElementWithPlayer(
+ ElementId element_id,
scoped_refptr<AnimationTimeline> timeline,
double duration,
float start_brightness,
@@ -346,14 +346,14 @@ int AddAnimatedFilterToLayerWithPlayer(
scoped_refptr<AnimationPlayer> player =
AnimationPlayer::Create(AnimationIdProvider::NextPlayerId());
timeline->AttachPlayer(player);
- player->AttachElement(layer_id);
+ player->AttachElement(element_id);
DCHECK(player->element_animations());
return AddAnimatedFilterToPlayer(player.get(), duration, start_brightness,
end_brightness);
}
-int AddAnimatedTransformToLayerWithPlayer(
- int layer_id,
+int AddAnimatedTransformToElementWithPlayer(
+ ElementId element_id,
scoped_refptr<AnimationTimeline> timeline,
double duration,
int delta_x,
@@ -361,13 +361,13 @@ int AddAnimatedTransformToLayerWithPlayer(
scoped_refptr<AnimationPlayer> player =
AnimationPlayer::Create(AnimationIdProvider::NextPlayerId());
timeline->AttachPlayer(player);
- player->AttachElement(layer_id);
+ player->AttachElement(element_id);
DCHECK(player->element_animations());
return AddAnimatedTransformToPlayer(player.get(), duration, delta_x, delta_y);
}
-int AddAnimatedTransformToLayerWithPlayer(
- int layer_id,
+int AddAnimatedTransformToElementWithPlayer(
+ ElementId element_id,
scoped_refptr<AnimationTimeline> timeline,
double duration,
TransformOperations start_operations,
@@ -375,14 +375,14 @@ int AddAnimatedTransformToLayerWithPlayer(
scoped_refptr<AnimationPlayer> player =
AnimationPlayer::Create(AnimationIdProvider::NextPlayerId());
timeline->AttachPlayer(player);
- player->AttachElement(layer_id);
+ player->AttachElement(element_id);
DCHECK(player->element_animations());
return AddAnimatedTransformToPlayer(player.get(), duration, start_operations,
operations);
}
-int AddOpacityTransitionToLayerWithPlayer(
- int layer_id,
+int AddOpacityTransitionToElementWithPlayer(
+ ElementId element_id,
scoped_refptr<AnimationTimeline> timeline,
double duration,
float start_opacity,
@@ -391,17 +391,18 @@ int AddOpacityTransitionToLayerWithPlayer(
scoped_refptr<AnimationPlayer> player =
AnimationPlayer::Create(AnimationIdProvider::NextPlayerId());
timeline->AttachPlayer(player);
- player->AttachElement(layer_id);
+ player->AttachElement(element_id);
DCHECK(player->element_animations());
return AddOpacityTransitionToPlayer(player.get(), duration, start_opacity,
end_opacity, use_timing_function);
}
-void AbortAnimationsOnLayerWithPlayer(int layer_id,
- scoped_refptr<AnimationTimeline> timeline,
- TargetProperty::Type target_property) {
+void AbortAnimationsOnElementWithPlayer(
+ ElementId element_id,
+ scoped_refptr<AnimationTimeline> timeline,
+ TargetProperty::Type target_property) {
scoped_refptr<ElementAnimations> element_animations =
- timeline->animation_host()->GetElementAnimationsForElementId(layer_id);
+ timeline->animation_host()->GetElementAnimationsForElementId(element_id);
DCHECK(element_animations);
element_animations->AbortAnimations(target_property);
}
« no previous file with comments | « cc/test/animation_test_common.h ('k') | cc/test/animation_timelines_test_common.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698