Clean up cc/paint interfaces

third_party/WebKit/Source/core/paint/ClipPathClipper.cpp

 // Copyright 2016 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "core/paint/ClipPathClipper.h"
 
 #include "core/dom/ElementTraversal.h"
 #include "core/layout/svg/LayoutSVGResourceClipper.h"
 #include "core/layout/svg/SVGLayoutSupport.h"
 #include "core/layout/svg/SVGResources.h"
@@ skipping 194 matching lines @@
                                                  contentTransform);
       maskContext.getPaintController().createAndAppend<DrawingDisplayItem>(
           m_layoutObject, DisplayItem::kSVGClip,
           m_resourceClipper->createPaintRecord());
     }
   }
 
   LayoutObjectDrawingRecorder drawingRecorder(
       m_context, m_layoutObject, DisplayItem::kSVGClip, targetVisualRect);
   sk_sp<PaintRecord> maskPaintRecord = maskBuilder.endRecording();
-  m_context.drawRecord(maskPaintRecord.get());
+  m_context.drawRecord(maskPaintRecord);

[danakj, 2017/03/16 20:06:15]

move(), or just pass endRecording() directly

   return true;
 }
 
 void ClipPathClipper::finishEffect() {
   switch (m_clipperState) {
     case ClipperState::AppliedPath:
       // Path-only clipping, no layers to restore but we need to emit an end to
       // the clip path display item.
       m_context.getPaintController().endItem<EndClipPathDisplayItem>(
           m_layoutObject);
       break;
     case ClipperState::AppliedMask:
       // Transfer content -> clip mask (SrcIn)
       m_maskContentRecorder.reset();
 
       // Transfer clip mask -> bg (SrcOver)
       m_maskClipRecorder.reset();
       break;
     case ClipperState::NotApplied:
       NOTREACHED();
       break;
   }
 }
 
 }  // namespace blink