commit 07f80cc3936b3d68c668a942ef1089a5ba12bf9e
author Sergey Silkin <ssilkin@webrtc.org> Mon Apr 09 13:11:59 2018 +0200
committer Commit Bot <commit-bot@chromium.org> Tue Apr 10 12:09:55 2018 +0000
tree 0dd2e282b3ed8ae014fc2c4b74b74b4d70b8ba58
parent bda78c9464f4d0e6ff26b0035624a967269c3b0a

Set first_frame_in_picture on first encoded frame of picture.

Set the flag based on coded length of buffered frame which is reset
after picture is encoded and, thus, is equal to zero when encoder
delivers first frame of next picture.

Before this change first_frame_in_picture was set based on index of
spatial layer of encoded frame. This is not right anymore since encoder
can drop base layer but deliver upper layers.

Bug: chromium:828350
Change-Id: I12c7534240de8bc4905f04ff368cc3704720a70b
Reviewed-on: https://webrtc-review.googlesource.com/68561
Reviewed-by: Rasmus Brandt <brandtr@webrtc.org>
Commit-Queue: Sergey Silkin <ssilkin@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#22805}

modules/video_coding/codecs/vp9/vp9_impl.h

diff --git a/modules/video_coding/codecs/vp9/vp9_impl.h b/modules/video_coding/codecs/vp9/vp9_impl.h
index 3f10d26..b158523 100644
--- a/modules/video_coding/codecs/vp9/vp9_impl.h
+++ b/modules/video_coding/codecs/vp9/vp9_impl.h
@@ -74,7 +74,8 @@
 
   void PopulateCodecSpecific(CodecSpecificInfo* codec_specific,
                              const vpx_codec_cx_pkt& pkt,
-                             uint32_t timestamp);
+                             uint32_t timestamp,
+                             bool first_frame_in_picture);
 
   bool ExplicitlyConfiguredSpatialLayers() const;
   bool SetSvcRates(const BitrateAllocation& bitrate_allocation);