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

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

Issue 2622043003: Replaced RefPtr::release with std::move in Source/core. (Closed)
Patch Set: Created 3 years, 11 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/CSSFilterListInterpolationType.cpp
diff --git a/third_party/WebKit/Source/core/animation/CSSFilterListInterpolationType.cpp b/third_party/WebKit/Source/core/animation/CSSFilterListInterpolationType.cpp
index aeba30d5c10cd2cf14ef94d8d40772f628222627..6ae7c6f08f86f4c7604738fcf8fc0ce9c58228fb 100644
--- a/third_party/WebKit/Source/core/animation/CSSFilterListInterpolationType.cpp
+++ b/third_party/WebKit/Source/core/animation/CSSFilterListInterpolationType.cpp
@@ -92,7 +92,7 @@ InterpolationValue convertFilterList(const FilterOperations& filterOperations,
if (!filterResult)
return nullptr;
interpolableList->set(i, std::move(filterResult.interpolableValue));
- nonInterpolableValues[i] = filterResult.nonInterpolableValue.release();
+ nonInterpolableValues[i] = std::move(filterResult.nonInterpolableValue);
}
return InterpolationValue(
std::move(interpolableList),
@@ -155,7 +155,7 @@ InterpolationValue CSSFilterListInterpolationType::maybeConvertValue(
if (!itemResult)
return nullptr;
interpolableList->set(i, std::move(itemResult.interpolableValue));
- nonInterpolableValues[i] = itemResult.nonInterpolableValue.release();
+ nonInterpolableValues[i] = std::move(itemResult.nonInterpolableValue);
}
return InterpolationValue(
std::move(interpolableList),
@@ -187,10 +187,11 @@ PairwiseInterpolationValue CSSFilterListInterpolationType::maybeMergeSingles(
return nullptr;
}
- if (startLength == endLength)
+ if (startLength == endLength) {
return PairwiseInterpolationValue(std::move(start.interpolableValue),
std::move(end.interpolableValue),
- start.nonInterpolableValue.release());
+ std::move(start.nonInterpolableValue));
+ }
// Extend the shorter InterpolableList with neutral values that are compatible
// with corresponding filters in the longer list.
@@ -219,7 +220,7 @@ PairwiseInterpolationValue CSSFilterListInterpolationType::maybeMergeSingles(
return PairwiseInterpolationValue(std::move(start.interpolableValue),
std::move(end.interpolableValue),
- longer.nonInterpolableValue.release());
+ std::move(longer.nonInterpolableValue));
}
void CSSFilterListInterpolationType::composite(

Powered by Google App Engine
This is Rietveld 408576698