Index: third_party/WebKit/Source/platform/scroll/ScrollAnimatorCompositorCoordinator.cpp |
diff --git a/third_party/WebKit/Source/platform/scroll/ScrollAnimatorCompositorCoordinator.cpp b/third_party/WebKit/Source/platform/scroll/ScrollAnimatorCompositorCoordinator.cpp |
index 70fa7f9c8fcd895a82bf3e86de76840c8ddab77a..534b91040472936147c7895dae44e761d3e1e3c0 100644 |
--- a/third_party/WebKit/Source/platform/scroll/ScrollAnimatorCompositorCoordinator.cpp |
+++ b/third_party/WebKit/Source/platform/scroll/ScrollAnimatorCompositorCoordinator.cpp |
@@ -22,6 +22,7 @@ ScrollAnimatorCompositorCoordinator::ScrollAnimatorCompositorCoordinator() |
, m_runState(RunState::Idle) |
, m_compositorAnimationId(0) |
, m_compositorAnimationGroupId(0) |
+ , m_implOnlyAnimationTakeover(false) |
{ |
ThreadState::current()->registerPreFinalizer(this); |
m_compositorPlayer = adoptPtr(CompositorFactory::current().createAnimationPlayer()); |
@@ -48,7 +49,7 @@ void ScrollAnimatorCompositorCoordinator::resetAnimationState() |
bool ScrollAnimatorCompositorCoordinator::hasAnimationThatRequiresService() const |
{ |
- if (!m_implOnlyAnimationAdjustment.isZero()) |
+ if (hasImplOnlyAnimationUpdate()) |
return true; |
switch (m_runState) { |
@@ -119,10 +120,12 @@ void ScrollAnimatorCompositorCoordinator::cancelAnimation() |
} |
} |
-void ScrollAnimatorCompositorCoordinator::takeoverCompositorAnimation() |
+void ScrollAnimatorCompositorCoordinator::takeOverCompositorAnimation() |
{ |
switch (m_runState) { |
case RunState::Idle: |
+ takeOverImplOnlyScrollOffsetAnimation(); |
+ break; |
case RunState::WaitingToCancelOnCompositor: |
case RunState::WaitingToCancelOnCompositorButNewScroll: |
case RunState::PostAnimationCleanup: |
@@ -242,23 +245,34 @@ FloatPoint ScrollAnimatorCompositorCoordinator::blinkOffsetFromCompositorOffset( |
return offset; |
} |
+bool ScrollAnimatorCompositorCoordinator::hasImplOnlyAnimationUpdate() const |
+{ |
+ return !m_implOnlyAnimationAdjustment.isZero() || m_implOnlyAnimationTakeover; |
+} |
+ |
void ScrollAnimatorCompositorCoordinator::updateCompositorAnimations() |
{ |
- if (!getScrollableArea()->scrollAnimatorEnabled() || m_implOnlyAnimationAdjustment.isZero()) |
+ if (!getScrollableArea()->scrollAnimatorEnabled() || !hasImplOnlyAnimationUpdate()) |
return; |
GraphicsLayer* layer = getScrollableArea()->layerForScrolling(); |
CompositorAnimationTimeline* timeline = getScrollableArea()->compositorAnimationTimeline(); |
if (layer && timeline && !timeline->compositorAnimationHost().isNull()) { |
CompositorAnimationHost host = timeline->compositorAnimationHost(); |
- host.updateImplOnlyScrollOffsetAnimation( |
- gfx::Vector2dF(m_implOnlyAnimationAdjustment.width(), m_implOnlyAnimationAdjustment.height()), |
- layer->platformLayer()->id()); |
+ int elementId = layer->platformLayer()->id(); |
+ if (!m_implOnlyAnimationAdjustment.isZero()) { |
+ host.adjustImplOnlyScrollOffsetAnimation( |
+ elementId, |
+ gfx::Vector2dF(m_implOnlyAnimationAdjustment.width(), m_implOnlyAnimationAdjustment.height())); |
+ } |
+ if (m_implOnlyAnimationTakeover) |
+ host.takeOverImplOnlyScrollOffsetAnimation(elementId); |
} |
m_implOnlyAnimationAdjustment = FloatSize(); |
+ m_implOnlyAnimationTakeover = false; |
} |
-void ScrollAnimatorCompositorCoordinator::updateImplOnlyScrollOffsetAnimation( |
+void ScrollAnimatorCompositorCoordinator::adjustImplOnlyScrollOffsetAnimation( |
const FloatSize& adjustment) |
{ |
if (!getScrollableArea()->scrollAnimatorEnabled()) |
@@ -269,6 +283,16 @@ void ScrollAnimatorCompositorCoordinator::updateImplOnlyScrollOffsetAnimation( |
getScrollableArea()->registerForAnimation(); |
} |
+void ScrollAnimatorCompositorCoordinator::takeOverImplOnlyScrollOffsetAnimation() |
+{ |
+ if (!getScrollableArea()->scrollAnimatorEnabled()) |
+ return; |
+ |
+ m_implOnlyAnimationTakeover = true; |
+ |
+ getScrollableArea()->registerForAnimation(); |
+} |
+ |
String ScrollAnimatorCompositorCoordinator::runStateAsText() const |
{ |
switch (m_runState) { |