Commit cfda2651 authored by Olga Gerchikov's avatar Olga Gerchikov Committed by Commit Bot

Removed unused members from cc:AnimationEvent.

This change removes opacity, transform and filters members of cc:AnimationEvent that are
not used anymore.


Bug: 1013715
Change-Id: Ide6889a26ae39ac86d904fd5e2135350af3f0b05
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1863731Reviewed-by: default avatarRobert Flack <flackr@chromium.org>
Commit-Queue: Olga Gerchikov <gerchiko@microsoft.com>
Cr-Commit-Position: refs/heads/master@{#706463}
parent fa43fbd0
...@@ -18,7 +18,6 @@ AnimationEvent::AnimationEvent(AnimationEvent::Type type, ...@@ -18,7 +18,6 @@ AnimationEvent::AnimationEvent(AnimationEvent::Type type,
target_property(target_property), target_property(target_property),
monotonic_time(monotonic_time), monotonic_time(monotonic_time),
is_impl_only(false), is_impl_only(false),
opacity(0.f),
local_time() {} local_time() {}
AnimationEvent::AnimationEvent(WorkletAnimationId worklet_animation_id, AnimationEvent::AnimationEvent(WorkletAnimationId worklet_animation_id,
...@@ -30,7 +29,6 @@ AnimationEvent::AnimationEvent(WorkletAnimationId worklet_animation_id, ...@@ -30,7 +29,6 @@ AnimationEvent::AnimationEvent(WorkletAnimationId worklet_animation_id,
target_property(), target_property(),
monotonic_time(), monotonic_time(),
is_impl_only(false), is_impl_only(false),
opacity(0.f),
local_time(local_time) {} local_time(local_time) {}
AnimationEvent::AnimationEvent(const AnimationEvent& other) { AnimationEvent::AnimationEvent(const AnimationEvent& other) {
...@@ -40,9 +38,6 @@ AnimationEvent::AnimationEvent(const AnimationEvent& other) { ...@@ -40,9 +38,6 @@ AnimationEvent::AnimationEvent(const AnimationEvent& other) {
target_property = other.target_property; target_property = other.target_property;
monotonic_time = other.monotonic_time; monotonic_time = other.monotonic_time;
is_impl_only = other.is_impl_only; is_impl_only = other.is_impl_only;
opacity = other.opacity;
transform = other.transform;
filters = other.filters;
animation_start_time = other.animation_start_time; animation_start_time = other.animation_start_time;
if (other.curve) if (other.curve)
curve = other.curve->Clone(); curve = other.curve->Clone();
...@@ -57,9 +52,6 @@ AnimationEvent& AnimationEvent::operator=(const AnimationEvent& other) { ...@@ -57,9 +52,6 @@ AnimationEvent& AnimationEvent::operator=(const AnimationEvent& other) {
target_property = other.target_property; target_property = other.target_property;
monotonic_time = other.monotonic_time; monotonic_time = other.monotonic_time;
is_impl_only = other.is_impl_only; is_impl_only = other.is_impl_only;
opacity = other.opacity;
transform = other.transform;
filters = other.filters;
animation_start_time = other.animation_start_time; animation_start_time = other.animation_start_time;
if (other.curve) if (other.curve)
curve = other.curve->Clone(); curve = other.curve->Clone();
......
...@@ -12,9 +12,7 @@ ...@@ -12,9 +12,7 @@
#include "cc/animation/animation_export.h" #include "cc/animation/animation_export.h"
#include "cc/animation/keyframe_model.h" #include "cc/animation/keyframe_model.h"
#include "cc/paint/element_id.h" #include "cc/paint/element_id.h"
#include "cc/paint/filter_operations.h"
#include "cc/trees/mutator_host.h" #include "cc/trees/mutator_host.h"
#include "ui/gfx/transform.h"
namespace cc { namespace cc {
...@@ -47,11 +45,6 @@ struct CC_ANIMATION_EXPORT AnimationEvent { ...@@ -47,11 +45,6 @@ struct CC_ANIMATION_EXPORT AnimationEvent {
int target_property; int target_property;
base::TimeTicks monotonic_time; base::TimeTicks monotonic_time;
bool is_impl_only; bool is_impl_only;
// TODO(http://crbug.com/1013715): Remove unused members (opacity, transform,
// filters).
float opacity;
gfx::Transform transform;
FilterOperations filters;
// For continuing a scroll offset animation on the main thread. // For continuing a scroll offset animation on the main thread.
base::TimeTicks animation_start_time; base::TimeTicks animation_start_time;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment