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

Unified Diff: media/mp4/mp4_stream_parser_unittest.cc

Issue 11819013: Add support for accepting MPEG2 AAC-LC bitstreams. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 7 years, 11 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
« media/mp4/mp4_stream_parser.cc ('K') | « media/mp4/mp4_stream_parser.cc ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: media/mp4/mp4_stream_parser_unittest.cc
diff --git a/media/mp4/mp4_stream_parser_unittest.cc b/media/mp4/mp4_stream_parser_unittest.cc
index 66f3410e97e386bfb6d6b2552588bb13bba849b6..d43072585f557fecf630cf359fc9147110736439 100644
--- a/media/mp4/mp4_stream_parser_unittest.cc
+++ b/media/mp4/mp4_stream_parser_unittest.cc
@@ -15,6 +15,7 @@
#include "media/base/stream_parser_buffer.h"
#include "media/base/test_data_util.h"
#include "media/base/video_decoder_config.h"
+#include "media/mp4/es_descriptor.h"
#include "media/mp4/mp4_stream_parser.h"
#include "testing/gtest/include/gtest/gtest.h"
@@ -29,8 +30,10 @@ static const char kMp4InitDataType[] = "video/mp4";
class MP4StreamParserTest : public testing::Test {
public:
MP4StreamParserTest()
- : parser_(new MP4StreamParser(false)),
- configs_received_(false) {
+ : configs_received_(false) {
+ std::set<int> audio_object_types;
+ audio_object_types.insert(kISO_14496_3);
scherkus (not reviewing) 2013/01/08 23:51:08 OOC doing this provides sufficient test coverage?
acolwell GONE FROM CHROMIUM 2013/01/16 17:35:09 Added test for MPEG2 AAC-LC code path.
+ parser_.reset(new MP4StreamParser(audio_object_types, false));
}
protected:
« media/mp4/mp4_stream_parser.cc ('K') | « media/mp4/mp4_stream_parser.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698