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

Unified Diff: media/filters/pipeline_integration_test.cc

Issue 10447035: Introducing DecoderBuffer and general Buffer cleanup. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Buffer Bonanza! Created 8 years, 7 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
Index: media/filters/pipeline_integration_test.cc
diff --git a/media/filters/pipeline_integration_test.cc b/media/filters/pipeline_integration_test.cc
index a9347efd3e160df0e9ce1d890e9882a294bae485..da88a2625b27a9fccf1b7d3a3e0247de415333ed 100644
--- a/media/filters/pipeline_integration_test.cc
+++ b/media/filters/pipeline_integration_test.cc
@@ -24,10 +24,10 @@ class MockMediaSource : public ChunkDemuxerClient {
: url_(GetTestDataURL(filename)),
current_position_(0),
initial_append_size_(initial_append_size) {
- ReadTestDataFile(filename, &file_data_, &file_data_size_);
+ file_data_ = ReadTestDataFile(filename);
DCHECK_GT(initial_append_size_, 0);
- DCHECK_LE(initial_append_size_, file_data_size_);
+ DCHECK_LE(initial_append_size_, file_data_->GetDataSize());
}
virtual ~MockMediaSource() {}
@@ -45,7 +45,7 @@ class MockMediaSource : public ChunkDemuxerClient {
chunk_demuxer_->FlushData();
DCHECK_GE(new_position, 0);
- DCHECK_LT(new_position, file_data_size_);
+ DCHECK_LT(new_position, file_data_->GetDataSize());
current_position_ = new_position;
AppendData(seek_append_size);
@@ -53,11 +53,10 @@ class MockMediaSource : public ChunkDemuxerClient {
void AppendData(int size) {
DCHECK(chunk_demuxer_.get());
- DCHECK_LT(current_position_, file_data_size_);
- DCHECK_LE(current_position_ + size, file_data_size_);
- CHECK(chunk_demuxer_->AppendData(kSourceId,
- file_data_.get() + current_position_,
- size));
+ DCHECK_LT(current_position_, file_data_->GetDataSize());
+ DCHECK_LE(current_position_ + size, file_data_->GetDataSize());
+ CHECK(chunk_demuxer_->AppendData(
+ kSourceId, file_data_->GetData() + current_position_, size));
current_position_ += size;
}
@@ -98,8 +97,7 @@ class MockMediaSource : public ChunkDemuxerClient {
private:
std::string url_;
- scoped_array<uint8> file_data_;
- int file_data_size_;
+ scoped_refptr<DecoderBuffer> file_data_;
int current_position_;
int initial_append_size_;
scoped_refptr<ChunkDemuxer> chunk_demuxer_;

Powered by Google App Engine
This is Rietveld 408576698