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

Unified Diff: chrome/browser/chromeos/drive/file_system/download_operation_unittest.cc

Issue 408143014: Rename OperationObserver to OperationDelegate (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 6 years, 5 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: chrome/browser/chromeos/drive/file_system/download_operation_unittest.cc
diff --git a/chrome/browser/chromeos/drive/file_system/download_operation_unittest.cc b/chrome/browser/chromeos/drive/file_system/download_operation_unittest.cc
index 0f3936238a6eebefdff8d942b52747753af3f920..fae4a280444aec5f734048676f21ae473ef16fe6 100644
--- a/chrome/browser/chromeos/drive/file_system/download_operation_unittest.cc
+++ b/chrome/browser/chromeos/drive/file_system/download_operation_unittest.cc
@@ -27,7 +27,7 @@ class DownloadOperationTest : public OperationTestBase {
OperationTestBase::SetUp();
operation_.reset(new DownloadOperation(
- blocking_task_runner(), observer(), scheduler(), metadata(), cache(),
+ blocking_task_runner(), delegate(), scheduler(), metadata(), cache(),
temp_dir()));
}
@@ -63,8 +63,8 @@ TEST_F(DownloadOperationTest,
// The transfered file is cached and the change of "offline available"
// attribute is notified.
- EXPECT_EQ(1U, observer()->get_changed_files().size());
- EXPECT_EQ(1U, observer()->get_changed_files().count(file_in_root));
+ EXPECT_EQ(1U, delegate()->get_changed_files().size());
+ EXPECT_EQ(1U, delegate()->get_changed_files().count(file_in_root));
}
TEST_F(DownloadOperationTest,
@@ -137,9 +137,9 @@ TEST_F(DownloadOperationTest,
// The transfered file is cached and the change of "offline available"
// attribute is notified.
- EXPECT_EQ(2U, observer()->get_changed_files().size());
- EXPECT_TRUE(observer()->get_changed_files().count(file_in_root));
- EXPECT_TRUE(observer()->get_changed_files().count(cached_file));
+ EXPECT_EQ(2U, delegate()->get_changed_files().size());
+ EXPECT_TRUE(delegate()->get_changed_files().count(file_in_root));
+ EXPECT_TRUE(delegate()->get_changed_files().count(cached_file));
// The cache for the other file should be removed in order to free up space.
ResourceEntry cached_file_entry;
@@ -269,8 +269,8 @@ TEST_F(DownloadOperationTest, EnsureFileDownloadedByLocalId) {
// The transfered file is cached and the change of "offline available"
// attribute is notified.
- EXPECT_EQ(1U, observer()->get_changed_files().size());
- EXPECT_EQ(1U, observer()->get_changed_files().count(file_in_root));
+ EXPECT_EQ(1U, delegate()->get_changed_files().size());
+ EXPECT_EQ(1U, delegate()->get_changed_files().count(file_in_root));
}
TEST_F(DownloadOperationTest,
@@ -306,8 +306,8 @@ TEST_F(DownloadOperationTest,
// The transfered file is cached and the change of "offline available"
// attribute is notified.
- EXPECT_EQ(1U, observer()->get_changed_files().size());
- EXPECT_EQ(1U, observer()->get_changed_files().count(file_in_root));
+ EXPECT_EQ(1U, delegate()->get_changed_files().size());
+ EXPECT_EQ(1U, delegate()->get_changed_files().count(file_in_root));
}
{

Powered by Google App Engine
This is Rietveld 408576698