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

Unified Diff: third_party/WebKit/Source/platform/fonts/shaping/ShapingLineBreaker.cpp

Issue 2957513002: Removed calls to RefPtr::Release in return statements with auto move. (Closed)
Patch Set: rebased Created 3 years, 6 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/platform/fonts/shaping/ShapingLineBreaker.cpp
diff --git a/third_party/WebKit/Source/platform/fonts/shaping/ShapingLineBreaker.cpp b/third_party/WebKit/Source/platform/fonts/shaping/ShapingLineBreaker.cpp
index 351cb23fbd866937a399a590a55ab341bd4dce23..c4ade885c296668ccf1d7e9a3240ea02af10ff04 100644
--- a/third_party/WebKit/Source/platform/fonts/shaping/ShapingLineBreaker.cpp
+++ b/third_party/WebKit/Source/platform/fonts/shaping/ShapingLineBreaker.cpp
@@ -86,7 +86,7 @@ inline PassRefPtr<ShapeResult> ShapingLineBreaker::Shape(
RefPtr<ShapeResult> result = shaper_->Shape(font_, direction, start, end);
result->ApplySpacing(*spacing_, direction);
- return result.Release();
+ return result;
}
// Shapes a line of text by finding a valid and appropriate break opportunity
@@ -244,7 +244,7 @@ PassRefPtr<ShapeResult> ShapingLineBreaker::ShapeLine(
line_result->NumCharacters());
*break_offset = break_opportunity;
- return line_result.Release();
+ return line_result;
}
// Shape from the specified offset to the end of the ShapeResult.
@@ -271,7 +271,7 @@ PassRefPtr<ShapeResult> ShapingLineBreaker::ShapeToEnd(
// If no safe-to-break in the ragne, reshape the whole range.
line_result = Shape(direction, start, range_end);
}
- return line_result.Release();
+ return line_result;
}
} // namespace blink

Powered by Google App Engine
This is Rietveld 408576698