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

Unified Diff: ui/compositor/layer_animator_unittest.cc

Issue 2550933002: Make all LayerAnimationElement::Create*Element return unique_ptr (Closed)
Patch Set: Make all LayerAnimationElement::Create*Element return unique_ptr 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_animator_unittest.cc
diff --git a/ui/compositor/layer_animator_unittest.cc b/ui/compositor/layer_animator_unittest.cc
index 1d7f07482e0af2bdc347b1e14e3b8adbea97ef3b..41ebd846f756b6be35b0f69bf1968665f6cdd5db 100644
--- a/ui/compositor/layer_animator_unittest.cc
+++ b/ui/compositor/layer_animator_unittest.cc
@@ -44,12 +44,12 @@ std::string ColorToString(SkColor color) {
// Creates vector with two LayerAnimationSequences, based on |first| and
// |second| layer animation elements.
std::vector<LayerAnimationSequence*> CreateMultiSequence(
- LayerAnimationElement* first,
- LayerAnimationElement* second) {
+ std::unique_ptr<LayerAnimationElement> first,
+ std::unique_ptr<LayerAnimationElement> second) {
LayerAnimationSequence* first_sequence = new LayerAnimationSequence();
- first_sequence->AddElement(first);
+ first_sequence->AddElement(std::move(first));
LayerAnimationSequence* second_sequence = new LayerAnimationSequence();
- second_sequence->AddElement(second);
+ second_sequence->AddElement(std::move(second));
std::vector<ui::LayerAnimationSequence*> animations;
animations.push_back(first_sequence);

Powered by Google App Engine
This is Rietveld 408576698