Change H264AnnexBBitstreamConverter to use AVCDecoderConfigurationRecord.

media/formats/mp4/box_definitions.h

Index: media/formats/mp4/box_definitions.h
diff --git a/media/formats/mp4/box_definitions.h b/media/formats/mp4/box_definitions.h
index 9daa82e136c11e188854f377d741971f0b854ff8..6ca5523645e61eff25417d1cce6a61a51b047c69 100644
--- a/media/formats/mp4/box_definitions.h
+++ b/media/formats/mp4/box_definitions.h
@@ -150,6 +150,8 @@ struct MEDIA_EXPORT HandlerReference : Box {
 struct MEDIA_EXPORT AVCDecoderConfigurationRecord : Box {
   DECLARE_BOX_METHODS(AVCDecoderConfigurationRecord);
 
+  bool Parse(const uint8* data, int data_size);

[scherkus (not reviewing), 2014/05/05 18:22:17]

docs / how is this different from the one from DECLARE_BOX_METHODS():

virtual bool Parse(BoxReader* reader) OVERRIDE;


can we have the caller construct a BoxReader instead?

[acolwell GONE FROM CHROMIUM, 2014/05/05 18:49:02]

Done.
No we can't use the BoxReader because the data passed to this function doesn't
actually have the box header. This function allows you to parse the contents of
the box, w/o the need for the box header.

+
   uint8 version;
   uint8 profile_indication;
   uint8 profile_compatibility;
@@ -161,6 +163,9 @@ struct MEDIA_EXPORT AVCDecoderConfigurationRecord : Box {
 
   std::vector<SPS> sps_list;
   std::vector<PPS> pps_list;
+
+private:

[scherkus (not reviewing), 2014/05/05 18:22:17]

indent one more

[acolwell GONE FROM CHROMIUM, 2014/05/05 18:49:02]

Done.

+  bool ParseInternal(BufferReader* reader);
 };
 
 struct MEDIA_EXPORT PixelAspectRatioBox : Box {