Add support for multiple pixel formats in VideoFrameBuffer

webrtc/api/video/i420_buffer.h

 /*
  *  Copyright (c) 2015 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.
  */
 
 #ifndef WEBRTC_API_VIDEO_I420_BUFFER_H_
 #define WEBRTC_API_VIDEO_I420_BUFFER_H_
 
 #include <memory>
 
 #include "webrtc/api/video/video_rotation.h"
 #include "webrtc/api/video/video_frame_buffer.h"
 #include "webrtc/system_wrappers/include/aligned_malloc.h"
 
 namespace webrtc {
 
 // Plain I420 buffer in standard memory.
-class I420Buffer : public VideoFrameBuffer {
+class I420Buffer : public PlanarYuvBuffer {
  public:
   static rtc::scoped_refptr<I420Buffer> Create(int width, int height);
   static rtc::scoped_refptr<I420Buffer> Create(int width,
                                                int height,
                                                int stride_y,
                                                int stride_u,
                                                int stride_v);
 
   // Create a new buffer and copy the pixel data.
   static rtc::scoped_refptr<I420Buffer> Copy(const VideoFrameBuffer& buffer);
@@ skipping 12 matching lines @@
   static void SetBlack(I420Buffer* buffer);
 
   // Sets all three planes to all zeros. Used to work around for
   // quirks in memory checkers
   // (https://bugs.chromium.org/p/libyuv/issues/detail?id=377) and
   // ffmpeg (http://crbug.com/390941).
   // TODO(nisse): Deprecated. Should be deleted if/when those issues
   // are resolved in a better way. Or in the mean time, use SetBlack.
   void InitializeData();
 
-  // TODO(nisse): Deprecated, use static method instead.

[nisse-webrtc, 2017/05/04 08:15:29]

Also deleted in cl https://codereview.webrtc.org/2854873003/ which I'm
attempting to land.

[magjed_webrtc, 2017/05/04 12:25:13]

Wops, this is a mistake from rebasing. I'm trying to not make unrelated changes.

-  void SetToBlack() { SetBlack(this); }
-
+  PixelFormat Format() const override;
   int width() const override;
   int height() const override;
   const uint8_t* DataY() const override;
   const uint8_t* DataU() const override;
   const uint8_t* DataV() const override;
 
   int StrideY() const override;
   int StrideU() const override;
   int StrideV() const override;
 
-  void* native_handle() const override;
-  rtc::scoped_refptr<VideoFrameBuffer> NativeToI420Buffer() override;
-
   uint8_t* MutableDataY();
   uint8_t* MutableDataU();
   uint8_t* MutableDataV();
 
   // Scale the cropped area of |src| to the size of |this| buffer, and
   // write the result into |this|.
   void CropAndScaleFrom(const VideoFrameBuffer& src,
                         int offset_x,
                         int offset_y,
                         int crop_width,
@@ skipping 25 matching lines @@
   const int height_;
   const int stride_y_;
   const int stride_u_;
   const int stride_v_;
   const std::unique_ptr<uint8_t, AlignedFreeDeleter> data_;
 };
 
 }  // namespace webrtc
 
 #endif  // WEBRTC_API_VIDEO_I420_BUFFER_H_