Chromium Code Reviews| OLD | NEW |
|---|---|
| 1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. | 1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. |
| 2 // Use of this source code is governed by a BSD-style license that can be | 2 // Use of this source code is governed by a BSD-style license that can be |
| 3 // found in the LICENSE file. | 3 // found in the LICENSE file. |
| 4 | 4 |
| 5 #include <algorithm> | 5 #include <algorithm> |
| 6 #include <string> | 6 #include <string> |
| 7 | 7 |
| 8 #include "base/bind.h" | 8 #include "base/bind.h" |
| 9 #include "base/bind_helpers.h" | 9 #include "base/bind_helpers.h" |
| 10 #include "base/logging.h" | 10 #include "base/logging.h" |
| (...skipping 16 matching lines...) Expand all Loading... | |
| 27 class MP4StreamParserTest : public testing::Test { | 27 class MP4StreamParserTest : public testing::Test { |
| 28 public: | 28 public: |
| 29 MP4StreamParserTest() | 29 MP4StreamParserTest() |
| 30 : parser_(new MP4StreamParser), | 30 : parser_(new MP4StreamParser), |
| 31 got_configs_(false) { | 31 got_configs_(false) { |
| 32 } | 32 } |
| 33 | 33 |
| 34 protected: | 34 protected: |
| 35 scoped_ptr<MP4StreamParser> parser_; | 35 scoped_ptr<MP4StreamParser> parser_; |
| 36 base::TimeDelta segment_start_; | 36 base::TimeDelta segment_start_; |
| 37 bool got_configs_; | 37 bool got_configs_; |
|
ddorwin
2012/07/17 01:14:21
s/got/have/ seems better
strobe_
2012/07/19 02:43:35
Done.
| |
| 38 | 38 |
| 39 bool AppendData(const uint8* data, size_t length) { | 39 bool AppendData(const uint8* data, size_t length) { |
| 40 parser_->Parse(data, length); | 40 return parser_->Parse(data, length); |
| 41 return true; | |
| 42 } | 41 } |
| 43 | 42 |
| 44 bool AppendDataInPieces(const uint8* data, size_t length) { | 43 bool AppendDataInPieces(const uint8* data, size_t length) { |
| 45 return AppendDataInPieces(data, length, 7); | 44 return AppendDataInPieces(data, length, 7); |
|
ddorwin
2012/07/17 01:14:21
Why 7? That's an odd size.
strobe_
2012/07/19 02:43:35
Method unused, removed.
| |
| 46 } | 45 } |
| 47 | 46 |
| 48 bool AppendDataInPieces(const uint8* data, size_t length, size_t piece_size) { | 47 bool AppendDataInPieces(const uint8* data, size_t length, size_t piece_size) { |
| 49 const uint8* start = data; | 48 const uint8* start = data; |
| 50 const uint8* end = data + length; | 49 const uint8* end = data + length; |
| 51 while (start < end) { | 50 while (start < end) { |
| 52 size_t append_size = std::min(piece_size, | 51 size_t append_size = std::min(piece_size, |
| 53 static_cast<size_t>(end - start)); | 52 static_cast<size_t>(end - start)); |
| 54 if (!AppendData(start, append_size)) | 53 if (!AppendData(start, append_size)) |
| 55 return false; | 54 return false; |
| (...skipping 20 matching lines...) Expand all Loading... | |
| 76 return true; | 75 return true; |
| 77 } | 76 } |
| 78 | 77 |
| 79 bool NewBuffersF(const StreamParser::BufferQueue& bufs) { | 78 bool NewBuffersF(const StreamParser::BufferQueue& bufs) { |
| 80 DVLOG(2) << "NewBuffersF: " << bufs.size() << " buffers"; | 79 DVLOG(2) << "NewBuffersF: " << bufs.size() << " buffers"; |
| 81 for (StreamParser::BufferQueue::const_iterator buf = bufs.begin(); | 80 for (StreamParser::BufferQueue::const_iterator buf = bufs.begin(); |
| 82 buf != bufs.end(); buf++) { | 81 buf != bufs.end(); buf++) { |
| 83 DVLOG(3) << " n=" << buf - bufs.begin() | 82 DVLOG(3) << " n=" << buf - bufs.begin() |
| 84 << ", size=" << (*buf)->GetDataSize() | 83 << ", size=" << (*buf)->GetDataSize() |
| 85 << ", dur=" << (*buf)->GetDuration().InMilliseconds(); | 84 << ", dur=" << (*buf)->GetDuration().InMilliseconds(); |
| 86 EXPECT_LE(segment_start_, (*buf)->GetTimestamp()); | 85 EXPECT_LE(segment_start_, (*buf)->GetTimestamp()); |
|
ddorwin
2012/07/17 01:14:21
You're really testing the timestamp, which means t
strobe_
2012/07/19 02:43:35
Well, not according to the macro definition ;) Aar
| |
| 87 } | 86 } |
| 88 return true; | 87 return true; |
| 89 } | 88 } |
| 90 | 89 |
| 91 bool KeyNeededF(scoped_array<uint8> init_data, int init_data_size) { | 90 bool KeyNeededF(scoped_array<uint8> init_data, int init_data_size) { |
| 92 DVLOG(1) << "KeyNeededF: " << init_data_size; | 91 DVLOG(1) << "KeyNeededF: " << init_data_size; |
| 93 return true; | 92 return true; |
| 94 } | 93 } |
| 95 | 94 |
| 96 void NewSegmentF(TimeDelta start_dts) { | 95 void NewSegmentF(TimeDelta start_dts) { |
| 97 DVLOG(1) << "NewSegmentF: " << start_dts.InMilliseconds(); | 96 DVLOG(1) << "NewSegmentF: " << start_dts.InMilliseconds(); |
| 98 segment_start_ = start_dts; | 97 segment_start_ = start_dts; |
| 99 } | 98 } |
| 100 | 99 |
| 101 void InitializeParser() { | 100 void InitializeParser() { |
| 102 parser_->Init( | 101 parser_->Init( |
| 103 base::Bind(&MP4StreamParserTest::InitF, base::Unretained(this)), | 102 base::Bind(&MP4StreamParserTest::InitF, base::Unretained(this)), |
| 104 base::Bind(&MP4StreamParserTest::NewConfigF, base::Unretained(this)), | 103 base::Bind(&MP4StreamParserTest::NewConfigF, base::Unretained(this)), |
| 105 base::Bind(&MP4StreamParserTest::NewBuffersF, base::Unretained(this)), | 104 base::Bind(&MP4StreamParserTest::NewBuffersF, base::Unretained(this)), |
| 106 base::Bind(&MP4StreamParserTest::NewBuffersF, base::Unretained(this)), | 105 base::Bind(&MP4StreamParserTest::NewBuffersF, base::Unretained(this)), |
| 107 base::Bind(&MP4StreamParserTest::KeyNeededF, base::Unretained(this)), | 106 base::Bind(&MP4StreamParserTest::KeyNeededF, base::Unretained(this)), |
| 108 base::Bind(&MP4StreamParserTest::NewSegmentF, base::Unretained(this))); | 107 base::Bind(&MP4StreamParserTest::NewSegmentF, base::Unretained(this))); |
| 109 } | 108 } |
| 110 | 109 |
| 111 bool ParseMP4File(const std::string& filename, int append_size) { | 110 bool ParseMP4File(const std::string& filename, int append_size) { |
|
ddorwin
2012/07/17 01:14:21
size=bytes?
strobe_
2012/07/19 02:43:35
Done.
| |
| 112 InitializeParser(); | 111 InitializeParser(); |
| 113 | 112 |
| 114 scoped_refptr<DecoderBuffer> buffer = ReadTestDataFile(filename); | 113 scoped_refptr<DecoderBuffer> buffer = ReadTestDataFile(filename); |
| 115 EXPECT_TRUE(AppendDataInPieces(buffer->GetData(), | 114 EXPECT_TRUE(AppendDataInPieces(buffer->GetData(), |
| 116 buffer->GetDataSize(), | 115 buffer->GetDataSize(), |
| 117 append_size)); | 116 append_size)); |
| 118 return true; | 117 return true; |
| 119 } | 118 } |
| 120 }; | 119 }; |
| 121 | 120 |
| 122 TEST_F(MP4StreamParserTest, TestParseBearDASH) { | 121 TEST_F(MP4StreamParserTest, TestParseBearDASH) { |
|
ddorwin
2012/07/17 01:14:21
Is "bear" really relevant to the test? What is int
strobe_
2012/07/19 02:43:35
Renamed and commented.
| |
| 123 ParseMP4File("bear.1280x720_dash.mp4", 512); | 122 ParseMP4File("bear.1280x720_dash.mp4", 512); |
|
ddorwin
2012/07/17 01:14:21
Unrelated to this CL, is this file really "dash" o
strobe_
2012/07/19 02:43:35
It complies with 23009-1 6.3.4 and 14496-12 FAMD3
| |
| 124 } | 123 } |
| 125 | 124 |
| 126 TEST_F(MP4StreamParserTest, TestMultiFragmentAppend) { | 125 TEST_F(MP4StreamParserTest, TestMultiFragmentAppend) { |
| 127 ParseMP4File("bear.1280x720_dash.mp4", 768432); | 126 ParseMP4File("bear.1280x720_dash.mp4", 768432); |
|
ddorwin
2012/07/17 01:14:21
What do the sizes mean?
strobe_
2012/07/19 02:43:35
Commented.
| |
| 128 } | 127 } |
| 129 | 128 |
| 130 TEST_F(MP4StreamParserTest, TestReinitialization) { | 129 TEST_F(MP4StreamParserTest, TestReinitialization) { |
| 131 InitializeParser(); | 130 InitializeParser(); |
|
ddorwin
2012/07/17 01:14:21
Where is the second initialization? Shouldn't re-i
strobe_
2012/07/19 02:43:35
Commented.
ddorwin
2012/07/19 06:11:10
Where?
| |
| 132 | 131 |
| 133 scoped_refptr<DecoderBuffer> buffer = | 132 scoped_refptr<DecoderBuffer> buffer = |
| 134 ReadTestDataFile("bear.1280x720_dash.mp4"); | 133 ReadTestDataFile("bear.1280x720_dash.mp4"); |
| 135 EXPECT_TRUE(AppendDataInPieces(buffer->GetData(), | 134 EXPECT_TRUE(AppendDataInPieces(buffer->GetData(), |
| 136 buffer->GetDataSize(), | 135 buffer->GetDataSize(), |
| 137 512)); | 136 512)); |
| 138 EXPECT_TRUE(AppendDataInPieces(buffer->GetData(), | 137 EXPECT_TRUE(AppendDataInPieces(buffer->GetData(), |
| 139 buffer->GetDataSize(), | 138 buffer->GetDataSize(), |
| 140 512)); | 139 512)); |
| 141 } | 140 } |
| 142 | 141 |
| 143 } // namespace mp4 | 142 } // namespace mp4 |
| 144 } // namespace media | 143 } // namespace media |
| OLD | NEW |