Added EncodedImage::GetBufferPaddingBytes.

webrtc/video_frame.h

 /*
  *  Copyright (c) 2014 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 155 matching lines @@
   uint32_t timestamp_;
   int64_t ntp_time_ms_;
   int64_t render_time_ms_;
   VideoRotation rotation_;
 };
 
 
 // TODO(pbos): Rename EncodedFrame and reformat this class' members.
 class EncodedImage {
  public:
+  // Some decoders require encoded image buffers to be padded with a small
+  // number of additional bytes (due to over-reading byte readers).
+  static size_t GetBufferPadding(VideoCodecType codec_type);

[hbos, 2016/01/19 14:22:18]

(I originally wanted to place this in VideoDecoder but experienced circular
dependencies. This place makes sense too though.)

[stefan-webrtc, 2016/01/19 15:06:20]

Somewhat... VideoDecoder would probably have been slightly nicer, but if the
dependencies are weird maybe this is good enough.

[hbos, 2016/01/19 16:50:34]

They look weird to me.

+
   EncodedImage() : EncodedImage(nullptr, 0, 0) {}
   EncodedImage(uint8_t* buffer, size_t length, size_t size)
       : _buffer(buffer), _length(length), _size(size) {}
 
   struct AdaptReason {
     AdaptReason()
         : quality_resolution_downscales(-1),
           bw_resolutions_disabled(-1) {}
 
     int quality_resolution_downscales;  // Number of times this frame is down
@@ skipping 13 matching lines @@
   uint8_t* _buffer;
   size_t _length;
   size_t _size;
   bool _completeFrame = false;
   AdaptReason adapt_reason_;
   int qp_ = -1;  // Quantizer value.
 };
 
 }  // namespace webrtc
 #endif  // WEBRTC_VIDEO_FRAME_H_