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

Unified Diff: third_party/WebKit/Source/platform/scroll/ScrollAnimatorCompositorCoordinator.cpp

Issue 2559273003: Blink Compositor Animation: Introduce CompositorAnimationHost correctly. (Closed)
Patch Set: Add nullcheck for m_layerTreeView in WebViewImpl Created 4 years 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
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 98bca4b9307bfad1e983291c2d86f6a8b9a5472a..92684a1bbcdcba4d7455fdff7975ce99ad993c9b 100644
--- a/third_party/WebKit/Source/platform/scroll/ScrollAnimatorCompositorCoordinator.cpp
+++ b/third_party/WebKit/Source/platform/scroll/ScrollAnimatorCompositorCoordinator.cpp
@@ -256,18 +256,17 @@ void ScrollAnimatorCompositorCoordinator::updateImplOnlyCompositorAnimations() {
return;
GraphicsLayer* layer = getScrollableArea()->layerForScrolling();
- CompositorAnimationTimeline* timeline =
- getScrollableArea()->compositorAnimationTimeline();
- if (layer && timeline && !timeline->compositorAnimationHost().isNull()) {
- CompositorAnimationHost host = timeline->compositorAnimationHost();
- cc::ElementId elementId = layer->platformLayer()->elementId();
+ CompositorAnimationHost* host =
+ getScrollableArea()->compositorAnimationHost();
+ if (layer && host) {
+ CompositorElementId elementId = layer->platformLayer()->elementId();
if (!m_implOnlyAnimationAdjustment.isZero()) {
- host.adjustImplOnlyScrollOffsetAnimation(
+ host->adjustImplOnlyScrollOffsetAnimation(
elementId, gfx::Vector2dF(m_implOnlyAnimationAdjustment.width(),
m_implOnlyAnimationAdjustment.height()));
}
if (m_implOnlyAnimationTakeover)
- host.takeOverImplOnlyScrollOffsetAnimation(elementId);
+ host->takeOverImplOnlyScrollOffsetAnimation(elementId);
}
m_implOnlyAnimationAdjustment = IntSize();
m_implOnlyAnimationTakeover = false;

Powered by Google App Engine
This is Rietveld 408576698