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

Unified Diff: third_party/WebKit/Source/core/paint/PaintPropertyTreeBuilder.cpp

Issue 2359553003: Replaced PassRefPtr copies with moves. (Closed)
Patch Set: Created 4 years, 3 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/paint/PaintPropertyTreeBuilder.cpp
diff --git a/third_party/WebKit/Source/core/paint/PaintPropertyTreeBuilder.cpp b/third_party/WebKit/Source/core/paint/PaintPropertyTreeBuilder.cpp
index 98d62c8c2fa164070698f109a86f51b53d277f0e..0ce6ff6e4306a250840b5e70777ef9786ecd3089 100644
--- a/third_party/WebKit/Source/core/paint/PaintPropertyTreeBuilder.cpp
+++ b/third_party/WebKit/Source/core/paint/PaintPropertyTreeBuilder.cpp
@@ -51,9 +51,9 @@ void createOrUpdateFrameViewPreTranslation(FrameView& frameView,
{
DCHECK(!RuntimeEnabledFeatures::rootLayerScrollingEnabled());
if (TransformPaintPropertyNode* existingPreTranslation = frameView.preTranslation())
- existingPreTranslation->update(parent, matrix, origin);
+ existingPreTranslation->update(std::move(parent), matrix, origin);
else
- frameView.setPreTranslation(TransformPaintPropertyNode::create(parent, matrix, origin));
+ frameView.setPreTranslation(TransformPaintPropertyNode::create(std::move(parent), matrix, origin));
}
void createOrUpdateFrameViewContentClip(FrameView& frameView,
@@ -63,9 +63,9 @@ void createOrUpdateFrameViewContentClip(FrameView& frameView,
{
DCHECK(!RuntimeEnabledFeatures::rootLayerScrollingEnabled());
if (ClipPaintPropertyNode* existingContentClip = frameView.contentClip())
- existingContentClip->update(parent, localTransformSpace, clipRect);
+ existingContentClip->update(std::move(parent), std::move(localTransformSpace), clipRect);
else
- frameView.setContentClip(ClipPaintPropertyNode::create(parent, localTransformSpace, clipRect));
+ frameView.setContentClip(ClipPaintPropertyNode::create(std::move(parent), std::move(localTransformSpace), clipRect));
}
void createOrUpdateFrameViewScrollTranslation(FrameView& frameView,
@@ -75,9 +75,9 @@ void createOrUpdateFrameViewScrollTranslation(FrameView& frameView,
{
DCHECK(!RuntimeEnabledFeatures::rootLayerScrollingEnabled());
if (TransformPaintPropertyNode* existingScrollTranslation = frameView.scrollTranslation())
- existingScrollTranslation->update(parent, matrix, origin);
+ existingScrollTranslation->update(std::move(parent), matrix, origin);
else
- frameView.setScrollTranslation(TransformPaintPropertyNode::create(parent, matrix, origin));
+ frameView.setScrollTranslation(TransformPaintPropertyNode::create(std::move(parent), matrix, origin));
}
void createOrUpdateFrameViewScroll(FrameView& frameView,
@@ -89,9 +89,9 @@ void createOrUpdateFrameViewScroll(FrameView& frameView,
{
DCHECK(!RuntimeEnabledFeatures::rootLayerScrollingEnabled());
if (ScrollPaintPropertyNode* existingScroll = frameView.scroll())
- existingScroll->update(parent, scrollOffset, clip, bounds, userScrollableHorizontal, userScrollableVertical);
+ existingScroll->update(std::move(parent), std::move(scrollOffset), clip, bounds, userScrollableHorizontal, userScrollableVertical);
else
- frameView.setScroll(ScrollPaintPropertyNode::create(parent, scrollOffset, clip, bounds, userScrollableHorizontal, userScrollableVertical));
+ frameView.setScroll(ScrollPaintPropertyNode::create(std::move(parent), std::move(scrollOffset), clip, bounds, userScrollableHorizontal, userScrollableVertical));
}
void PaintPropertyTreeBuilder::buildTreeNodes(FrameView& frameView, PaintPropertyTreeBuilderContext& context)
« no previous file with comments | « third_party/WebKit/Source/core/dom/ElementRareData.h ('k') | third_party/WebKit/Source/core/style/StyleVariableData.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698