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

Unified Diff: media/formats/mpeg/mpeg1_audio_stream_parser_unittest.cc

Issue 506943003: Support MPEG1 audio in the MPEG2-TS stream parser. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Address CR comments from patch set #5. Created 6 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/formats/mpeg/mpeg1_audio_stream_parser.cc ('k') | media/media.gyp » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: media/formats/mpeg/mpeg1_audio_stream_parser_unittest.cc
diff --git a/media/formats/mpeg/mp3_stream_parser_unittest.cc b/media/formats/mpeg/mpeg1_audio_stream_parser_unittest.cc
similarity index 85%
rename from media/formats/mpeg/mp3_stream_parser_unittest.cc
rename to media/formats/mpeg/mpeg1_audio_stream_parser_unittest.cc
index 1638752715998371cd37b1b3932de409d5c0eb23..1c245161303182d760d5496f87a484aeccebf3f1 100644
--- a/media/formats/mpeg/mp3_stream_parser_unittest.cc
+++ b/media/formats/mpeg/mpeg1_audio_stream_parser_unittest.cc
@@ -4,22 +4,23 @@
#include "media/base/test_data_util.h"
#include "media/formats/common/stream_parser_test_base.h"
-#include "media/formats/mpeg/mp3_stream_parser.h"
+#include "media/formats/mpeg/mpeg1_audio_stream_parser.h"
#include "testing/gtest/include/gtest/gtest.h"
namespace media {
-class MP3StreamParserTest : public StreamParserTestBase, public testing::Test {
+class MPEG1AudioStreamParserTest
+ : public StreamParserTestBase, public testing::Test {
public:
- MP3StreamParserTest()
+ MPEG1AudioStreamParserTest()
: StreamParserTestBase(
- scoped_ptr<StreamParser>(new MP3StreamParser()).Pass()) {}
- virtual ~MP3StreamParserTest() {}
+ scoped_ptr<StreamParser>(new MPEG1AudioStreamParser()).Pass()) {}
+ virtual ~MPEG1AudioStreamParserTest() {}
};
// Test parsing with small prime sized chunks to smoke out "power of
// 2" field size assumptions.
-TEST_F(MP3StreamParserTest, UnalignedAppend) {
+TEST_F(MPEG1AudioStreamParserTest, UnalignedAppend) {
const std::string expected =
"NewSegment"
"{ 0K }"
@@ -45,7 +46,7 @@ TEST_F(MP3StreamParserTest, UnalignedAppend) {
// Test parsing with a larger piece size to verify that multiple buffers
// are passed to |new_buffer_cb_|.
-TEST_F(MP3StreamParserTest, UnalignedAppend512) {
+TEST_F(MPEG1AudioStreamParserTest, UnalignedAppend512) {
const std::string expected =
"NewSegment"
"{ 0K 26K 52K 78K }"
@@ -59,7 +60,7 @@ TEST_F(MP3StreamParserTest, UnalignedAppend512) {
EXPECT_GT(last_audio_config().codec_delay(), 0);
}
-TEST_F(MP3StreamParserTest, MetadataParsing) {
+TEST_F(MPEG1AudioStreamParserTest, MetadataParsing) {
scoped_refptr<DecoderBuffer> buffer = ReadTestDataFile("sfx.mp3");
const uint8_t* buffer_ptr = buffer->data();
« no previous file with comments | « media/formats/mpeg/mpeg1_audio_stream_parser.cc ('k') | media/media.gyp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698