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

Unified Diff: third_party/WebKit/Source/core/animation/CompositorAnimations.cpp

Issue 1977763002: Remove OwnPtr::release() calls in core/ (part 1). (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Remove static_pointer_cast<>s. Created 4 years, 7 months 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/core/animation/CompositorAnimations.cpp
diff --git a/third_party/WebKit/Source/core/animation/CompositorAnimations.cpp b/third_party/WebKit/Source/core/animation/CompositorAnimations.cpp
index fd3f332118dd73fa4655720c2944f46a511938ed..3648c200619781549c61f8a4ce49521562bfc50a 100644
--- a/third_party/WebKit/Source/core/animation/CompositorAnimations.cpp
+++ b/third_party/WebKit/Source/core/animation/CompositorAnimations.cpp
@@ -617,7 +617,7 @@ void CompositorAnimationsImpl::addKeyframesToCurve(CompositorAnimationCurve& cur
OwnPtr<CompositorFilterOperations> ops = adoptPtr(CompositorFactory::current().createFilterOperations());
toCompositorFilterOperations(toAnimatableFilterOperations(value)->operations(), ops.get());
- CompositorFilterKeyframe filterKeyframe(keyframe->offset(), ops.release());
+ CompositorFilterKeyframe filterKeyframe(keyframe->offset(), std::move(ops));
CompositorFilterAnimationCurve* filterCurve = static_cast<CompositorFilterAnimationCurve*>(&curve);
addKeyframeWithTimingFunction(*filterCurve, filterKeyframe, keyframeTimingFunction);
break;
@@ -632,7 +632,7 @@ void CompositorAnimationsImpl::addKeyframesToCurve(CompositorAnimationCurve& cur
OwnPtr<CompositorTransformOperations> ops = adoptPtr(CompositorFactory::current().createTransformOperations());
toCompositorTransformOperations(toAnimatableTransform(value)->transformOperations(), ops.get());
- CompositorTransformKeyframe transformKeyframe(keyframe->offset(), ops.release());
+ CompositorTransformKeyframe transformKeyframe(keyframe->offset(), std::move(ops));
CompositorTransformAnimationCurve* transformCurve = static_cast<CompositorTransformAnimationCurve*>(&curve);
addKeyframeWithTimingFunction(*transformCurve, transformKeyframe, keyframeTimingFunction);
break;
@@ -744,7 +744,7 @@ void CompositorAnimationsImpl::getAnimationOnCompositor(const Timing& timing, in
default:
ASSERT_NOT_REACHED();
}
- animations.append(animation.release());
+ animations.append(std::move(animation));
}
ASSERT(!animations.isEmpty());
}

Powered by Google App Engine
This is Rietveld 408576698