Use std::unique_ptr<> to pass frame ownership in DesktopCapturer impls.

webrtc/modules/desktop_capture/shared_desktop_frame.cc

Index: webrtc/modules/desktop_capture/shared_desktop_frame.cc
diff --git a/webrtc/modules/desktop_capture/shared_desktop_frame.cc b/webrtc/modules/desktop_capture/shared_desktop_frame.cc
index 8d10827e29cc2b78d472e8c28beaeb90064f1b81..24da9d4a4f003950366a073f6d268415b9f83142 100644
--- a/webrtc/modules/desktop_capture/shared_desktop_frame.cc
+++ b/webrtc/modules/desktop_capture/shared_desktop_frame.cc
@@ -19,7 +19,7 @@ namespace webrtc {
 
 class SharedDesktopFrame::Core {

[Wez, 2016/05/18 01:29:50]

nit: While you're here... couldn't this effectively be replaced with:

rtc::RefCountedObject<std::unique_ptr<DesktopFrame>>

Or would that lead to too much get() ugliness?

[Sergey Ulanov, 2016/05/31 12:02:49]

Done.

  public:
-  Core(DesktopFrame* frame) : frame_(frame) {}
+  Core(std::unique_ptr<DesktopFrame> frame) : frame_(std::move(frame)) {}
 
   DesktopFrame* frame() { return frame_.get(); }
 
@@ -49,17 +49,22 @@ class SharedDesktopFrame::Core {
 SharedDesktopFrame::~SharedDesktopFrame() {}
 
 // static
+std::unique_ptr<SharedDesktopFrame> SharedDesktopFrame::Wrap(
+    std::unique_ptr<DesktopFrame> desktop_frame) {
+  return std::unique_ptr<SharedDesktopFrame>(
+      new SharedDesktopFrame(new Core(std::move(desktop_frame))));
+}
+
 SharedDesktopFrame* SharedDesktopFrame::Wrap(DesktopFrame* desktop_frame) {
-  rtc::scoped_refptr<Core> core(new Core(desktop_frame));
-  return new SharedDesktopFrame(core);
+  return Wrap(std::unique_ptr<DesktopFrame>(desktop_frame)).release();
 }
 
 DesktopFrame* SharedDesktopFrame::GetUnderlyingFrame() {
   return core_->frame();
 }
 
-SharedDesktopFrame* SharedDesktopFrame::Share() {
-  SharedDesktopFrame* result = new SharedDesktopFrame(core_);
+std::unique_ptr<SharedDesktopFrame> SharedDesktopFrame::Share() {
+  std::unique_ptr<SharedDesktopFrame> result(new SharedDesktopFrame(core_));
   result->set_dpi(dpi());
   result->set_capture_time_ms(capture_time_ms());
   *result->mutable_updated_region() = updated_region();