Replace many skia::RefPtr with sk_sp<> in cc/

cc/playback/display_item_list.cc

Index: cc/playback/display_item_list.cc
diff --git a/cc/playback/display_item_list.cc b/cc/playback/display_item_list.cc
index 235ee3f86c9640a98d4debfd1fc7d7f564fbb44a..d4af490690c8c641adf7eb8e2cc43b0ab90caca8 100644
--- a/cc/playback/display_item_list.cc
+++ b/cc/playback/display_item_list.cc
@@ -89,7 +89,7 @@ DisplayItemList::DisplayItemList(gfx::Rect layer_rect,
   if (settings_.use_cached_picture) {
     SkRTreeFactory factory;
     recorder_.reset(new SkPictureRecorder());
-    canvas_ = skia::SharePtr(recorder_->beginRecording(
+    canvas_ = sk_ref_sp(recorder_->beginRecording(

[f(malita), 2016/04/21 15:45:47]

(note to self, maybe add a TODO(fmalita)?)

Tracking the recorder canvas separately seems fragile - it is intrinsically tied
to the recorder state.  We should probably dump canvas_ and just use
recorder_->getCanvas() instead.

[tomhudson, 2016/04/25 20:20:51]

Acknowledged.

         layer_rect_.width(), layer_rect_.height(), &factory));
     canvas_->translate(-layer_rect_.x(), -layer_rect_.y());
     canvas_->clipRect(gfx::RectToSkRect(layer_rect_));
@@ -189,7 +189,7 @@ void DisplayItemList::Finalize() {
     picture_memory_usage_ =
         SkPictureUtils::ApproximateBytesUsed(picture_.get());
     recorder_.reset();
-    canvas_.clear();
+    canvas_.reset();
     is_suitable_for_gpu_rasterization_ =
         picture_->suitableForGpuRasterization(nullptr);
   }