Switching some interfaces to use std::unique_ptr<>.

webrtc/pc/videocapturertracksource.h

Index: webrtc/pc/videocapturertracksource.h
diff --git a/webrtc/pc/videocapturertracksource.h b/webrtc/pc/videocapturertracksource.h
index 30991a11ac70c14b1e632aeb3010a4fd323d7c46..af87cc9b824122171cd5bac6a7ba43bb78fac128 100644
--- a/webrtc/pc/videocapturertracksource.h
+++ b/webrtc/pc/videocapturertracksource.h
@@ -34,18 +34,18 @@ class VideoCapturerTrackSource : public VideoTrackSource,
                                  public sigslot::has_slots<> {
  public:
   // Creates an instance of VideoCapturerTrackSource.
-  // VideoCapturerTrackSource take ownership of |capturer|.
+  // VideoCapturerTrackSource takes ownership of |capturer|.

[nisse-webrtc, 2017/02/13 08:25:58]

Another redundant ownership comment.

[Taylor Brandstetter, 2017/02/13 17:47:45]

Fixed.

   // |constraints| can be NULL and in that case the camera is opened using a
   // default resolution.
   static rtc::scoped_refptr<VideoTrackSourceInterface> Create(
       rtc::Thread* worker_thread,
-      cricket::VideoCapturer* capturer,
+      std::unique_ptr<cricket::VideoCapturer> capturer,
       const webrtc::MediaConstraintsInterface* constraints,
       bool remote);
 
   static rtc::scoped_refptr<VideoTrackSourceInterface> Create(
       rtc::Thread* worker_thread,
-      cricket::VideoCapturer* capturer,
+      std::unique_ptr<cricket::VideoCapturer> capturer,
       bool remote);
 
   bool is_screencast() const override {
@@ -59,7 +59,7 @@ class VideoCapturerTrackSource : public VideoTrackSource,
 
  protected:
   VideoCapturerTrackSource(rtc::Thread* worker_thread,
-                           cricket::VideoCapturer* capturer,
+                           std::unique_ptr<cricket::VideoCapturer> capturer,
                            bool remote);
   virtual ~VideoCapturerTrackSource();
   void Initialize(const webrtc::MediaConstraintsInterface* constraints);