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

Unified Diff: cc/animation/animation_host.cc

Issue 1950243005: Communicate MT changes to impl-only scroll offset animations (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: attempt to fix rebase issue Created 4 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
« no previous file with comments | « cc/animation/animation_host.h ('k') | cc/animation/scroll_offset_animation_curve.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: cc/animation/animation_host.cc
diff --git a/cc/animation/animation_host.cc b/cc/animation/animation_host.cc
index 66ebaa482f038abfe391f0a740c934022f1587b3..32aa7ca6eb2d1d7dbf903dc713f15aeeddccf333 100644
--- a/cc/animation/animation_host.cc
+++ b/cc/animation/animation_host.cc
@@ -17,6 +17,8 @@
#include "cc/animation/animation_timeline.h"
#include "cc/animation/element_animations.h"
#include "cc/animation/scroll_offset_animation_curve.h"
+#include "cc/animation/scroll_offset_animations.h"
+#include "cc/animation/scroll_offset_animations_impl.h"
#include "cc/animation/timing_function.h"
#include "ui/gfx/geometry/box_f.h"
#include "ui/gfx/geometry/scroll_offset.h"
@@ -33,9 +35,13 @@ AnimationHost::AnimationHost(ThreadInstance thread_instance)
thread_instance_(thread_instance),
supports_scroll_animations_(false),
animation_waiting_for_deletion_(false) {
- if (thread_instance_ == ThreadInstance::IMPL)
+ if (thread_instance_ == ThreadInstance::IMPL) {
scroll_offset_animations_impl_ =
base::WrapUnique(new ScrollOffsetAnimationsImpl(this));
+ } else {
+ scroll_offset_animations_ =
+ base::WrapUnique(new ScrollOffsetAnimations(this));
+ }
}
AnimationHost::~AnimationHost() {
@@ -185,7 +191,8 @@ void AnimationHost::RemoveTimelinesFromImplThread(
}
void AnimationHost::PushPropertiesToImplThread(AnimationHost* host_impl) {
- // Firstly, sync all players with impl thread to create ElementAnimations.
+ // Sync all players with impl thread to create ElementAnimations. This needs
+ // to happen before the element animations are synced below.
for (auto& kv : id_to_timeline_map_) {
AnimationTimeline* timeline = kv.second.get();
AnimationTimeline* timeline_impl =
@@ -194,7 +201,7 @@ void AnimationHost::PushPropertiesToImplThread(AnimationHost* host_impl) {
timeline->PushPropertiesTo(timeline_impl);
}
- // Secondly, sync properties for created ElementAnimations.
+ // Sync properties for created ElementAnimations.
for (auto& kv : element_to_animations_map_) {
const auto& element_animations = kv.second;
auto element_animations_impl =
@@ -202,6 +209,10 @@ void AnimationHost::PushPropertiesToImplThread(AnimationHost* host_impl) {
if (element_animations_impl)
element_animations->PushPropertiesTo(std::move(element_animations_impl));
}
+
+ // Update the impl-only scroll offset animations.
+ scroll_offset_animations_->PushPropertiesTo(
+ host_impl->scroll_offset_animations_impl_.get());
}
scoped_refptr<ElementAnimations>
@@ -533,6 +544,11 @@ bool AnimationHost::ImplOnlyScrollAnimationUpdateTarget(
element_id, scroll_delta, max_scroll_offset, frame_monotonic_time);
}
+ScrollOffsetAnimations& AnimationHost::scroll_offset_animations() const {
+ DCHECK(scroll_offset_animations_);
+ return *scroll_offset_animations_.get();
+}
+
void AnimationHost::ScrollAnimationAbort(bool needs_completion) {
DCHECK(scroll_offset_animations_impl_);
return scroll_offset_animations_impl_->ScrollAnimationAbort(needs_completion);
« no previous file with comments | « cc/animation/animation_host.h ('k') | cc/animation/scroll_offset_animation_curve.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698