Move key frame flag from StreamParserBuffer to DecoderBuffer

media/base/decoder_buffer.h

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef MEDIA_BASE_DECODER_BUFFER_H_
 #define MEDIA_BASE_DECODER_BUFFER_H_
 
 #include <string>
 #include <utility>
 
@@ skipping 24 matching lines @@
   enum {
     kPaddingSize = 32,
 #if defined(ARCH_CPU_ARM_FAMILY)
     kAlignmentSize = 16
 #else
     kAlignmentSize = 32
 #endif
   };
 
   // Allocates buffer with |size| >= 0.  Buffer will be padded and aligned
-  // as necessary.
+  // as necessary, and is_key_frame() will default to false.

[xhwang, 2014/11/11 23:24:26]

s/is_key_frame()/|is_key_frame_|

[wolenetz, 2014/11/11 23:52:54]

Done.

   explicit DecoderBuffer(int size);
 
   // Create a DecoderBuffer whose |data_| is copied from |data|.  Buffer will be
   // padded and aligned as necessary.  |data| must not be NULL and |size| >= 0.
+  // The buffer's is_key_frame() will default to false.

[xhwang, 2014/11/11 23:24:26]

ditto

[wolenetz, 2014/11/11 23:52:54]

Done.

   static scoped_refptr<DecoderBuffer> CopyFrom(const uint8* data, int size);
 
   // Create a DecoderBuffer whose |data_| is copied from |data| and |side_data_|
   // is copied from |side_data|. Buffers will be padded and aligned as necessary
-  // Data pointers must not be NULL and sizes must be >= 0.
+  // Data pointers must not be NULL and sizes must be >= 0. The buffer's
+  // is_key_frame() will default to false.

[xhwang, 2014/11/11 23:24:26]

ditto

[wolenetz, 2014/11/11 23:52:54]

Done.

   static scoped_refptr<DecoderBuffer> CopyFrom(const uint8* data, int size,
                                                const uint8* side_data,
                                                int side_data_size);
 
   // Create a DecoderBuffer indicating we've reached end of stream.
   //
   // Calling any method other than end_of_stream() on the resulting buffer
   // is disallowed.
   static scoped_refptr<DecoderBuffer> CreateEOSBuffer();
 
@@ skipping 82 matching lines @@
     return splice_timestamp_;
   }
 
   // When set to anything but kNoTimestamp() indicates this buffer is part of a
   // splice around |splice_timestamp|.
   void set_splice_timestamp(base::TimeDelta splice_timestamp) {
     DCHECK(!end_of_stream());
     splice_timestamp_ = splice_timestamp;
   }
 
+  bool is_key_frame() const {
+    DCHECK(!end_of_stream());
+    return is_key_frame_;
+  }
+
+  void set_is_key_frame(bool is_key_frame) {
+    DCHECK(!end_of_stream());
+    is_key_frame_ = is_key_frame;
+  }
+
   // Returns a human-readable string describing |*this|.
   std::string AsHumanReadableString();
 
  protected:
   friend class base::RefCountedThreadSafe<DecoderBuffer>;
 
   // Allocates a buffer of size |size| >= 0 and copies |data| into it.  Buffer
   // will be padded and aligned as necessary.  If |data| is NULL then |data_| is
-  // set to NULL and |buffer_size_| to 0.
+  // set to NULL and |buffer_size_| to 0.  is_key_frame() will default to false.

[xhwang, 2014/11/11 23:24:26]

ditto

[wolenetz, 2014/11/11 23:52:54]

Done.

   DecoderBuffer(const uint8* data, int size,
                 const uint8* side_data, int side_data_size);
   virtual ~DecoderBuffer();
 
  private:
   base::TimeDelta timestamp_;
   base::TimeDelta duration_;
 
   int size_;
   scoped_ptr<uint8, base::AlignedFreeDeleter> data_;
   int side_data_size_;
   scoped_ptr<uint8, base::AlignedFreeDeleter> side_data_;
   scoped_ptr<DecryptConfig> decrypt_config_;
   DiscardPadding discard_padding_;
   base::TimeDelta splice_timestamp_;
+  bool is_key_frame_;
 
   // Constructor helper method for memory allocations.
   void Initialize();
 
   DISALLOW_COPY_AND_ASSIGN(DecoderBuffer);
 };
 
 }  // namespace media
 
 #endif  // MEDIA_BASE_DECODER_BUFFER_H_