Update new Surface callsites

ui/ozone/platform/drm/gpu/drm_buffer.cc

 // Copyright 2014 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 "ui/ozone/platform/drm/gpu/drm_buffer.h"
 
 #include <drm_fourcc.h>
 
 #include "base/logging.h"
 #include "ui/ozone/platform/drm/gpu/drm_device.h"
@@ skipping 20 matching lines @@
       return 0;
   }
 }
 
 }  // namespace
 
 DrmBuffer::DrmBuffer(const scoped_refptr<DrmDevice>& drm) : drm_(drm) {
 }
 
 DrmBuffer::~DrmBuffer() {
-  surface_.clear();

[f(malita), 2016/04/05 14:31:52]

I think we should leave this alone (security/DRM implications?).

[tomhudson, 2016/04/05 14:52:50]

I thought surface_->clear() would be significant, but surface_.clear() is just
zeroing out the pointer, which the sk_sp destructor will do for us?

Or am I misreading?

[f(malita), 2016/04/05 14:56:02]

You're right - I was thinking this clears the surface, not the pointer. 
Disregard my comment.

-
   if (framebuffer_ && !drm_->RemoveFramebuffer(framebuffer_))
     PLOG(ERROR) << "DrmBuffer: RemoveFramebuffer: fb " << framebuffer_;
 
   if (mmap_base_ && !drm_->UnmapDumbBuffer(mmap_base_, mmap_size_))
     PLOG(ERROR) << "DrmBuffer: UnmapDumbBuffer: handle " << handle_;
 
   if (handle_ && !drm_->DestroyDumbBuffer(handle_))
     PLOG(ERROR) << "DrmBuffer: DestroyDumbBuffer: handle " << handle_;
 }
 
@@ skipping 18 matching lines @@
     strides[0] = stride_;
     uint32_t offsets[4] = {0};
     fb_pixel_format_ = GetFourCCCodeForSkColorType(info.colorType());
     if (!drm_->AddFramebuffer2(info.width(), info.height(), fb_pixel_format_,
                                handles, strides, offsets, &framebuffer_, 0)) {
       PLOG(ERROR) << "DrmBuffer: AddFramebuffer2: handle " << handle_;
       return false;
     }
   }
 
-  surface_ =
-      skia::AdoptRef(SkSurface::NewRasterDirect(info, mmap_base_, stride_));
+  surface_ = SkSurface::MakeRasterDirect(info, mmap_base_, stride_);
   if (!surface_) {
     LOG(ERROR) << "DrmBuffer: Failed to create SkSurface: handle " << handle_;
     return false;
   }
 
   return true;
 }
 
 SkCanvas* DrmBuffer::GetCanvas() const {
   return surface_->getCanvas();
@@ skipping 13 matching lines @@
 
 gfx::Size DrmBuffer::GetSize() const {
   return gfx::Size(surface_->width(), surface_->height());
 }
 
 bool DrmBuffer::RequiresGlFinish() const {
   return false;
 }
 
 }  // namespace ui