| OLD | NEW |
| 1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. | 1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. |
| 2 // Use of this source code is governed by a BSD-style license that can be | 2 // Use of this source code is governed by a BSD-style license that can be |
| 3 // found in the LICENSE file. | 3 // found in the LICENSE file. |
| 4 | 4 |
| 5 #include "chrome/browser/chromeos/drive/sync_client.h" | 5 #include "chrome/browser/chromeos/drive/sync_client.h" |
| 6 | 6 |
| 7 #include "base/files/file_path.h" | 7 #include "base/files/file_path.h" |
| 8 #include "base/files/file_util.h" | 8 #include "base/files/file_util.h" |
| 9 #include "base/files/scoped_temp_dir.h" | 9 #include "base/files/scoped_temp_dir.h" |
| 10 #include "base/memory/scoped_ptr.h" | 10 #include "base/memory/scoped_ptr.h" |
| 11 #include "base/prefs/testing_pref_service.h" | 11 #include "base/prefs/testing_pref_service.h" |
| 12 #include "base/run_loop.h" | 12 #include "base/run_loop.h" |
| 13 #include "base/single_thread_task_runner.h" |
| 13 #include "base/test/test_timeouts.h" | 14 #include "base/test/test_timeouts.h" |
| 15 #include "base/thread_task_runner_handle.h" |
| 14 #include "chrome/browser/chromeos/drive/change_list_loader.h" | 16 #include "chrome/browser/chromeos/drive/change_list_loader.h" |
| 15 #include "chrome/browser/chromeos/drive/drive.pb.h" | 17 #include "chrome/browser/chromeos/drive/drive.pb.h" |
| 16 #include "chrome/browser/chromeos/drive/fake_free_disk_space_getter.h" | 18 #include "chrome/browser/chromeos/drive/fake_free_disk_space_getter.h" |
| 17 #include "chrome/browser/chromeos/drive/file_cache.h" | 19 #include "chrome/browser/chromeos/drive/file_cache.h" |
| 18 #include "chrome/browser/chromeos/drive/file_change.h" | 20 #include "chrome/browser/chromeos/drive/file_change.h" |
| 19 #include "chrome/browser/chromeos/drive/file_system/move_operation.h" | 21 #include "chrome/browser/chromeos/drive/file_system/move_operation.h" |
| 20 #include "chrome/browser/chromeos/drive/file_system/operation_delegate.h" | 22 #include "chrome/browser/chromeos/drive/file_system/operation_delegate.h" |
| 21 #include "chrome/browser/chromeos/drive/file_system/remove_operation.h" | 23 #include "chrome/browser/chromeos/drive/file_system/remove_operation.h" |
| 22 #include "chrome/browser/chromeos/drive/file_system_util.h" | 24 #include "chrome/browser/chromeos/drive/file_system_util.h" |
| 23 #include "chrome/browser/chromeos/drive/job_scheduler.h" | 25 #include "chrome/browser/chromeos/drive/job_scheduler.h" |
| (...skipping 26 matching lines...) Expand all Loading... |
| 50 | 52 |
| 51 // FakeDriveService override: | 53 // FakeDriveService override: |
| 52 google_apis::CancelCallback DownloadFile( | 54 google_apis::CancelCallback DownloadFile( |
| 53 const base::FilePath& local_cache_path, | 55 const base::FilePath& local_cache_path, |
| 54 const std::string& resource_id, | 56 const std::string& resource_id, |
| 55 const google_apis::DownloadActionCallback& download_action_callback, | 57 const google_apis::DownloadActionCallback& download_action_callback, |
| 56 const google_apis::GetContentCallback& get_content_callback, | 58 const google_apis::GetContentCallback& get_content_callback, |
| 57 const google_apis::ProgressCallback& progress_callback) override { | 59 const google_apis::ProgressCallback& progress_callback) override { |
| 58 ++download_file_count_; | 60 ++download_file_count_; |
| 59 if (resource_id == resource_id_to_be_cancelled_) { | 61 if (resource_id == resource_id_to_be_cancelled_) { |
| 60 base::MessageLoopProxy::current()->PostTask( | 62 base::ThreadTaskRunnerHandle::Get()->PostTask( |
| 61 FROM_HERE, | 63 FROM_HERE, |
| 62 base::Bind(download_action_callback, | 64 base::Bind(download_action_callback, |
| 63 google_apis::DRIVE_CANCELLED, | 65 google_apis::DRIVE_CANCELLED, |
| 64 base::FilePath())); | 66 base::FilePath())); |
| 65 return google_apis::CancelCallback(); | 67 return google_apis::CancelCallback(); |
| 66 } | 68 } |
| 67 if (resource_id == resource_id_to_be_paused_) { | 69 if (resource_id == resource_id_to_be_paused_) { |
| 68 paused_action_ = base::Bind(download_action_callback, | 70 paused_action_ = base::Bind(download_action_callback, |
| 69 google_apis::DRIVE_OTHER_ERROR, | 71 google_apis::DRIVE_OTHER_ERROR, |
| 70 base::FilePath()); | 72 base::FilePath()); |
| (...skipping 35 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... |
| 106 pref_service_.reset(new TestingPrefServiceSimple); | 108 pref_service_.reset(new TestingPrefServiceSimple); |
| 107 test_util::RegisterDrivePrefs(pref_service_->registry()); | 109 test_util::RegisterDrivePrefs(pref_service_->registry()); |
| 108 | 110 |
| 109 fake_network_change_notifier_.reset( | 111 fake_network_change_notifier_.reset( |
| 110 new test_util::FakeNetworkChangeNotifier); | 112 new test_util::FakeNetworkChangeNotifier); |
| 111 | 113 |
| 112 logger_.reset(new EventLogger); | 114 logger_.reset(new EventLogger); |
| 113 | 115 |
| 114 drive_service_.reset(new SyncClientTestDriveService); | 116 drive_service_.reset(new SyncClientTestDriveService); |
| 115 | 117 |
| 116 scheduler_.reset(new JobScheduler(pref_service_.get(), | 118 scheduler_.reset(new JobScheduler( |
| 117 logger_.get(), | 119 pref_service_.get(), |
| 118 drive_service_.get(), | 120 logger_.get(), |
| 119 base::MessageLoopProxy::current().get())); | 121 drive_service_.get(), |
| 122 base::ThreadTaskRunnerHandle::Get().get())); |
| 120 | 123 |
| 121 metadata_storage_.reset(new ResourceMetadataStorage( | 124 metadata_storage_.reset(new ResourceMetadataStorage( |
| 122 temp_dir_.path(), base::MessageLoopProxy::current().get())); | 125 temp_dir_.path(), base::ThreadTaskRunnerHandle::Get().get())); |
| 123 ASSERT_TRUE(metadata_storage_->Initialize()); | 126 ASSERT_TRUE(metadata_storage_->Initialize()); |
| 124 | 127 |
| 125 cache_.reset(new FileCache(metadata_storage_.get(), | 128 cache_.reset(new FileCache(metadata_storage_.get(), |
| 126 temp_dir_.path(), | 129 temp_dir_.path(), |
| 127 base::MessageLoopProxy::current().get(), | 130 base::ThreadTaskRunnerHandle::Get().get(), |
| 128 NULL /* free_disk_space_getter */)); | 131 NULL /* free_disk_space_getter */)); |
| 129 ASSERT_TRUE(cache_->Initialize()); | 132 ASSERT_TRUE(cache_->Initialize()); |
| 130 | 133 |
| 131 metadata_.reset(new internal::ResourceMetadata( | 134 metadata_.reset(new internal::ResourceMetadata( |
| 132 metadata_storage_.get(), cache_.get(), | 135 metadata_storage_.get(), cache_.get(), |
| 133 base::MessageLoopProxy::current())); | 136 base::ThreadTaskRunnerHandle::Get())); |
| 134 ASSERT_EQ(FILE_ERROR_OK, metadata_->Initialize()); | 137 ASSERT_EQ(FILE_ERROR_OK, metadata_->Initialize()); |
| 135 | 138 |
| 136 about_resource_loader_.reset(new AboutResourceLoader(scheduler_.get())); | 139 about_resource_loader_.reset(new AboutResourceLoader(scheduler_.get())); |
| 137 loader_controller_.reset(new LoaderController); | 140 loader_controller_.reset(new LoaderController); |
| 138 change_list_loader_.reset(new ChangeListLoader( | 141 change_list_loader_.reset(new ChangeListLoader( |
| 139 logger_.get(), | 142 logger_.get(), |
| 140 base::MessageLoopProxy::current().get(), | 143 base::ThreadTaskRunnerHandle::Get().get(), |
| 141 metadata_.get(), | 144 metadata_.get(), |
| 142 scheduler_.get(), | 145 scheduler_.get(), |
| 143 about_resource_loader_.get(), | 146 about_resource_loader_.get(), |
| 144 loader_controller_.get())); | 147 loader_controller_.get())); |
| 145 ASSERT_NO_FATAL_FAILURE(SetUpTestData()); | 148 ASSERT_NO_FATAL_FAILURE(SetUpTestData()); |
| 146 | 149 |
| 147 sync_client_.reset(new SyncClient(base::MessageLoopProxy::current().get(), | 150 sync_client_.reset(new SyncClient(base::ThreadTaskRunnerHandle::Get().get(), |
| 148 &delegate_, | 151 &delegate_, |
| 149 scheduler_.get(), | 152 scheduler_.get(), |
| 150 metadata_.get(), | 153 metadata_.get(), |
| 151 cache_.get(), | 154 cache_.get(), |
| 152 loader_controller_.get(), | 155 loader_controller_.get(), |
| 153 temp_dir_.path())); | 156 temp_dir_.path())); |
| 154 | 157 |
| 155 // Disable delaying so that DoSyncLoop() starts immediately. | 158 // Disable delaying so that DoSyncLoop() starts immediately. |
| 156 sync_client_->set_delay_for_testing(base::TimeDelta::FromSeconds(0)); | 159 sync_client_->set_delay_for_testing(base::TimeDelta::FromSeconds(0)); |
| 157 } | 160 } |
| (...skipping 52 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... |
| 210 EXPECT_EQ(FILE_ERROR_OK, cache_->Pin(GetLocalId("fetched"))); | 213 EXPECT_EQ(FILE_ERROR_OK, cache_->Pin(GetLocalId("fetched"))); |
| 211 | 214 |
| 212 // Prepare a pinned-and-fetched-and-dirty file. | 215 // Prepare a pinned-and-fetched-and-dirty file. |
| 213 EXPECT_EQ(FILE_ERROR_OK, | 216 EXPECT_EQ(FILE_ERROR_OK, |
| 214 cache_->Store(GetLocalId("dirty"), std::string(), | 217 cache_->Store(GetLocalId("dirty"), std::string(), |
| 215 temp_file, FileCache::FILE_OPERATION_COPY)); | 218 temp_file, FileCache::FILE_OPERATION_COPY)); |
| 216 EXPECT_EQ(FILE_ERROR_OK, cache_->Pin(GetLocalId("dirty"))); | 219 EXPECT_EQ(FILE_ERROR_OK, cache_->Pin(GetLocalId("dirty"))); |
| 217 | 220 |
| 218 // Prepare a removed file. | 221 // Prepare a removed file. |
| 219 file_system::RemoveOperation remove_operation( | 222 file_system::RemoveOperation remove_operation( |
| 220 base::MessageLoopProxy::current().get(), &delegate_, metadata_.get(), | 223 base::ThreadTaskRunnerHandle::Get().get(), &delegate_, metadata_.get(), |
| 221 cache_.get()); | 224 cache_.get()); |
| 222 remove_operation.Remove( | 225 remove_operation.Remove( |
| 223 util::GetDriveMyDriveRootPath().AppendASCII("removed"), | 226 util::GetDriveMyDriveRootPath().AppendASCII("removed"), |
| 224 false, // is_recursive | 227 false, // is_recursive |
| 225 google_apis::test_util::CreateCopyResultCallback(&error)); | 228 google_apis::test_util::CreateCopyResultCallback(&error)); |
| 226 base::RunLoop().RunUntilIdle(); | 229 base::RunLoop().RunUntilIdle(); |
| 227 EXPECT_EQ(FILE_ERROR_OK, error); | 230 EXPECT_EQ(FILE_ERROR_OK, error); |
| 228 | 231 |
| 229 // Prepare a moved file. | 232 // Prepare a moved file. |
| 230 file_system::MoveOperation move_operation( | 233 file_system::MoveOperation move_operation( |
| 231 base::MessageLoopProxy::current().get(), &delegate_, metadata_.get()); | 234 base::ThreadTaskRunnerHandle::Get().get(), &delegate_, metadata_.get()); |
| 232 move_operation.Move( | 235 move_operation.Move( |
| 233 util::GetDriveMyDriveRootPath().AppendASCII("moved"), | 236 util::GetDriveMyDriveRootPath().AppendASCII("moved"), |
| 234 util::GetDriveMyDriveRootPath().AppendASCII("moved_new_title"), | 237 util::GetDriveMyDriveRootPath().AppendASCII("moved_new_title"), |
| 235 google_apis::test_util::CreateCopyResultCallback(&error)); | 238 google_apis::test_util::CreateCopyResultCallback(&error)); |
| 236 base::RunLoop().RunUntilIdle(); | 239 base::RunLoop().RunUntilIdle(); |
| 237 EXPECT_EQ(FILE_ERROR_OK, error); | 240 EXPECT_EQ(FILE_ERROR_OK, error); |
| 238 } | 241 } |
| 239 | 242 |
| 240 protected: | 243 protected: |
| 241 std::string GetLocalId(const std::string& title) { | 244 std::string GetLocalId(const std::string& title) { |
| (...skipping 138 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... |
| 380 EXPECT_EQ(kLocalContent, content); | 383 EXPECT_EQ(kLocalContent, content); |
| 381 } | 384 } |
| 382 | 385 |
| 383 TEST_F(SyncClientTest, RetryOnDisconnection) { | 386 TEST_F(SyncClientTest, RetryOnDisconnection) { |
| 384 // Let the service go down. | 387 // Let the service go down. |
| 385 drive_service_->set_offline(true); | 388 drive_service_->set_offline(true); |
| 386 // Change the network connection state after some delay, to test that | 389 // Change the network connection state after some delay, to test that |
| 387 // FILE_ERROR_NO_CONNECTION is handled by SyncClient correctly. | 390 // FILE_ERROR_NO_CONNECTION is handled by SyncClient correctly. |
| 388 // Without this delay, JobScheduler will keep the jobs unrun and SyncClient | 391 // Without this delay, JobScheduler will keep the jobs unrun and SyncClient |
| 389 // will receive no error. | 392 // will receive no error. |
| 390 base::MessageLoopProxy::current()->PostDelayedTask( | 393 base::ThreadTaskRunnerHandle::Get()->PostDelayedTask( |
| 391 FROM_HERE, | 394 FROM_HERE, |
| 392 base::Bind(&test_util::FakeNetworkChangeNotifier::SetConnectionType, | 395 base::Bind(&test_util::FakeNetworkChangeNotifier::SetConnectionType, |
| 393 base::Unretained(fake_network_change_notifier_.get()), | 396 base::Unretained(fake_network_change_notifier_.get()), |
| 394 net::NetworkChangeNotifier::CONNECTION_NONE), | 397 net::NetworkChangeNotifier::CONNECTION_NONE), |
| 395 TestTimeouts::tiny_timeout()); | 398 TestTimeouts::tiny_timeout()); |
| 396 | 399 |
| 397 // Try fetch and upload. | 400 // Try fetch and upload. |
| 398 sync_client_->AddFetchTask(GetLocalId("foo")); | 401 sync_client_->AddFetchTask(GetLocalId("foo")); |
| 399 sync_client_->AddUpdateTask(ClientContext(USER_INITIATED), | 402 sync_client_->AddUpdateTask(ClientContext(USER_INITIATED), |
| 400 GetLocalId("dirty")); | 403 GetLocalId("dirty")); |
| (...skipping 110 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... |
| 511 local_id, google_apis::test_util::CreateCopyResultCallback(&error))); | 514 local_id, google_apis::test_util::CreateCopyResultCallback(&error))); |
| 512 | 515 |
| 513 base::RunLoop().RunUntilIdle(); | 516 base::RunLoop().RunUntilIdle(); |
| 514 | 517 |
| 515 // The callback is called. | 518 // The callback is called. |
| 516 EXPECT_EQ(FILE_ERROR_OK, error); | 519 EXPECT_EQ(FILE_ERROR_OK, error); |
| 517 } | 520 } |
| 518 | 521 |
| 519 } // namespace internal | 522 } // namespace internal |
| 520 } // namespace drive | 523 } // namespace drive |
| OLD | NEW |