Make all LayerAnimationElement::Create*Element return unique_ptr

ash/rotator/screen_rotation_animation_unittest.cc

 // Copyright 2015 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "ash/rotator/screen_rotation_animation.h"
 
 #include <memory>
 
 #include "ash/test/ash_test_base.h"
 #include "base/macros.h"
@@ skipping 28 matching lines @@
 void ScreenRotationAnimationTest::SetUp() {
   AshTestBase::SetUp();
   non_zero_duration_mode_.reset(new ui::ScopedAnimationDurationScaleMode(
       ui::ScopedAnimationDurationScaleMode::NON_ZERO_DURATION));
 }
 
 TEST_F(ScreenRotationAnimationTest, LayerTransformGetsSetToTargetWhenAborted) {
   std::unique_ptr<aura::Window> window(CreateTestWindowInShellWithId(9));
   ui::Layer* layer = window->layer();
 
-  std::unique_ptr<ScreenRotationAnimation> screen_rotation(
-      new ScreenRotationAnimation(
+  std::unique_ptr<ScreenRotationAnimation> screen_rotation =
+      base::MakeUnique<ScreenRotationAnimation>(
           layer, 45 /* start_degrees */, 0 /* end_degrees */,
           0.5f /* initial_opacity */, 1.0f /* target_opacity */,
           gfx::Point(10, 10) /* pivot */,
-          base::TimeDelta::FromSeconds(10) /* duration */, gfx::Tween::LINEAR));
+          base::TimeDelta::FromSeconds(10) /* duration */, gfx::Tween::LINEAR);
 
   ui::LayerAnimator* animator = layer->GetAnimator();
   animator->set_preemption_strategy(
       ui::LayerAnimator::REPLACE_QUEUED_ANIMATIONS);
-  std::unique_ptr<ui::LayerAnimationSequence> animation_sequence(
-      new ui::LayerAnimationSequence(screen_rotation.release()));
+  std::unique_ptr<ui::LayerAnimationSequence> animation_sequence =
+      base::MakeUnique<ui::LayerAnimationSequence>(std::move(screen_rotation));
   animator->StartAnimation(animation_sequence.release());
 
   const gfx::Transform identity_transform;
 
   ASSERT_EQ(identity_transform, layer->GetTargetTransform());
   ASSERT_NE(identity_transform, layer->transform());
 
   layer->GetAnimator()->AbortAllAnimations();
 
   EXPECT_EQ(identity_transform, layer->transform());
 }
 
 // Tests that ScreenRotationAnimation::OnAbort() doesn't segfault when passed a
 // null delegate in response to its ui::Layer being destroyed:
 // http://crbug.com/661313
 TEST_F(ScreenRotationAnimationTest, DestroyLayerDuringAnimation) {
   // Create a ui::Layer directly rather than an aura::Window, as the latter
   // finishes all of its animation before destroying its layer.
   std::unique_ptr<ui::Layer> layer = base::MakeUnique<ui::Layer>();
 
   ui::Layer* root_layer = CurrentContext()->layer();
   layer->SetBounds(gfx::Rect(root_layer->bounds().size()));
   root_layer->Add(layer.get());
 
-  std::unique_ptr<ScreenRotationAnimation> screen_rotation(
-      new ScreenRotationAnimation(layer.get(), 45, 0, 1.0f, 1.0f, gfx::Point(),
-                                  base::TimeDelta::FromSeconds(1),
-                                  gfx::Tween::LINEAR));
+  std::unique_ptr<ScreenRotationAnimation> screen_rotation =
+      base::MakeUnique<ScreenRotationAnimation>(
+          layer.get(), 45, 0, 1.0f, 1.0f, gfx::Point(),
+          base::TimeDelta::FromSeconds(1), gfx::Tween::LINEAR);
   ui::LayerAnimator* animator = layer->GetAnimator();
-  std::unique_ptr<ui::LayerAnimationSequence> animation_sequence(
-      new ui::LayerAnimationSequence(screen_rotation.release()));
+  std::unique_ptr<ui::LayerAnimationSequence> animation_sequence =
+      base::MakeUnique<ui::LayerAnimationSequence>(std::move(screen_rotation));
   animator->StartAnimation(animation_sequence.release());
 
   // Explicitly destroy the layer to verify that the animation doesn't crash.
   layer.reset();
 }
 
 }  // namespace test
 }  // namespace ash