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

Unified Diff: cc/animation/element_animations.h

Issue 2357533002: CC Animation: Use std::bitset to update animation state. (Closed)
Patch Set: Reparent. Created 4 years, 2 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/animation/animation_player.cc ('k') | cc/animation/element_animations.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: cc/animation/element_animations.h
diff --git a/cc/animation/element_animations.h b/cc/animation/element_animations.h
index ce514ae63190dd0d34e200ff7e91b8c5656d5a68..c72d324aae3e10fc17108d61700f5c12031ab3a3 100644
--- a/cc/animation/element_animations.h
+++ b/cc/animation/element_animations.h
@@ -179,10 +179,8 @@ class CC_EXPORT ElementAnimations : public base::RefCounted<ElementAnimations> {
void SetNeedsPushProperties();
bool needs_push_properties() const { return needs_push_properties_; }
- // TODO(loyso): Rework UpdateClientAnimationState to use bitset.
- void UpdateClientAnimationState(TargetProperty::Type target_property);
- void UpdateClientAnimationState(bool transform, bool opacity, bool filter);
- void SetNeedsUpdateImplClientState(bool transform, bool opacity, bool filter);
+ void UpdateClientAnimationState();
+ void SetNeedsUpdateImplClientState();
void UpdateActivationNormal();
@@ -209,14 +207,6 @@ class CC_EXPORT ElementAnimations : public base::RefCounted<ElementAnimations> {
enum class ActivationType { NORMAL, FORCE };
void UpdateActivation(ActivationType type);
- void UpdateClientAnimationStateInternal(TargetProperty::Type property);
-
- void NotifyClientAnimationChanged(
- TargetProperty::Type property,
- ElementListType list_type,
- bool notify_elements_about_potential_animation,
- bool notify_elements_about_running_animation);
-
void OnFilterAnimated(ElementListType list_type,
const FilterOperations& filters);
void OnOpacityAnimated(ElementListType list_type, float opacity);
@@ -224,12 +214,8 @@ class CC_EXPORT ElementAnimations : public base::RefCounted<ElementAnimations> {
const gfx::Transform& transform);
void OnScrollOffsetAnimated(ElementListType list_type,
const gfx::ScrollOffset& scroll_offset);
- void IsAnimatingChanged(ElementListType list_type,
- TargetProperty::Type property,
- AnimationChangeType change_type,
- bool is_animating);
- void ClearNeedsUpdateImplClientState();
+ static TargetProperties GetPropertiesMaskForAnimationState();
std::unique_ptr<PlayersList> players_list_;
AnimationHost* animation_host_;
@@ -247,13 +233,10 @@ class CC_EXPORT ElementAnimations : public base::RefCounted<ElementAnimations> {
bool needs_push_properties_;
- PropertyAnimationState filter_animation_state_;
- PropertyAnimationState opacity_animation_state_;
- PropertyAnimationState transform_animation_state_;
+ PropertyAnimationState active_state_;
+ PropertyAnimationState pending_state_;
- bool needs_update_impl_client_state_transform_ : 1;
- bool needs_update_impl_client_state_opacity_ : 1;
- bool needs_update_impl_client_state_filter_ : 1;
+ bool needs_update_impl_client_state_;
DISALLOW_COPY_AND_ASSIGN(ElementAnimations);
};
« no previous file with comments | « cc/animation/animation_player.cc ('k') | cc/animation/element_animations.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698