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

Unified Diff: media/filters/blocking_url_protocol_unittest.cc

Issue 2719883005: Revert of Replace FFmpegDemuxer thread per element with base::TaskScheduler. (Closed)
Patch Set: Created 3 years, 10 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/blocking_url_protocol.cc ('k') | media/filters/ffmpeg_demuxer.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: media/filters/blocking_url_protocol_unittest.cc
diff --git a/media/filters/blocking_url_protocol_unittest.cc b/media/filters/blocking_url_protocol_unittest.cc
index dda4000651059fb8c6c3f6bd01cfe274310b95f0..266f893b5452e5cee48d8f22a23b69673a7ef426 100644
--- a/media/filters/blocking_url_protocol_unittest.cc
+++ b/media/filters/blocking_url_protocol_unittest.cc
@@ -20,10 +20,9 @@
class BlockingUrlProtocolTest : public testing::Test {
public:
BlockingUrlProtocolTest()
- : url_protocol_(new BlockingUrlProtocol(
- &data_source_,
- base::Bind(&BlockingUrlProtocolTest::OnDataSourceError,
- base::Unretained(this)))) {
+ : url_protocol_(&data_source_,
+ base::Bind(&BlockingUrlProtocolTest::OnDataSourceError,
+ base::Unretained(this))) {
CHECK(data_source_.Initialize(GetTestDataFilePath("bear-320x240.webm")));
}
@@ -34,7 +33,7 @@
MOCK_METHOD0(OnDataSourceError, void());
FileDataSource data_source_;
- std::unique_ptr<BlockingUrlProtocol> url_protocol_;
+ BlockingUrlProtocol url_protocol_;
private:
DISALLOW_COPY_AND_ASSIGN(BlockingUrlProtocolTest);
@@ -44,35 +43,35 @@
TEST_F(BlockingUrlProtocolTest, Read) {
// Set read head to zero as Initialize() will have parsed a bit of the file.
int64_t position = 0;
- EXPECT_TRUE(url_protocol_->SetPosition(0));
- EXPECT_TRUE(url_protocol_->GetPosition(&position));
+ EXPECT_TRUE(url_protocol_.SetPosition(0));
+ EXPECT_TRUE(url_protocol_.GetPosition(&position));
EXPECT_EQ(0, position);
// Read 32 bytes from offset zero and verify position.
uint8_t buffer[32];
- EXPECT_EQ(32, url_protocol_->Read(32, buffer));
- EXPECT_TRUE(url_protocol_->GetPosition(&position));
+ EXPECT_EQ(32, url_protocol_.Read(32, buffer));
+ EXPECT_TRUE(url_protocol_.GetPosition(&position));
EXPECT_EQ(32, position);
// Read an additional 32 bytes and verify position.
- EXPECT_EQ(32, url_protocol_->Read(32, buffer));
- EXPECT_TRUE(url_protocol_->GetPosition(&position));
+ EXPECT_EQ(32, url_protocol_.Read(32, buffer));
+ EXPECT_TRUE(url_protocol_.GetPosition(&position));
EXPECT_EQ(64, position);
// Seek to end and read until EOF.
int64_t size = 0;
- EXPECT_TRUE(url_protocol_->GetSize(&size));
- EXPECT_TRUE(url_protocol_->SetPosition(size - 48));
- EXPECT_EQ(32, url_protocol_->Read(32, buffer));
- EXPECT_TRUE(url_protocol_->GetPosition(&position));
+ EXPECT_TRUE(url_protocol_.GetSize(&size));
+ EXPECT_TRUE(url_protocol_.SetPosition(size - 48));
+ EXPECT_EQ(32, url_protocol_.Read(32, buffer));
+ EXPECT_TRUE(url_protocol_.GetPosition(&position));
EXPECT_EQ(size - 16, position);
- EXPECT_EQ(16, url_protocol_->Read(32, buffer));
- EXPECT_TRUE(url_protocol_->GetPosition(&position));
+ EXPECT_EQ(16, url_protocol_.Read(32, buffer));
+ EXPECT_TRUE(url_protocol_.GetPosition(&position));
EXPECT_EQ(size, position);
- EXPECT_EQ(0, url_protocol_->Read(32, buffer));
- EXPECT_TRUE(url_protocol_->GetPosition(&position));
+ EXPECT_EQ(0, url_protocol_.Read(32, buffer));
+ EXPECT_TRUE(url_protocol_.GetPosition(&position));
EXPECT_EQ(size, position);
}
@@ -81,23 +80,23 @@
uint8_t buffer[32];
EXPECT_CALL(*this, OnDataSourceError());
- EXPECT_EQ(AVERROR(EIO), url_protocol_->Read(32, buffer));
+ EXPECT_EQ(AVERROR(EIO), url_protocol_.Read(32, buffer));
}
TEST_F(BlockingUrlProtocolTest, GetSetPosition) {
int64_t size;
int64_t position;
- EXPECT_TRUE(url_protocol_->GetSize(&size));
- EXPECT_TRUE(url_protocol_->GetPosition(&position));
+ EXPECT_TRUE(url_protocol_.GetSize(&size));
+ EXPECT_TRUE(url_protocol_.GetPosition(&position));
- EXPECT_TRUE(url_protocol_->SetPosition(512));
- EXPECT_FALSE(url_protocol_->SetPosition(size + 1));
- EXPECT_FALSE(url_protocol_->SetPosition(-1));
- EXPECT_TRUE(url_protocol_->GetPosition(&position));
+ EXPECT_TRUE(url_protocol_.SetPosition(512));
+ EXPECT_FALSE(url_protocol_.SetPosition(size + 1));
+ EXPECT_FALSE(url_protocol_.SetPosition(-1));
+ EXPECT_TRUE(url_protocol_.GetPosition(&position));
EXPECT_EQ(512, position);
- EXPECT_TRUE(url_protocol_->SetPosition(size));
- EXPECT_TRUE(url_protocol_->GetPosition(&position));
+ EXPECT_TRUE(url_protocol_.SetPosition(size));
+ EXPECT_TRUE(url_protocol_.GetPosition(&position));
EXPECT_EQ(size, position);
}
@@ -105,21 +104,18 @@
int64_t data_source_size = 0;
int64_t url_protocol_size = 0;
EXPECT_TRUE(data_source_.GetSize(&data_source_size));
- EXPECT_TRUE(url_protocol_->GetSize(&url_protocol_size));
+ EXPECT_TRUE(url_protocol_.GetSize(&url_protocol_size));
EXPECT_NE(0, data_source_size);
EXPECT_EQ(data_source_size, url_protocol_size);
}
TEST_F(BlockingUrlProtocolTest, IsStreaming) {
EXPECT_FALSE(data_source_.IsStreaming());
- EXPECT_FALSE(url_protocol_->IsStreaming());
+ EXPECT_FALSE(url_protocol_.IsStreaming());
data_source_.force_streaming_for_testing();
- url_protocol_.reset(new BlockingUrlProtocol(
- &data_source_, base::Bind(&BlockingUrlProtocolTest::OnDataSourceError,
- base::Unretained(this))));
EXPECT_TRUE(data_source_.IsStreaming());
- EXPECT_TRUE(url_protocol_->IsStreaming());
+ EXPECT_TRUE(url_protocol_.IsStreaming());
}
} // namespace media
« no previous file with comments | « media/filters/blocking_url_protocol.cc ('k') | media/filters/ffmpeg_demuxer.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698