Remove V4L2CaptureDelegate{Single,Multi}Plane, VCD::Client::OnIncomingCapturedYuvData()

media/capture/video/linux/video_capture_device_linux.h

 // Copyright (c) 2012 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.
 
 // Linux specific implementation of VideoCaptureDevice.
 // V4L2 is used for capturing. V4L2 does not provide its own thread for
 // capturing so this implementation uses a Chromium thread for fetching frames
 // from V4L2.
 
 #ifndef MEDIA_VIDEO_CAPTURE_LINUX_VIDEO_CAPTURE_DEVICE_LINUX_H_
@@ skipping 30 matching lines @@
 
  protected:
   void SetRotation(int rotation);
 
  private:
   static int TranslatePowerLineFrequencyToV4L2(PowerLineFrequency frequency);
 
   // Internal delegate doing the actual capture setting, buffer allocation and
   // circulation with the V4L2 API. Created and deleted in the thread where
   // VideoCaptureDeviceLinux lives but otherwise operating on |v4l2_thread_|.
-  scoped_refptr<V4L2CaptureDelegate> capture_impl_;
+  // Weak.
+  V4L2CaptureDelegate* capture_impl_;

[perkj_chrome, 2016/02/12 11:01:40]

this looks like a memory leak. Should be scoped_ptr - or just keep 
V4L2CaptureDelegate as beeing ref counted.

[mcasas, 2016/02/12 21:32:43]

Acknowledged.

 
   base::Thread v4l2_thread_;  // Thread used for reading data from the device.
 
   const Name device_name_;
 
   DISALLOW_IMPLICIT_CONSTRUCTORS(VideoCaptureDeviceLinux);
 };
 
 }  // namespace media
 
 #endif  // MEDIA_VIDEO_CAPTURE_LINUX_VIDEO_CAPTURE_DEVICE_LINUX_H_