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

Unified Diff: ui/compositor/layer_animation_element.h

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_animation_element.h
diff --git a/ui/compositor/layer_animation_element.h b/ui/compositor/layer_animation_element.h
index aaeb51c2344f40bfc9c465155cdd34a8448aba2f..53fbc9a458785e84dbe8340c7ff1b8a77998f35e 100644
--- a/ui/compositor/layer_animation_element.h
+++ b/ui/compositor/layer_animation_element.h
@@ -70,7 +70,7 @@ class COMPOSITOR_EXPORT LayerAnimationElement {
// Creates an element that transitions to the given transform. The caller owns
// the return value.
- static LayerAnimationElement* CreateTransformElement(
+ static std::unique_ptr<LayerAnimationElement> CreateTransformElement(
const gfx::Transform& transform,
base::TimeDelta duration);
@@ -81,49 +81,50 @@ class COMPOSITOR_EXPORT LayerAnimationElement {
// transform and the last value the interpolated transform will assume. It is
// therefore important that the value of the interpolated at time 0 matches
// the current transform.
- static LayerAnimationElement* CreateInterpolatedTransformElement(
+ static std::unique_ptr<LayerAnimationElement>
+ CreateInterpolatedTransformElement(
InterpolatedTransform* interpolated_transform,
base::TimeDelta duration);
// Creates an element that transitions to the given bounds. The caller owns
// the return value.
- static LayerAnimationElement* CreateBoundsElement(
+ static std::unique_ptr<LayerAnimationElement> CreateBoundsElement(
const gfx::Rect& bounds,
base::TimeDelta duration);
// Creates an element that transitions to the given opacity. The caller owns
// the return value.
- static LayerAnimationElement* CreateOpacityElement(
+ static std::unique_ptr<LayerAnimationElement> CreateOpacityElement(
float opacity,
base::TimeDelta duration);
// Creates an element that sets visibily following a delay. The caller owns
// the return value.
- static LayerAnimationElement* CreateVisibilityElement(
+ static std::unique_ptr<LayerAnimationElement> CreateVisibilityElement(
bool visibility,
base::TimeDelta duration);
// Creates an element that transitions to the given brightness.
// The caller owns the return value.
- static LayerAnimationElement* CreateBrightnessElement(
+ static std::unique_ptr<LayerAnimationElement> CreateBrightnessElement(
float brightness,
base::TimeDelta duration);
// Creates an element that transitions to the given grayscale value.
// The caller owns the return value.
- static LayerAnimationElement* CreateGrayscaleElement(
+ static std::unique_ptr<LayerAnimationElement> CreateGrayscaleElement(
float grayscale,
base::TimeDelta duration);
// Creates an element that pauses the given properties. The caller owns the
// return value.
- static LayerAnimationElement* CreatePauseElement(
+ static std::unique_ptr<LayerAnimationElement> CreatePauseElement(
AnimatableProperties properties,
base::TimeDelta duration);
// Creates an element that transitions to the given color. The caller owns the
// return value.
- static LayerAnimationElement* CreateColorElement(
+ static std::unique_ptr<LayerAnimationElement> CreateColorElement(
SkColor color,
base::TimeDelta duration);

Powered by Google App Engine
This is Rietveld 408576698