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

Unified Diff: media/filters/file_data_source_unittest.cc

Issue 212803004: Separate DemuxerHost from DataSourceHost. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Rebase. Created 6 years, 9 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/file_data_source.cc ('k') | media/filters/pipeline_integration_test.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: media/filters/file_data_source_unittest.cc
diff --git a/media/filters/file_data_source_unittest.cc b/media/filters/file_data_source_unittest.cc
index f6c9b8988586c13f4a1db7bfe7ed3c4fc550840b..959118d15b2225e92058e46c8b23d1f3dd073459 100644
--- a/media/filters/file_data_source_unittest.cc
+++ b/media/filters/file_data_source_unittest.cc
@@ -44,31 +44,15 @@ base::FilePath TestFileURL() {
return data_dir;
}
-// Test that FileDataSource call the appropriate methods on its filter host.
-TEST(FileDataSourceTest, OpenFile) {
- StrictMock<MockDataSourceHost> host;
- EXPECT_CALL(host, SetTotalBytes(10));
- EXPECT_CALL(host, AddBufferedByteRange(0, 10));
-
- FileDataSource data_source;
- data_source.set_host(&host);
- EXPECT_TRUE(data_source.Initialize(TestFileURL()));
-
- data_source.Stop(NewExpectedClosure());
-}
-
// Use the mock filter host to directly call the Read and GetPosition methods.
TEST(FileDataSourceTest, ReadData) {
int64 size;
uint8 ten_bytes[10];
// Create our mock filter host and initialize the data source.
- NiceMock<MockDataSourceHost> host;
FileDataSource data_source;
- data_source.set_host(&host);
EXPECT_TRUE(data_source.Initialize(TestFileURL()));
-
EXPECT_TRUE(data_source.GetSize(&size));
EXPECT_EQ(10, size);
« no previous file with comments | « media/filters/file_data_source.cc ('k') | media/filters/pipeline_integration_test.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698