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

Unified Diff: ui/compositor/layer_unittest.cc

Issue 2550933002: Make all LayerAnimationElement::Create*Element return unique_ptr (Closed)
Patch Set: Complete inclusion 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: ui/compositor/layer_unittest.cc
diff --git a/ui/compositor/layer_unittest.cc b/ui/compositor/layer_unittest.cc
index 9a55e398ff6f907067826ff789e893e7eced2631..c38f5f9fb55612484c998fa1aacacbe7d77e89d1 100644
--- a/ui/compositor/layer_unittest.cc
+++ b/ui/compositor/layer_unittest.cc
@@ -2162,10 +2162,11 @@ TEST_F(LayerWithDelegateTest, NonAnimatingAnimatorsAreRemovedFromCollection) {
LayerRemovingLayerAnimationObserver observer(root.get(), parent.get());
child->GetAnimator()->AddObserver(&observer);
- LayerAnimationElement* element =
+ std::unique_ptr<LayerAnimationElement> element =
ui::LayerAnimationElement::CreateOpacityElement(
0.5f, base::TimeDelta::FromSeconds(1));
- LayerAnimationSequence* sequence = new LayerAnimationSequence(element);
+ LayerAnimationSequence* sequence =
+ new LayerAnimationSequence(std::move(element));
child->GetAnimator()->StartAnimation(sequence);
EXPECT_TRUE(compositor()->layer_animator_collection()->HasActiveAnimators());

Powered by Google App Engine
This is Rietveld 408576698