Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(181)

Unified Diff: webrtc/modules/video_coding/codecs/h264/h264.cc

Issue 1602523004: Added EncodedImage::GetBufferPaddingBytes. (Closed) Base URL: https://chromium.googlesource.com/external/webrtc.git@master
Patch Set: 0 additional bytes, make ios compile Created 4 years, 11 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: webrtc/modules/video_coding/codecs/h264/h264.cc
diff --git a/webrtc/modules/video_coding/codecs/h264/h264.cc b/webrtc/modules/video_coding/codecs/h264/h264.cc
index 645ed2cad7af3bdf99643ee0fc5972a20e0b96d8..2b86977a3a8682b178a8c0f500e9e8cd1eaa2f79 100644
--- a/webrtc/modules/video_coding/codecs/h264/h264.cc
+++ b/webrtc/modules/video_coding/codecs/h264/h264.cc
@@ -20,6 +20,10 @@
namespace webrtc {
+// TODO(hbos): The FFmpeg video decoder will require up to 8 bytes, update this
+// when the FFmpeg decoder is added.
stefan-webrtc 2016/01/19 15:06:20 Should we also file a bug to the ffmpeg project? S
hbos 2016/01/19 16:50:34 It is documented in their source code (https://cod
stefan-webrtc 2016/01/21 10:34:47 I think we should at least ask whether they would
hbos 2016/01/21 11:17:26 Agreed, I created a ticket (https://trac.ffmpeg.or
+const size_t kEncodedImagePaddingH264 = 0;
+
// We need this file to be C++ only so it will compile properly for all
// platforms. In order to write ObjC specific implementations we use private
// externs. This function is defined in h264.mm.

Powered by Google App Engine
This is Rietveld 408576698