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

webrtc/pc/videocapturertracksource.h

 /*
  *  Copyright 2012 The WebRTC project authors. All Rights Reserved.
  *
  *  Use of this source code is governed by a BSD-style license
  *  that can be found in the LICENSE file in the root of the source
  *  tree. An additional intellectual property rights grant can be found
  *  in the file PATENTS.  All contributing project authors may
  *  be found in the AUTHORS file in the root of the source tree.
  */
 
@@ skipping 16 matching lines @@
 // If the constraint can't be met or the capturer fails to start, the state
 // transition to kEnded, otherwise it transitions to kLive.
 namespace webrtc {
 
 class MediaConstraintsInterface;
 
 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 {
     return video_capturer_->IsScreencast();
   }
   rtc::Optional<bool> needs_denoising() const override {
     return needs_denoising_;
   }
 
   bool GetStats(Stats* stats) override;
 
  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);
 
  private:
   void Stop();
 
   void OnStateChange(cricket::VideoCapturer* capturer,
                      cricket::CaptureState capture_state);
 
   rtc::Thread* signaling_thread_;
   rtc::Thread* worker_thread_;
   rtc::AsyncInvoker invoker_;
   std::unique_ptr<cricket::VideoCapturer> video_capturer_;
   bool started_;
   cricket::VideoFormat format_;
   rtc::Optional<bool> needs_denoising_;
 };
 
 }  // namespace webrtc
 
 #endif  // WEBRTC_PC_VIDEOCAPTURERTRACKSOURCE_H_