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

Unified Diff: media/formats/mp2t/es_parser_adts.h

Issue 2378443002: Fix MSE ADTS parsing on Android. (Closed)
Patch Set: Address comments. Created 4 years, 3 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
« no previous file with comments | « media/filters/audio_decoder_unittest.cc ('k') | media/formats/mp2t/es_parser_adts.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: media/formats/mp2t/es_parser_adts.h
diff --git a/media/formats/mp2t/es_parser_adts.h b/media/formats/mp2t/es_parser_adts.h
index 2350f250806437d178ba1c7adf0af126f36d5342..1a0cb71ff5d8304bf2842bb1817720c5d1405548 100644
--- a/media/formats/mp2t/es_parser_adts.h
+++ b/media/formats/mp2t/es_parser_adts.h
@@ -18,6 +18,7 @@
#include "media/base/audio_decoder_config.h"
#include "media/base/media_export.h"
#include "media/formats/mp2t/es_parser.h"
+#include "media/formats/mpeg/adts_stream_parser.h"
namespace media {
class AudioTimestampHelper;
@@ -61,7 +62,7 @@ class MEDIA_EXPORT EsParserAdts : public EsParser {
// Signal any audio configuration change (if any).
// Return false if the current audio config is not
// a supported ADTS audio config.
- bool UpdateAudioConfiguration(const uint8_t* adts_header);
+ bool UpdateAudioConfiguration(const uint8_t* adts_header, int size);
// Callbacks:
// - to signal a new audio configuration,
@@ -79,6 +80,8 @@ class MEDIA_EXPORT EsParserAdts : public EsParser {
// Last audio config.
AudioDecoderConfig last_audio_decoder_config_;
+ ADTSStreamParser adts_parser_;
+
DISALLOW_COPY_AND_ASSIGN(EsParserAdts);
};
« no previous file with comments | « media/filters/audio_decoder_unittest.cc ('k') | media/formats/mp2t/es_parser_adts.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698