commit bd7b0e3aa7104b94ef5fdf1b1821179346a99e69
author Vignesh Venkatasubramanian <vigneshv@google.com> Tue Apr 10 11:31:59 2018 -0700
committer Yaowu Xu <yaowu@google.com> Wed Apr 11 15:03:42 2018 +0000
tree 1a2b67877bf0a81b79f7408f7b1010822261313e
parent ef6e2df84bc4d3159de19e735fe3ee1def320efb

obu: Remove trailing_bits for OBU_FRAME

It is sufficient to have byte_alignment after the frame_header_obu
in an OBU_FRAME per the spec and as per the discussion in the bug.

BUG=aomedia:1712

Change-Id: I29e6d1b2513d1f1ad7cd86aec6dc5667af73217e

av1/decoder/obu.c

diff --git a/av1/decoder/obu.c b/av1/decoder/obu.c
index e84fa85..f376775 100644
--- a/av1/decoder/obu.c
+++ b/av1/decoder/obu.c
@@ -248,8 +248,10 @@
 static uint32_t read_frame_header_obu(AV1Decoder *pbi,
                                       struct aom_read_bit_buffer *rb,
                                       const uint8_t *data,
-                                      const uint8_t **p_data_end) {
-  av1_decode_frame_headers_and_setup(pbi, rb, data, p_data_end);
+                                      const uint8_t **p_data_end,
+                                      int trailing_bits_present) {
+  av1_decode_frame_headers_and_setup(pbi, rb, data, p_data_end,
+                                     trailing_bits_present);
   return (uint32_t)(pbi->uncomp_hdr_size);
 }
 
@@ -512,7 +514,8 @@
         // Only decode first frame header received
         if (!frame_header_received) {
           av1_init_read_bit_buffer(pbi, &rb, data, data_end);
-          frame_header_size = read_frame_header_obu(pbi, &rb, data, p_data_end);
+          frame_header_size = read_frame_header_obu(
+              pbi, &rb, data, p_data_end, obu_header.type != OBU_FRAME);
           frame_header_received = 1;
         }
         decoded_payload_size = frame_header_size;