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

Unified Diff: third_party/WebKit/Source/platform/animation/CompositorAnimationTimelineTest.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/animation/CompositorAnimationTimelineTest.cpp
diff --git a/third_party/WebKit/Source/platform/animation/CompositorAnimationTimelineTest.cpp b/third_party/WebKit/Source/platform/animation/CompositorAnimationTimelineTest.cpp
index 72422d3a2cdd025c2d0019b7dc527abb5cd12e0e..9ec00e630cc4d6da18efdd4a5496e9fc995fca00 100644
--- a/third_party/WebKit/Source/platform/animation/CompositorAnimationTimelineTest.cpp
+++ b/third_party/WebKit/Source/platform/animation/CompositorAnimationTimelineTest.cpp
@@ -6,6 +6,7 @@
#include "base/memory/ref_counted.h"
#include "cc/animation/animation_host.h"
+#include "platform/animation/CompositorAnimationHost.h"
#include "platform/animation/CompositorAnimationPlayer.h"
#include "platform/testing/CompositorTest.h"
#include "platform/testing/WebLayerTreeViewImplForTesting.h"
@@ -29,8 +30,10 @@ TEST_F(CompositorAnimationTimelineTest,
WTF::wrapUnique(new WebLayerTreeViewImplForTesting);
DCHECK(layerTreeHost);
- layerTreeHost->attachCompositorAnimationTimeline(
- timeline->animationTimeline());
+ auto compositorAnimationHost = WTF::makeUnique<CompositorAnimationHost>(
+ layerTreeHost->compositorAnimationHost());
+
+ compositorAnimationHost->addTimeline(timeline.get());
cc::AnimationHost* animationHost = ccTimeline->animation_host();
EXPECT_TRUE(animationHost);
EXPECT_TRUE(animationHost->GetTimelineById(ccTimeline->id()));

Powered by Google App Engine
This is Rietveld 408576698