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

Unified Diff: chrome/browser/chromeos/drive/change_list_loader_unittest.cc

Issue 305913002: drive: Replace GetResourceListCallback in DriveServiceInterface with FileListCallback (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 6 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
« no previous file with comments | « no previous file | chrome/browser/chromeos/drive/fake_file_system.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/chromeos/drive/change_list_loader_unittest.cc
diff --git a/chrome/browser/chromeos/drive/change_list_loader_unittest.cc b/chrome/browser/chromeos/drive/change_list_loader_unittest.cc
index 924dfce44ce76bd3051584396fbb031e9ecb0e62..51ea89849948ee1040a3d5fdd564763fe156ad5d 100644
--- a/chrome/browser/chromeos/drive/change_list_loader_unittest.cc
+++ b/chrome/browser/chromeos/drive/change_list_loader_unittest.cc
@@ -164,7 +164,7 @@ TEST_F(ChangeListLoaderTest, Load) {
int64 changestamp = 0;
EXPECT_EQ(FILE_ERROR_OK, metadata_->GetLargestChangestamp(&changestamp));
EXPECT_LT(0, changestamp);
- EXPECT_EQ(1, drive_service_->resource_list_load_count());
+ EXPECT_EQ(1, drive_service_->file_list_load_count());
EXPECT_EQ(1, drive_service_->about_resource_load_count());
EXPECT_EQ(1, observer.initial_load_complete_count());
EXPECT_EQ(1, observer.load_from_server_complete_count());
@@ -200,8 +200,8 @@ TEST_F(ChangeListLoaderTest, Load_LocalMetadataAvailable) {
// Start loading. Because local metadata is available, the load results in
// returning FILE_ERROR_OK without fetching full list of resources.
- const int previous_resource_list_load_count =
- drive_service_->resource_list_load_count();
+ const int previous_file_list_load_count =
+ drive_service_->file_list_load_count();
TestChangeListLoaderObserver observer(change_list_loader_.get());
change_list_loader_->LoadIfNeeded(
@@ -209,8 +209,8 @@ TEST_F(ChangeListLoaderTest, Load_LocalMetadataAvailable) {
EXPECT_TRUE(change_list_loader_->IsRefreshing());
base::RunLoop().RunUntilIdle();
EXPECT_EQ(FILE_ERROR_OK, error);
- EXPECT_EQ(previous_resource_list_load_count,
- drive_service_->resource_list_load_count());
+ EXPECT_EQ(previous_file_list_load_count,
+ drive_service_->file_list_load_count());
EXPECT_EQ(1, observer.initial_load_complete_count());
// Update should be checked by Load().
@@ -242,7 +242,7 @@ TEST_F(ChangeListLoaderTest, CheckForUpdates) {
int64 changestamp = 0;
EXPECT_EQ(FILE_ERROR_OK, metadata_->GetLargestChangestamp(&changestamp));
EXPECT_EQ(0, changestamp);
- EXPECT_EQ(0, drive_service_->resource_list_load_count());
+ EXPECT_EQ(0, drive_service_->file_list_load_count());
EXPECT_EQ(0, drive_service_->about_resource_load_count());
// Start initial load.
@@ -262,7 +262,7 @@ TEST_F(ChangeListLoaderTest, CheckForUpdates) {
EXPECT_EQ(FILE_ERROR_OK, check_for_updates_error);
EXPECT_EQ(FILE_ERROR_OK, metadata_->GetLargestChangestamp(&changestamp));
EXPECT_LT(0, changestamp);
- EXPECT_EQ(1, drive_service_->resource_list_load_count());
+ EXPECT_EQ(1, drive_service_->file_list_load_count());
int64 previous_changestamp = 0;
EXPECT_EQ(FILE_ERROR_OK,
« no previous file with comments | « no previous file | chrome/browser/chromeos/drive/fake_file_system.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698