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

Unified Diff: third_party/WebKit/Source/core/animation/CSSBorderImageLengthBoxInterpolationType.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/CSSBorderImageLengthBoxInterpolationType.cpp
diff --git a/third_party/WebKit/Source/core/animation/CSSBorderImageLengthBoxInterpolationType.cpp b/third_party/WebKit/Source/core/animation/CSSBorderImageLengthBoxInterpolationType.cpp
index 4feaa9e792893c1f0abd2aa210fb625a47516b9b..c32c70a7ce1fe56d00ddfe7a425ba1cbffe42bae 100644
--- a/third_party/WebKit/Source/core/animation/CSSBorderImageLengthBoxInterpolationType.cpp
+++ b/third_party/WebKit/Source/core/animation/CSSBorderImageLengthBoxInterpolationType.cpp
@@ -147,12 +147,12 @@ InterpolationValue convertBorderImageLengthBox(const BorderImageLengthBox& box,
InterpolationValue convertedSide = CSSLengthInterpolationType::maybeConvertLength(side.length(), zoom);
if (!convertedSide)
return nullptr;
- list->set(i, convertedSide.interpolableValue.release());
+ list->set(i, std::move(convertedSide.interpolableValue));
nonInterpolableValues[i] = convertedSide.nonInterpolableValue.release();
}
}
- return InterpolationValue(list.release(), CSSBorderImageLengthBoxNonInterpolableValue::create(SideNumbers(box), std::move(nonInterpolableValues)));
+ return InterpolationValue(std::move(list), CSSBorderImageLengthBoxNonInterpolableValue::create(SideNumbers(box), std::move(nonInterpolableValues)));
}
} // namespace
@@ -207,12 +207,12 @@ InterpolationValue CSSBorderImageLengthBoxInterpolationType::maybeConvertValue(c
InterpolationValue convertedSide = CSSLengthInterpolationType::maybeConvertCSSValue(side);
if (!convertedSide)
return nullptr;
- list->set(i, convertedSide.interpolableValue.release());
+ list->set(i, std::move(convertedSide.interpolableValue));
nonInterpolableValues[i] = convertedSide.nonInterpolableValue.release();
}
}
- return InterpolationValue(list.release(), CSSBorderImageLengthBoxNonInterpolableValue::create(SideNumbers(quad), std::move(nonInterpolableValues)));
+ return InterpolationValue(std::move(list), CSSBorderImageLengthBoxNonInterpolableValue::create(SideNumbers(quad), std::move(nonInterpolableValues)));
}
InterpolationValue CSSBorderImageLengthBoxInterpolationType::maybeConvertUnderlyingValue(const InterpolationEnvironment& environment) const
@@ -229,7 +229,7 @@ PairwiseInterpolationValue CSSBorderImageLengthBoxInterpolationType::maybeMergeS
if (startSideNumbers != endSideNumbers)
return nullptr;
- return PairwiseInterpolationValue(start.interpolableValue.release(), end.interpolableValue.release(), start.nonInterpolableValue.release());
+ return PairwiseInterpolationValue(std::move(start.interpolableValue), std::move(end.interpolableValue), start.nonInterpolableValue.release());
}
void CSSBorderImageLengthBoxInterpolationType::composite(UnderlyingValueOwner& underlyingValueOwner, double underlyingFraction, const InterpolationValue& value, double interpolationFraction) const

Powered by Google App Engine
This is Rietveld 408576698