Commit 229290b5 authored by pranay.kumar's avatar pranay.kumar Committed by Commit bot

Replace MessageLoopProxy usage with ThreadTaskRunnerHandle in chrome/browser/chromeos/drive module.

MessageLoopProxy is deprecated.
This basically does a search and replace of:

MessageLoopProxy::current() -> ThreadTaskRunnerHandle::Get().

MessageLoopProxy -> SingleThreadTaskRunner

BUG=391045

Review URL: https://codereview.chromium.org/1124813003

Cr-Commit-Position: refs/heads/master@{#329807}
parent fd9ae3c1
...@@ -11,6 +11,7 @@ ...@@ -11,6 +11,7 @@
#include "base/metrics/histogram.h" #include "base/metrics/histogram.h"
#include "base/strings/string_number_conversions.h" #include "base/strings/string_number_conversions.h"
#include "base/synchronization/cancellation_flag.h" #include "base/synchronization/cancellation_flag.h"
#include "base/thread_task_runner_handle.h"
#include "base/time/time.h" #include "base/time/time.h"
#include "chrome/browser/chromeos/drive/change_list_loader_observer.h" #include "chrome/browser/chromeos/drive/change_list_loader_observer.h"
#include "chrome/browser/chromeos/drive/change_list_processor.h" #include "chrome/browser/chromeos/drive/change_list_processor.h"
...@@ -226,7 +227,7 @@ void AboutResourceLoader::GetAboutResource( ...@@ -226,7 +227,7 @@ void AboutResourceLoader::GetAboutResource(
} }
if (cached_about_resource_) { if (cached_about_resource_) {
base::MessageLoopProxy::current()->PostTask( base::ThreadTaskRunnerHandle::Get()->PostTask(
FROM_HERE, FROM_HERE,
base::Bind( base::Bind(
callback, callback,
...@@ -461,7 +462,7 @@ void ChangeListLoader::OnChangeListLoadComplete(FileError error) { ...@@ -461,7 +462,7 @@ void ChangeListLoader::OnChangeListLoadComplete(FileError error) {
} }
for (size_t i = 0; i < pending_load_callback_.size(); ++i) { for (size_t i = 0; i < pending_load_callback_.size(); ++i) {
base::MessageLoopProxy::current()->PostTask( base::ThreadTaskRunnerHandle::Get()->PostTask(
FROM_HERE, FROM_HERE,
base::Bind(pending_load_callback_[i], error)); base::Bind(pending_load_callback_[i], error));
} }
......
...@@ -9,6 +9,8 @@ ...@@ -9,6 +9,8 @@
#include "base/memory/scoped_ptr.h" #include "base/memory/scoped_ptr.h"
#include "base/prefs/testing_pref_service.h" #include "base/prefs/testing_pref_service.h"
#include "base/run_loop.h" #include "base/run_loop.h"
#include "base/single_thread_task_runner.h"
#include "base/thread_task_runner_handle.h"
#include "chrome/browser/chromeos/drive/change_list_loader_observer.h" #include "chrome/browser/chromeos/drive/change_list_loader_observer.h"
#include "chrome/browser/chromeos/drive/file_cache.h" #include "chrome/browser/chromeos/drive/file_cache.h"
#include "chrome/browser/chromeos/drive/file_change.h" #include "chrome/browser/chromeos/drive/file_change.h"
...@@ -78,30 +80,31 @@ class ChangeListLoaderTest : public testing::Test { ...@@ -78,30 +80,31 @@ class ChangeListLoaderTest : public testing::Test {
drive_service_.reset(new FakeDriveService); drive_service_.reset(new FakeDriveService);
ASSERT_TRUE(test_util::SetUpTestEntries(drive_service_.get())); ASSERT_TRUE(test_util::SetUpTestEntries(drive_service_.get()));
scheduler_.reset(new JobScheduler(pref_service_.get(), scheduler_.reset(new JobScheduler(
logger_.get(), pref_service_.get(),
drive_service_.get(), logger_.get(),
base::MessageLoopProxy::current().get())); drive_service_.get(),
base::ThreadTaskRunnerHandle::Get().get()));
metadata_storage_.reset(new ResourceMetadataStorage( metadata_storage_.reset(new ResourceMetadataStorage(
temp_dir_.path(), base::MessageLoopProxy::current().get())); temp_dir_.path(), base::ThreadTaskRunnerHandle::Get().get()));
ASSERT_TRUE(metadata_storage_->Initialize()); ASSERT_TRUE(metadata_storage_->Initialize());
cache_.reset(new FileCache(metadata_storage_.get(), cache_.reset(new FileCache(metadata_storage_.get(),
temp_dir_.path(), temp_dir_.path(),
base::MessageLoopProxy::current().get(), base::ThreadTaskRunnerHandle::Get().get(),
NULL /* free_disk_space_getter */)); NULL /* free_disk_space_getter */));
ASSERT_TRUE(cache_->Initialize()); ASSERT_TRUE(cache_->Initialize());
metadata_.reset(new ResourceMetadata( metadata_.reset(new ResourceMetadata(
metadata_storage_.get(), cache_.get(), metadata_storage_.get(), cache_.get(),
base::MessageLoopProxy::current().get())); base::ThreadTaskRunnerHandle::Get().get()));
ASSERT_EQ(FILE_ERROR_OK, metadata_->Initialize()); ASSERT_EQ(FILE_ERROR_OK, metadata_->Initialize());
about_resource_loader_.reset(new AboutResourceLoader(scheduler_.get())); about_resource_loader_.reset(new AboutResourceLoader(scheduler_.get()));
loader_controller_.reset(new LoaderController); loader_controller_.reset(new LoaderController);
change_list_loader_.reset( change_list_loader_.reset(
new ChangeListLoader(logger_.get(), new ChangeListLoader(logger_.get(),
base::MessageLoopProxy::current().get(), base::ThreadTaskRunnerHandle::Get().get(),
metadata_.get(), metadata_.get(),
scheduler_.get(), scheduler_.get(),
about_resource_loader_.get(), about_resource_loader_.get(),
...@@ -247,7 +250,7 @@ TEST_F(ChangeListLoaderTest, Load_LocalMetadataAvailable) { ...@@ -247,7 +250,7 @@ TEST_F(ChangeListLoaderTest, Load_LocalMetadataAvailable) {
about_resource_loader_.reset(new AboutResourceLoader(scheduler_.get())); about_resource_loader_.reset(new AboutResourceLoader(scheduler_.get()));
change_list_loader_.reset( change_list_loader_.reset(
new ChangeListLoader(logger_.get(), new ChangeListLoader(logger_.get(),
base::MessageLoopProxy::current().get(), base::ThreadTaskRunnerHandle::Get().get(),
metadata_.get(), metadata_.get(),
scheduler_.get(), scheduler_.get(),
about_resource_loader_.get(), about_resource_loader_.get(),
......
...@@ -5,7 +5,8 @@ ...@@ -5,7 +5,8 @@
#include "chrome/browser/chromeos/drive/change_list_processor.h" #include "chrome/browser/chromeos/drive/change_list_processor.h"
#include "base/files/scoped_temp_dir.h" #include "base/files/scoped_temp_dir.h"
#include "base/message_loop/message_loop_proxy.h" #include "base/single_thread_task_runner.h"
#include "base/thread_task_runner_handle.h"
#include "base/values.h" #include "base/values.h"
#include "chrome/browser/chromeos/drive/drive.pb.h" #include "chrome/browser/chromeos/drive/drive.pb.h"
#include "chrome/browser/chromeos/drive/fake_free_disk_space_getter.h" #include "chrome/browser/chromeos/drive/fake_free_disk_space_getter.h"
...@@ -102,19 +103,19 @@ class ChangeListProcessorTest : public testing::Test { ...@@ -102,19 +103,19 @@ class ChangeListProcessorTest : public testing::Test {
ASSERT_TRUE(temp_dir_.CreateUniqueTempDir()); ASSERT_TRUE(temp_dir_.CreateUniqueTempDir());
metadata_storage_.reset(new ResourceMetadataStorage( metadata_storage_.reset(new ResourceMetadataStorage(
temp_dir_.path(), base::MessageLoopProxy::current().get())); temp_dir_.path(), base::ThreadTaskRunnerHandle::Get().get()));
ASSERT_TRUE(metadata_storage_->Initialize()); ASSERT_TRUE(metadata_storage_->Initialize());
fake_free_disk_space_getter_.reset(new FakeFreeDiskSpaceGetter); fake_free_disk_space_getter_.reset(new FakeFreeDiskSpaceGetter);
cache_.reset(new FileCache(metadata_storage_.get(), cache_.reset(new FileCache(metadata_storage_.get(),
temp_dir_.path(), temp_dir_.path(),
base::MessageLoopProxy::current().get(), base::ThreadTaskRunnerHandle::Get().get(),
fake_free_disk_space_getter_.get())); fake_free_disk_space_getter_.get()));
ASSERT_TRUE(cache_->Initialize()); ASSERT_TRUE(cache_->Initialize());
metadata_.reset(new internal::ResourceMetadata( metadata_.reset(new internal::ResourceMetadata(
metadata_storage_.get(), cache_.get(), metadata_storage_.get(), cache_.get(),
base::MessageLoopProxy::current())); base::ThreadTaskRunnerHandle::Get()));
ASSERT_EQ(FILE_ERROR_OK, metadata_->Initialize()); ASSERT_EQ(FILE_ERROR_OK, metadata_->Initialize());
} }
......
...@@ -9,6 +9,8 @@ ...@@ -9,6 +9,8 @@
#include "base/memory/scoped_ptr.h" #include "base/memory/scoped_ptr.h"
#include "base/prefs/testing_pref_service.h" #include "base/prefs/testing_pref_service.h"
#include "base/run_loop.h" #include "base/run_loop.h"
#include "base/single_thread_task_runner.h"
#include "base/thread_task_runner_handle.h"
#include "chrome/browser/chromeos/drive/change_list_loader.h" #include "chrome/browser/chromeos/drive/change_list_loader.h"
#include "chrome/browser/chromeos/drive/change_list_loader_observer.h" #include "chrome/browser/chromeos/drive/change_list_loader_observer.h"
#include "chrome/browser/chromeos/drive/file_cache.h" #include "chrome/browser/chromeos/drive/file_cache.h"
...@@ -75,30 +77,31 @@ class DirectoryLoaderTest : public testing::Test { ...@@ -75,30 +77,31 @@ class DirectoryLoaderTest : public testing::Test {
drive_service_.reset(new FakeDriveService); drive_service_.reset(new FakeDriveService);
ASSERT_TRUE(test_util::SetUpTestEntries(drive_service_.get())); ASSERT_TRUE(test_util::SetUpTestEntries(drive_service_.get()));
scheduler_.reset(new JobScheduler(pref_service_.get(), scheduler_.reset(new JobScheduler(
logger_.get(), pref_service_.get(),
drive_service_.get(), logger_.get(),
base::MessageLoopProxy::current().get())); drive_service_.get(),
base::ThreadTaskRunnerHandle::Get().get()));
metadata_storage_.reset(new ResourceMetadataStorage( metadata_storage_.reset(new ResourceMetadataStorage(
temp_dir_.path(), base::MessageLoopProxy::current().get())); temp_dir_.path(), base::ThreadTaskRunnerHandle::Get().get()));
ASSERT_TRUE(metadata_storage_->Initialize()); ASSERT_TRUE(metadata_storage_->Initialize());
cache_.reset(new FileCache(metadata_storage_.get(), cache_.reset(new FileCache(metadata_storage_.get(),
temp_dir_.path(), temp_dir_.path(),
base::MessageLoopProxy::current().get(), base::ThreadTaskRunnerHandle::Get().get(),
NULL /* free_disk_space_getter */)); NULL /* free_disk_space_getter */));
ASSERT_TRUE(cache_->Initialize()); ASSERT_TRUE(cache_->Initialize());
metadata_.reset(new ResourceMetadata( metadata_.reset(new ResourceMetadata(
metadata_storage_.get(), cache_.get(), metadata_storage_.get(), cache_.get(),
base::MessageLoopProxy::current().get())); base::ThreadTaskRunnerHandle::Get().get()));
ASSERT_EQ(FILE_ERROR_OK, metadata_->Initialize()); ASSERT_EQ(FILE_ERROR_OK, metadata_->Initialize());
about_resource_loader_.reset(new AboutResourceLoader(scheduler_.get())); about_resource_loader_.reset(new AboutResourceLoader(scheduler_.get()));
loader_controller_.reset(new LoaderController); loader_controller_.reset(new LoaderController);
directory_loader_.reset( directory_loader_.reset(
new DirectoryLoader(logger_.get(), new DirectoryLoader(logger_.get(),
base::MessageLoopProxy::current().get(), base::ThreadTaskRunnerHandle::Get().get(),
metadata_.get(), metadata_.get(),
scheduler_.get(), scheduler_.get(),
about_resource_loader_.get(), about_resource_loader_.get(),
......
...@@ -13,6 +13,8 @@ ...@@ -13,6 +13,8 @@
#include "base/files/scoped_temp_dir.h" #include "base/files/scoped_temp_dir.h"
#include "base/md5.h" #include "base/md5.h"
#include "base/path_service.h" #include "base/path_service.h"
#include "base/single_thread_task_runner.h"
#include "base/thread_task_runner_handle.h"
#include "chrome/browser/chromeos/drive/drive.pb.h" #include "chrome/browser/chromeos/drive/drive.pb.h"
#include "chrome/browser/chromeos/drive/fake_free_disk_space_getter.h" #include "chrome/browser/chromeos/drive/fake_free_disk_space_getter.h"
#include "chrome/browser/chromeos/drive/file_system_util.h" #include "chrome/browser/chromeos/drive/file_system_util.h"
...@@ -45,13 +47,13 @@ class FileCacheTest : public testing::Test { ...@@ -45,13 +47,13 @@ class FileCacheTest : public testing::Test {
metadata_storage_.reset(new ResourceMetadataStorage( metadata_storage_.reset(new ResourceMetadataStorage(
metadata_dir, metadata_dir,
base::MessageLoopProxy::current().get())); base::ThreadTaskRunnerHandle::Get().get()));
ASSERT_TRUE(metadata_storage_->Initialize()); ASSERT_TRUE(metadata_storage_->Initialize());
cache_.reset(new FileCache( cache_.reset(new FileCache(
metadata_storage_.get(), metadata_storage_.get(),
cache_files_dir_, cache_files_dir_,
base::MessageLoopProxy::current().get(), base::ThreadTaskRunnerHandle::Get().get(),
fake_free_disk_space_getter_.get())); fake_free_disk_space_getter_.get()));
ASSERT_TRUE(cache_->Initialize()); ASSERT_TRUE(cache_->Initialize());
} }
......
...@@ -9,9 +9,9 @@ ...@@ -9,9 +9,9 @@
#include "base/files/file.h" #include "base/files/file.h"
#include "base/files/file_path.h" #include "base/files/file_path.h"
#include "base/logging.h" #include "base/logging.h"
#include "base/message_loop/message_loop_proxy.h"
#include "base/sequenced_task_runner.h" #include "base/sequenced_task_runner.h"
#include "base/task_runner_util.h" #include "base/task_runner_util.h"
#include "base/thread_task_runner_handle.h"
#include "chrome/browser/chromeos/drive/drive.pb.h" #include "chrome/browser/chromeos/drive/drive.pb.h"
#include "chrome/browser/chromeos/drive/file_cache.h" #include "chrome/browser/chromeos/drive/file_cache.h"
#include "chrome/browser/chromeos/drive/file_errors.h" #include "chrome/browser/chromeos/drive/file_errors.h"
...@@ -84,7 +84,7 @@ void TruncateOperation::Truncate(const base::FilePath& file_path, ...@@ -84,7 +84,7 @@ void TruncateOperation::Truncate(const base::FilePath& file_path,
DCHECK(!callback.is_null()); DCHECK(!callback.is_null());
if (length < 0) { if (length < 0) {
base::MessageLoopProxy::current()->PostTask( base::ThreadTaskRunnerHandle::Get()->PostTask(
FROM_HERE, FROM_HERE,
base::Bind(callback, FILE_ERROR_INVALID_OPERATION)); base::Bind(callback, FILE_ERROR_INVALID_OPERATION));
return; return;
......
...@@ -12,9 +12,10 @@ ...@@ -12,9 +12,10 @@
#include "base/files/file_util.h" #include "base/files/file_util.h"
#include "base/files/scoped_temp_dir.h" #include "base/files/scoped_temp_dir.h"
#include "base/memory/scoped_ptr.h" #include "base/memory/scoped_ptr.h"
#include "base/message_loop/message_loop_proxy.h"
#include "base/prefs/testing_pref_service.h" #include "base/prefs/testing_pref_service.h"
#include "base/run_loop.h" #include "base/run_loop.h"
#include "base/single_thread_task_runner.h"
#include "base/thread_task_runner_handle.h"
#include "chrome/browser/chromeos/drive/change_list_loader.h" #include "chrome/browser/chromeos/drive/change_list_loader.h"
#include "chrome/browser/chromeos/drive/drive.pb.h" #include "chrome/browser/chromeos/drive/drive.pb.h"
#include "chrome/browser/chromeos/drive/fake_free_disk_space_getter.h" #include "chrome/browser/chromeos/drive/fake_free_disk_space_getter.h"
...@@ -106,10 +107,11 @@ class FileSystemTest : public testing::Test { ...@@ -106,10 +107,11 @@ class FileSystemTest : public testing::Test {
fake_free_disk_space_getter_.reset(new FakeFreeDiskSpaceGetter); fake_free_disk_space_getter_.reset(new FakeFreeDiskSpaceGetter);
scheduler_.reset(new JobScheduler(pref_service_.get(), scheduler_.reset(new JobScheduler(
logger_.get(), pref_service_.get(),
fake_drive_service_.get(), logger_.get(),
base::MessageLoopProxy::current().get())); fake_drive_service_.get(),
base::ThreadTaskRunnerHandle::Get().get()));
mock_directory_observer_.reset(new MockDirectoryChangeObserver); mock_directory_observer_.reset(new MockDirectoryChangeObserver);
...@@ -120,7 +122,7 @@ class FileSystemTest : public testing::Test { ...@@ -120,7 +122,7 @@ class FileSystemTest : public testing::Test {
const base::FilePath metadata_dir = temp_dir_.path().AppendASCII("meta"); const base::FilePath metadata_dir = temp_dir_.path().AppendASCII("meta");
ASSERT_TRUE(base::CreateDirectory(metadata_dir)); ASSERT_TRUE(base::CreateDirectory(metadata_dir));
metadata_storage_.reset(new internal::ResourceMetadataStorage( metadata_storage_.reset(new internal::ResourceMetadataStorage(
metadata_dir, base::MessageLoopProxy::current().get())); metadata_dir, base::ThreadTaskRunnerHandle::Get().get()));
ASSERT_TRUE(metadata_storage_->Initialize()); ASSERT_TRUE(metadata_storage_->Initialize());
const base::FilePath cache_dir = temp_dir_.path().AppendASCII("files"); const base::FilePath cache_dir = temp_dir_.path().AppendASCII("files");
...@@ -128,13 +130,13 @@ class FileSystemTest : public testing::Test { ...@@ -128,13 +130,13 @@ class FileSystemTest : public testing::Test {
cache_.reset(new internal::FileCache( cache_.reset(new internal::FileCache(
metadata_storage_.get(), metadata_storage_.get(),
cache_dir, cache_dir,
base::MessageLoopProxy::current().get(), base::ThreadTaskRunnerHandle::Get().get(),
fake_free_disk_space_getter_.get())); fake_free_disk_space_getter_.get()));
ASSERT_TRUE(cache_->Initialize()); ASSERT_TRUE(cache_->Initialize());
resource_metadata_.reset(new internal::ResourceMetadata( resource_metadata_.reset(new internal::ResourceMetadata(
metadata_storage_.get(), cache_.get(), metadata_storage_.get(), cache_.get(),
base::MessageLoopProxy::current())); base::ThreadTaskRunnerHandle::Get()));
ASSERT_EQ(FILE_ERROR_OK, resource_metadata_->Initialize()); ASSERT_EQ(FILE_ERROR_OK, resource_metadata_->Initialize());
const base::FilePath temp_file_dir = temp_dir_.path().AppendASCII("tmp"); const base::FilePath temp_file_dir = temp_dir_.path().AppendASCII("tmp");
...@@ -145,7 +147,7 @@ class FileSystemTest : public testing::Test { ...@@ -145,7 +147,7 @@ class FileSystemTest : public testing::Test {
cache_.get(), cache_.get(),
scheduler_.get(), scheduler_.get(),
resource_metadata_.get(), resource_metadata_.get(),
base::MessageLoopProxy::current().get(), base::ThreadTaskRunnerHandle::Get().get(),
temp_file_dir)); temp_file_dir));
file_system_->AddObserver(mock_directory_observer_.get()); file_system_->AddObserver(mock_directory_observer_.get());
...@@ -224,19 +226,19 @@ class FileSystemTest : public testing::Test { ...@@ -224,19 +226,19 @@ class FileSystemTest : public testing::Test {
scoped_ptr<internal::ResourceMetadataStorage, scoped_ptr<internal::ResourceMetadataStorage,
test_util::DestroyHelperForTests> metadata_storage( test_util::DestroyHelperForTests> metadata_storage(
new internal::ResourceMetadataStorage( new internal::ResourceMetadataStorage(
metadata_dir, base::MessageLoopProxy::current().get())); metadata_dir, base::ThreadTaskRunnerHandle::Get().get()));
const base::FilePath cache_dir = temp_dir_.path().AppendASCII("files"); const base::FilePath cache_dir = temp_dir_.path().AppendASCII("files");
scoped_ptr<internal::FileCache, test_util::DestroyHelperForTests> cache( scoped_ptr<internal::FileCache, test_util::DestroyHelperForTests> cache(
new internal::FileCache(metadata_storage.get(), new internal::FileCache(metadata_storage.get(),
cache_dir, cache_dir,
base::MessageLoopProxy::current().get(), base::ThreadTaskRunnerHandle::Get().get(),
fake_free_disk_space_getter_.get())); fake_free_disk_space_getter_.get()));
scoped_ptr<internal::ResourceMetadata, test_util::DestroyHelperForTests> scoped_ptr<internal::ResourceMetadata, test_util::DestroyHelperForTests>
resource_metadata(new internal::ResourceMetadata( resource_metadata(new internal::ResourceMetadata(
metadata_storage_.get(), cache.get(), metadata_storage_.get(), cache.get(),
base::MessageLoopProxy::current())); base::ThreadTaskRunnerHandle::Get()));
ASSERT_EQ(FILE_ERROR_OK, resource_metadata->Initialize()); ASSERT_EQ(FILE_ERROR_OK, resource_metadata->Initialize());
......
...@@ -16,11 +16,11 @@ ...@@ -16,11 +16,11 @@
#include "base/json/json_file_value_serializer.h" #include "base/json/json_file_value_serializer.h"
#include "base/logging.h" #include "base/logging.h"
#include "base/memory/scoped_ptr.h" #include "base/memory/scoped_ptr.h"
#include "base/message_loop/message_loop_proxy.h"
#include "base/prefs/pref_service.h" #include "base/prefs/pref_service.h"
#include "base/strings/string_number_conversions.h" #include "base/strings/string_number_conversions.h"
#include "base/strings/string_util.h" #include "base/strings/string_util.h"
#include "base/strings/stringprintf.h" #include "base/strings/stringprintf.h"
#include "base/thread_task_runner_handle.h"
#include "base/threading/sequenced_worker_pool.h" #include "base/threading/sequenced_worker_pool.h"
#include "chrome/browser/browser_process.h" #include "chrome/browser/browser_process.h"
#include "chrome/browser/chromeos/drive/drive.pb.h" #include "chrome/browser/chromeos/drive/drive.pb.h"
...@@ -298,7 +298,7 @@ void EnsureDirectoryExists(Profile* profile, ...@@ -298,7 +298,7 @@ void EnsureDirectoryExists(Profile* profile,
true /* is_recursive */, true /* is_recursive */,
callback); callback);
} else { } else {
base::MessageLoopProxy::current()->PostTask( base::ThreadTaskRunnerHandle::Get()->PostTask(
FROM_HERE, base::Bind(callback, FILE_ERROR_OK)); FROM_HERE, base::Bind(callback, FILE_ERROR_OK));
} }
} }
......
...@@ -9,7 +9,9 @@ ...@@ -9,7 +9,9 @@
#include "base/files/file_path.h" #include "base/files/file_path.h"
#include "base/files/file_util.h" #include "base/files/file_util.h"
#include "base/files/scoped_temp_dir.h" #include "base/files/scoped_temp_dir.h"
#include "base/single_thread_task_runner.h"
#include "base/strings/utf_string_conversions.h" #include "base/strings/utf_string_conversions.h"
#include "base/thread_task_runner_handle.h"
#include "chrome/browser/chromeos/profiles/profile_helper.h" #include "chrome/browser/chromeos/profiles/profile_helper.h"
#include "chrome/test/base/testing_browser_process.h" #include "chrome/test/base/testing_browser_process.h"
#include "chrome/test/base/testing_profile.h" #include "chrome/test/base/testing_profile.h"
...@@ -131,8 +133,8 @@ TEST_F(FileSystemUtilTest, ExtractDrivePathFromFileSystemUrl) { ...@@ -131,8 +133,8 @@ TEST_F(FileSystemUtilTest, ExtractDrivePathFromFileSystemUrl) {
storage::ExternalMountPoints::CreateRefCounted(); storage::ExternalMountPoints::CreateRefCounted();
scoped_refptr<storage::FileSystemContext> context( scoped_refptr<storage::FileSystemContext> context(
new storage::FileSystemContext( new storage::FileSystemContext(
base::MessageLoopProxy::current().get(), base::ThreadTaskRunnerHandle::Get().get(),
base::MessageLoopProxy::current().get(), base::ThreadTaskRunnerHandle::Get().get(),
mount_points.get(), mount_points.get(),
NULL, // special_storage_policy NULL, // special_storage_policy
NULL, // quota_manager_proxy, NULL, // quota_manager_proxy,
......
...@@ -7,6 +7,8 @@ ...@@ -7,6 +7,8 @@
#include "base/callback.h" #include "base/callback.h"
#include "base/files/file_path.h" #include "base/files/file_path.h"
#include "base/logging.h" #include "base/logging.h"
#include "base/single_thread_task_runner.h"
#include "base/thread_task_runner_handle.h"
#include "base/threading/sequenced_worker_pool.h" #include "base/threading/sequenced_worker_pool.h"
#include "chrome/browser/chromeos/drive/drive_integration_service.h" #include "chrome/browser/chromeos/drive/drive_integration_service.h"
#include "chrome/browser/chromeos/drive/file_system_util.h" #include "chrome/browser/chromeos/drive/file_system_util.h"
...@@ -58,7 +60,7 @@ void PostFileSystemCallback( ...@@ -58,7 +60,7 @@ void PostFileSystemCallback(
on_error_callback.is_null() ? on_error_callback.is_null() ?
base::Closure() : base::Closure() :
base::Bind(&google_apis::RunTaskWithTaskRunner, base::Bind(&google_apis::RunTaskWithTaskRunner,
base::MessageLoopProxy::current(), base::ThreadTaskRunnerHandle::Get(),
on_error_callback))); on_error_callback)));
} }
......
...@@ -7,6 +7,7 @@ ...@@ -7,6 +7,7 @@
#include "base/files/file_path.h" #include "base/files/file_path.h"
#include "base/logging.h" #include "base/logging.h"
#include "base/task_runner_util.h" #include "base/task_runner_util.h"
#include "base/thread_task_runner_handle.h"
#include "base/threading/sequenced_worker_pool.h" #include "base/threading/sequenced_worker_pool.h"
#include "chrome/browser/chromeos/drive/drive.pb.h" #include "chrome/browser/chromeos/drive/drive.pb.h"
#include "chrome/browser/chromeos/drive/file_errors.h" #include "chrome/browser/chromeos/drive/file_errors.h"
...@@ -340,7 +341,7 @@ void OpenFile(const base::FilePath& file_path, ...@@ -340,7 +341,7 @@ void OpenFile(const base::FilePath& file_path,
base::File::FLAG_WRITE | base::File::FLAG_WRITE |
base::File::FLAG_WRITE_ATTRIBUTES | base::File::FLAG_WRITE_ATTRIBUTES |
base::File::FLAG_APPEND)) { base::File::FLAG_APPEND)) {
base::MessageLoopProxy::current()->PostTask( base::ThreadTaskRunnerHandle::Get()->PostTask(
FROM_HERE, FROM_HERE,
base::Bind(callback, base::Bind(callback,
Passed(base::File(base::File::FILE_ERROR_FAILED)), Passed(base::File(base::File::FILE_ERROR_FAILED)),
......
...@@ -7,12 +7,12 @@ ...@@ -7,12 +7,12 @@
#include <algorithm> #include <algorithm>
#include "base/files/file_util.h" #include "base/files/file_util.h"
#include "base/message_loop/message_loop.h"
#include "base/metrics/histogram.h" #include "base/metrics/histogram.h"
#include "base/prefs/pref_service.h" #include "base/prefs/pref_service.h"
#include "base/rand_util.h" #include "base/rand_util.h"
#include "base/strings/string_number_conversions.h" #include "base/strings/string_number_conversions.h"
#include "base/strings/stringprintf.h" #include "base/strings/stringprintf.h"
#include "base/thread_task_runner_handle.h"
#include "chrome/browser/drive/event_logger.h" #include "chrome/browser/drive/event_logger.h"
#include "chrome/common/pref_names.h" #include "chrome/common/pref_names.h"
#include "content/public/browser/browser_thread.h" #include "content/public/browser/browser_thread.h"
...@@ -791,7 +791,7 @@ void JobScheduler::DoJobLoop(QueueType queue_type) { ...@@ -791,7 +791,7 @@ void JobScheduler::DoJobLoop(QueueType queue_type) {
// Wait when throttled. // Wait when throttled.
const base::Time now = base::Time::Now(); const base::Time now = base::Time::Now();
if (now < wait_until_) { if (now < wait_until_) {
base::MessageLoopProxy::current()->PostDelayedTask( base::ThreadTaskRunnerHandle::Get()->PostDelayedTask(
FROM_HERE, FROM_HERE,
base::Bind(&JobScheduler::DoJobLoop, base::Bind(&JobScheduler::DoJobLoop,
weak_ptr_factory_.GetWeakPtr(), weak_ptr_factory_.GetWeakPtr(),
...@@ -921,7 +921,8 @@ bool JobScheduler::OnJobDone(JobID job_id, ...@@ -921,7 +921,8 @@ bool JobScheduler::OnJobDone(JobID job_id,
// Post a task to continue the job loop. This allows us to finish handling // Post a task to continue the job loop. This allows us to finish handling
// the current job before starting the next one. // the current job before starting the next one.
base::MessageLoopProxy::current()->PostTask(FROM_HERE, base::ThreadTaskRunnerHandle::Get()->PostTask(
FROM_HERE,
base::Bind(&JobScheduler::DoJobLoop, base::Bind(&JobScheduler::DoJobLoop,
weak_ptr_factory_.GetWeakPtr(), weak_ptr_factory_.GetWeakPtr(),
queue_type)); queue_type));
...@@ -1152,8 +1153,8 @@ void JobScheduler::AbortNotRunningJob(JobEntry* job, ...@@ -1152,8 +1153,8 @@ void JobScheduler::AbortNotRunningJob(JobEntry* job,
queue_[GetJobQueueType(job->job_info.job_type)]->Remove(job->job_info.job_id); queue_[GetJobQueueType(job->job_info.job_type)]->Remove(job->job_info.job_id);
NotifyJobDone(job->job_info, error); NotifyJobDone(job->job_info, error);
job_map_.Remove(job->job_info.job_id); job_map_.Remove(job->job_info.job_id);
base::MessageLoopProxy::current()->PostTask(FROM_HERE, base::ThreadTaskRunnerHandle::Get()->PostTask(FROM_HERE,
base::Bind(callback, error)); base::Bind(callback, error));
} }
void JobScheduler::NotifyJobAdded(const JobInfo& job_info) { void JobScheduler::NotifyJobAdded(const JobInfo& job_info) {
......
...@@ -11,8 +11,10 @@ ...@@ -11,8 +11,10 @@
#include "base/files/scoped_temp_dir.h" #include "base/files/scoped_temp_dir.h"
#include "base/prefs/testing_pref_service.h" #include "base/prefs/testing_pref_service.h"
#include "base/run_loop.h" #include "base/run_loop.h"
#include "base/single_thread_task_runner.h"
#include "base/stl_util.h" #include "base/stl_util.h"
#include "base/strings/stringprintf.h" #include "base/strings/stringprintf.h"
#include "base/thread_task_runner_handle.h"
#include "chrome/browser/chromeos/drive/test_util.h" #include "chrome/browser/chromeos/drive/test_util.h"
#include "chrome/browser/drive/event_logger.h" #include "chrome/browser/drive/event_logger.h"
#include "chrome/browser/drive/fake_drive_service.h" #include "chrome/browser/drive/fake_drive_service.h"
...@@ -142,10 +144,11 @@ class JobSchedulerTest : public testing::Test { ...@@ -142,10 +144,11 @@ class JobSchedulerTest : public testing::Test {
test_util::SetUpTestEntries(fake_drive_service_.get()); test_util::SetUpTestEntries(fake_drive_service_.get());
fake_drive_service_->LoadAppListForDriveApi("drive/applist.json"); fake_drive_service_->LoadAppListForDriveApi("drive/applist.json");
scheduler_.reset(new JobScheduler(pref_service_.get(), scheduler_.reset(new JobScheduler(
logger_.get(), pref_service_.get(),
fake_drive_service_.get(), logger_.get(),
base::MessageLoopProxy::current().get())); fake_drive_service_.get(),
base::ThreadTaskRunnerHandle::Get().get()));
scheduler_->SetDisableThrottling(true); scheduler_->SetDisableThrottling(true);
} }
......
...@@ -8,6 +8,8 @@ ...@@ -8,6 +8,8 @@
#include "base/files/file_util.h" #include "base/files/file_util.h"
#include "base/files/scoped_temp_dir.h" #include "base/files/scoped_temp_dir.h"
#include "base/memory/scoped_ptr.h" #include "base/memory/scoped_ptr.h"
#include "base/single_thread_task_runner.h"
#include "base/thread_task_runner_handle.h"
#include "chrome/browser/chromeos/drive/drive.pb.h" #include "chrome/browser/chromeos/drive/drive.pb.h"
#include "chrome/browser/chromeos/drive/fake_free_disk_space_getter.h" #include "chrome/browser/chromeos/drive/fake_free_disk_space_getter.h"
#include "chrome/browser/chromeos/drive/file_system_util.h" #include "chrome/browser/chromeos/drive/file_system_util.h"
...@@ -29,16 +31,16 @@ class RemoveStaleCacheFilesTest : public testing::Test { ...@@ -29,16 +31,16 @@ class RemoveStaleCacheFilesTest : public testing::Test {
fake_free_disk_space_getter_.reset(new FakeFreeDiskSpaceGetter); fake_free_disk_space_getter_.reset(new FakeFreeDiskSpaceGetter);
metadata_storage_.reset(new ResourceMetadataStorage( metadata_storage_.reset(new ResourceMetadataStorage(
temp_dir_.path(), base::MessageLoopProxy::current().get())); temp_dir_.path(), base::ThreadTaskRunnerHandle::Get().get()));
cache_.reset(new FileCache(metadata_storage_.get(), cache_.reset(new FileCache(metadata_storage_.get(),
temp_dir_.path(), temp_dir_.path(),
base::MessageLoopProxy::current().get(), base::ThreadTaskRunnerHandle::Get().get(),
fake_free_disk_space_getter_.get())); fake_free_disk_space_getter_.get()));
resource_metadata_.reset(new ResourceMetadata( resource_metadata_.reset(new ResourceMetadata(
metadata_storage_.get(), cache_.get(), metadata_storage_.get(), cache_.get(),
base::MessageLoopProxy::current())); base::ThreadTaskRunnerHandle::Get()));
ASSERT_TRUE(metadata_storage_->Initialize()); ASSERT_TRUE(metadata_storage_->Initialize());
ASSERT_TRUE(cache_->Initialize()); ASSERT_TRUE(cache_->Initialize());
......
...@@ -8,7 +8,9 @@ ...@@ -8,7 +8,9 @@
#include "base/files/file_util.h" #include "base/files/file_util.h"
#include "base/files/scoped_temp_dir.h" #include "base/files/scoped_temp_dir.h"
#include "base/single_thread_task_runner.h"
#include "base/strings/string_split.h" #include "base/strings/string_split.h"
#include "base/thread_task_runner_handle.h"
#include "chrome/browser/chromeos/drive/drive.pb.h" #include "chrome/browser/chromeos/drive/drive.pb.h"
#include "chrome/browser/chromeos/drive/test_util.h" #include "chrome/browser/chromeos/drive/test_util.h"
#include "content/public/test/test_browser_thread_bundle.h" #include "content/public/test/test_browser_thread_bundle.h"
...@@ -25,7 +27,7 @@ class ResourceMetadataStorageTest : public testing::Test { ...@@ -25,7 +27,7 @@ class ResourceMetadataStorageTest : public testing::Test {
ASSERT_TRUE(temp_dir_.CreateUniqueTempDir()); ASSERT_TRUE(temp_dir_.CreateUniqueTempDir());
storage_.reset(new ResourceMetadataStorage( storage_.reset(new ResourceMetadataStorage(
temp_dir_.path(), base::MessageLoopProxy::current().get())); temp_dir_.path(), base::ThreadTaskRunnerHandle::Get().get()));
ASSERT_TRUE(storage_->Initialize()); ASSERT_TRUE(storage_->Initialize());
} }
...@@ -264,7 +266,7 @@ TEST_F(ResourceMetadataStorageTest, OpenExistingDB) { ...@@ -264,7 +266,7 @@ TEST_F(ResourceMetadataStorageTest, OpenExistingDB) {
// Close DB and reopen. // Close DB and reopen.
storage_.reset(new ResourceMetadataStorage( storage_.reset(new ResourceMetadataStorage(
temp_dir_.path(), base::MessageLoopProxy::current().get())); temp_dir_.path(), base::ThreadTaskRunnerHandle::Get().get()));
ASSERT_TRUE(storage_->Initialize()); ASSERT_TRUE(storage_->Initialize());
// Can read data. // Can read data.
...@@ -310,7 +312,7 @@ TEST_F(ResourceMetadataStorageTest, IncompatibleDB_M29) { ...@@ -310,7 +312,7 @@ TEST_F(ResourceMetadataStorageTest, IncompatibleDB_M29) {
storage_.reset(); storage_.reset();
EXPECT_TRUE(ResourceMetadataStorage::UpgradeOldDB(temp_dir_.path())); EXPECT_TRUE(ResourceMetadataStorage::UpgradeOldDB(temp_dir_.path()));
storage_.reset(new ResourceMetadataStorage( storage_.reset(new ResourceMetadataStorage(
temp_dir_.path(), base::MessageLoopProxy::current().get())); temp_dir_.path(), base::ThreadTaskRunnerHandle::Get().get()));
ASSERT_TRUE(storage_->Initialize()); ASSERT_TRUE(storage_->Initialize());
// Resource-ID-to-local-ID mapping is added. // Resource-ID-to-local-ID mapping is added.
...@@ -362,7 +364,7 @@ TEST_F(ResourceMetadataStorageTest, IncompatibleDB_M32) { ...@@ -362,7 +364,7 @@ TEST_F(ResourceMetadataStorageTest, IncompatibleDB_M32) {
storage_.reset(); storage_.reset();
EXPECT_TRUE(ResourceMetadataStorage::UpgradeOldDB(temp_dir_.path())); EXPECT_TRUE(ResourceMetadataStorage::UpgradeOldDB(temp_dir_.path()));
storage_.reset(new ResourceMetadataStorage( storage_.reset(new ResourceMetadataStorage(
temp_dir_.path(), base::MessageLoopProxy::current().get())); temp_dir_.path(), base::ThreadTaskRunnerHandle::Get().get()));
ASSERT_TRUE(storage_->Initialize()); ASSERT_TRUE(storage_->Initialize());
// Data is erased, except cache and id mapping entries. // Data is erased, except cache and id mapping entries.
...@@ -423,7 +425,7 @@ TEST_F(ResourceMetadataStorageTest, IncompatibleDB_M33) { ...@@ -423,7 +425,7 @@ TEST_F(ResourceMetadataStorageTest, IncompatibleDB_M33) {
storage_.reset(); storage_.reset();
EXPECT_TRUE(ResourceMetadataStorage::UpgradeOldDB(temp_dir_.path())); EXPECT_TRUE(ResourceMetadataStorage::UpgradeOldDB(temp_dir_.path()));
storage_.reset(new ResourceMetadataStorage( storage_.reset(new ResourceMetadataStorage(
temp_dir_.path(), base::MessageLoopProxy::current().get())); temp_dir_.path(), base::ThreadTaskRunnerHandle::Get().get()));
ASSERT_TRUE(storage_->Initialize()); ASSERT_TRUE(storage_->Initialize());
// No data is lost. // No data is lost.
...@@ -461,7 +463,7 @@ TEST_F(ResourceMetadataStorageTest, IncompatibleDB_Unknown) { ...@@ -461,7 +463,7 @@ TEST_F(ResourceMetadataStorageTest, IncompatibleDB_Unknown) {
storage_.reset(); storage_.reset();
EXPECT_FALSE(ResourceMetadataStorage::UpgradeOldDB(temp_dir_.path())); EXPECT_FALSE(ResourceMetadataStorage::UpgradeOldDB(temp_dir_.path()));
storage_.reset(new ResourceMetadataStorage( storage_.reset(new ResourceMetadataStorage(
temp_dir_.path(), base::MessageLoopProxy::current().get())); temp_dir_.path(), base::ThreadTaskRunnerHandle::Get().get()));
ASSERT_TRUE(storage_->Initialize()); ASSERT_TRUE(storage_->Initialize());
// Data is erased because of the incompatible version. // Data is erased because of the incompatible version.
...@@ -494,7 +496,7 @@ TEST_F(ResourceMetadataStorageTest, DeleteUnusedIDEntries) { ...@@ -494,7 +496,7 @@ TEST_F(ResourceMetadataStorageTest, DeleteUnusedIDEntries) {
storage_.reset(); storage_.reset();
EXPECT_TRUE(ResourceMetadataStorage::UpgradeOldDB(temp_dir_.path())); EXPECT_TRUE(ResourceMetadataStorage::UpgradeOldDB(temp_dir_.path()));
storage_.reset(new ResourceMetadataStorage( storage_.reset(new ResourceMetadataStorage(
temp_dir_.path(), base::MessageLoopProxy::current().get())); temp_dir_.path(), base::ThreadTaskRunnerHandle::Get().get()));
ASSERT_TRUE(storage_->Initialize()); ASSERT_TRUE(storage_->Initialize());
// Only the unused entry is deleted. // Only the unused entry is deleted.
...@@ -511,7 +513,7 @@ TEST_F(ResourceMetadataStorageTest, WrongPath) { ...@@ -511,7 +513,7 @@ TEST_F(ResourceMetadataStorageTest, WrongPath) {
ASSERT_TRUE(base::CreateTemporaryFileInDir(temp_dir_.path(), &path)); ASSERT_TRUE(base::CreateTemporaryFileInDir(temp_dir_.path(), &path));
storage_.reset(new ResourceMetadataStorage( storage_.reset(new ResourceMetadataStorage(
path, base::MessageLoopProxy::current().get())); path, base::ThreadTaskRunnerHandle::Get().get()));
// Cannot initialize DB beacause the path does not point a directory. // Cannot initialize DB beacause the path does not point a directory.
ASSERT_FALSE(storage_->Initialize()); ASSERT_FALSE(storage_->Initialize());
} }
...@@ -540,7 +542,7 @@ TEST_F(ResourceMetadataStorageTest, RecoverCacheEntriesFromTrashedResourceMap) { ...@@ -540,7 +542,7 @@ TEST_F(ResourceMetadataStorageTest, RecoverCacheEntriesFromTrashedResourceMap) {
// Reopen. This should result in trashing the DB. // Reopen. This should result in trashing the DB.
storage_.reset(new ResourceMetadataStorage( storage_.reset(new ResourceMetadataStorage(
temp_dir_.path(), base::MessageLoopProxy::current().get())); temp_dir_.path(), base::ThreadTaskRunnerHandle::Get().get()));
ASSERT_TRUE(storage_->Initialize()); ASSERT_TRUE(storage_->Initialize());
// Recover cache entries from the trashed DB. // Recover cache entries from the trashed DB.
......
...@@ -9,7 +9,9 @@ ...@@ -9,7 +9,9 @@
#include <vector> #include <vector>
#include "base/files/scoped_temp_dir.h" #include "base/files/scoped_temp_dir.h"
#include "base/single_thread_task_runner.h"
#include "base/strings/stringprintf.h" #include "base/strings/stringprintf.h"
#include "base/thread_task_runner_handle.h"
#include "chrome/browser/chromeos/drive/drive.pb.h" #include "chrome/browser/chromeos/drive/drive.pb.h"
#include "chrome/browser/chromeos/drive/fake_free_disk_space_getter.h" #include "chrome/browser/chromeos/drive/fake_free_disk_space_getter.h"
#include "chrome/browser/chromeos/drive/file_cache.h" #include "chrome/browser/chromeos/drive/file_cache.h"
...@@ -138,19 +140,19 @@ class ResourceMetadataTest : public testing::Test { ...@@ -138,19 +140,19 @@ class ResourceMetadataTest : public testing::Test {
ASSERT_TRUE(temp_dir_.CreateUniqueTempDir()); ASSERT_TRUE(temp_dir_.CreateUniqueTempDir());
metadata_storage_.reset(new ResourceMetadataStorage( metadata_storage_.reset(new ResourceMetadataStorage(
temp_dir_.path(), base::MessageLoopProxy::current().get())); temp_dir_.path(), base::ThreadTaskRunnerHandle::Get().get()));
ASSERT_TRUE(metadata_storage_->Initialize()); ASSERT_TRUE(metadata_storage_->Initialize());
fake_free_disk_space_getter_.reset(new FakeFreeDiskSpaceGetter); fake_free_disk_space_getter_.reset(new FakeFreeDiskSpaceGetter);
cache_.reset(new FileCache(metadata_storage_.get(), cache_.reset(new FileCache(metadata_storage_.get(),
temp_dir_.path(), temp_dir_.path(),
base::MessageLoopProxy::current().get(), base::ThreadTaskRunnerHandle::Get().get(),
fake_free_disk_space_getter_.get())); fake_free_disk_space_getter_.get()));
ASSERT_TRUE(cache_->Initialize()); ASSERT_TRUE(cache_->Initialize());
resource_metadata_.reset(new ResourceMetadata( resource_metadata_.reset(new ResourceMetadata(
metadata_storage_.get(), cache_.get(), metadata_storage_.get(), cache_.get(),
base::MessageLoopProxy::current())); base::ThreadTaskRunnerHandle::Get()));
ASSERT_EQ(FILE_ERROR_OK, resource_metadata_->Initialize()); ASSERT_EQ(FILE_ERROR_OK, resource_metadata_->Initialize());
......
...@@ -7,9 +7,10 @@ ...@@ -7,9 +7,10 @@
#include "base/files/file_util.h" #include "base/files/file_util.h"
#include "base/files/scoped_temp_dir.h" #include "base/files/scoped_temp_dir.h"
#include "base/i18n/string_search.h" #include "base/i18n/string_search.h"
#include "base/message_loop/message_loop_proxy.h"
#include "base/run_loop.h" #include "base/run_loop.h"
#include "base/single_thread_task_runner.h"
#include "base/strings/utf_string_conversions.h" #include "base/strings/utf_string_conversions.h"
#include "base/thread_task_runner_handle.h"
#include "chrome/browser/chromeos/drive/fake_free_disk_space_getter.h" #include "chrome/browser/chromeos/drive/fake_free_disk_space_getter.h"
#include "chrome/browser/chromeos/drive/file_cache.h" #include "chrome/browser/chromeos/drive/file_cache.h"
#include "chrome/browser/chromeos/drive/file_system_util.h" #include "chrome/browser/chromeos/drive/file_system_util.h"
...@@ -45,19 +46,19 @@ class SearchMetadataTest : public testing::Test { ...@@ -45,19 +46,19 @@ class SearchMetadataTest : public testing::Test {
fake_free_disk_space_getter_.reset(new FakeFreeDiskSpaceGetter); fake_free_disk_space_getter_.reset(new FakeFreeDiskSpaceGetter);
metadata_storage_.reset(new ResourceMetadataStorage( metadata_storage_.reset(new ResourceMetadataStorage(
temp_dir_.path(), base::MessageLoopProxy::current().get())); temp_dir_.path(), base::ThreadTaskRunnerHandle::Get().get()));
ASSERT_TRUE(metadata_storage_->Initialize()); ASSERT_TRUE(metadata_storage_->Initialize());
cache_.reset(new FileCache(metadata_storage_.get(), cache_.reset(new FileCache(metadata_storage_.get(),
temp_dir_.path(), temp_dir_.path(),
base::MessageLoopProxy::current().get(), base::ThreadTaskRunnerHandle::Get().get(),
fake_free_disk_space_getter_.get())); fake_free_disk_space_getter_.get()));
ASSERT_TRUE(cache_->Initialize()); ASSERT_TRUE(cache_->Initialize());
resource_metadata_.reset( resource_metadata_.reset(
new ResourceMetadata(metadata_storage_.get(), new ResourceMetadata(metadata_storage_.get(),
cache_.get(), cache_.get(),
base::MessageLoopProxy::current())); base::ThreadTaskRunnerHandle::Get().get()));
ASSERT_EQ(FILE_ERROR_OK, resource_metadata_->Initialize()); ASSERT_EQ(FILE_ERROR_OK, resource_metadata_->Initialize());
AddEntriesToMetadata(); AddEntriesToMetadata();
...@@ -163,7 +164,7 @@ TEST_F(SearchMetadataTest, SearchMetadata_ZeroMatches) { ...@@ -163,7 +164,7 @@ TEST_F(SearchMetadataTest, SearchMetadata_ZeroMatches) {
scoped_ptr<MetadataSearchResultVector> result; scoped_ptr<MetadataSearchResultVector> result;
SearchMetadata( SearchMetadata(
base::MessageLoopProxy::current(), resource_metadata_.get(), base::ThreadTaskRunnerHandle::Get(), resource_metadata_.get(),
"NonExistent", base::Bind(&MatchesType, SEARCH_METADATA_ALL), "NonExistent", base::Bind(&MatchesType, SEARCH_METADATA_ALL),
kDefaultAtMostNumMatches, kDefaultAtMostNumMatches,
google_apis::test_util::CreateCopyResultCallback(&error, &result)); google_apis::test_util::CreateCopyResultCallback(&error, &result));
...@@ -178,7 +179,7 @@ TEST_F(SearchMetadataTest, SearchMetadata_RegularFile) { ...@@ -178,7 +179,7 @@ TEST_F(SearchMetadataTest, SearchMetadata_RegularFile) {
scoped_ptr<MetadataSearchResultVector> result; scoped_ptr<MetadataSearchResultVector> result;
SearchMetadata( SearchMetadata(
base::MessageLoopProxy::current(), resource_metadata_.get(), base::ThreadTaskRunnerHandle::Get(), resource_metadata_.get(),
"SubDirectory File 1.txt", base::Bind(&MatchesType, SEARCH_METADATA_ALL), "SubDirectory File 1.txt", base::Bind(&MatchesType, SEARCH_METADATA_ALL),
kDefaultAtMostNumMatches, kDefaultAtMostNumMatches,
google_apis::test_util::CreateCopyResultCallback(&error, &result)); google_apis::test_util::CreateCopyResultCallback(&error, &result));
...@@ -198,7 +199,7 @@ TEST_F(SearchMetadataTest, SearchMetadata_CaseInsensitiveSearch) { ...@@ -198,7 +199,7 @@ TEST_F(SearchMetadataTest, SearchMetadata_CaseInsensitiveSearch) {
// The query is all in lower case. // The query is all in lower case.
SearchMetadata( SearchMetadata(
base::MessageLoopProxy::current(), resource_metadata_.get(), base::ThreadTaskRunnerHandle::Get(), resource_metadata_.get(),
"subdirectory file 1.txt", base::Bind(&MatchesType, SEARCH_METADATA_ALL), "subdirectory file 1.txt", base::Bind(&MatchesType, SEARCH_METADATA_ALL),
kDefaultAtMostNumMatches, kDefaultAtMostNumMatches,
google_apis::test_util::CreateCopyResultCallback(&error, &result)); google_apis::test_util::CreateCopyResultCallback(&error, &result));
...@@ -215,7 +216,7 @@ TEST_F(SearchMetadataTest, SearchMetadata_RegularFiles) { ...@@ -215,7 +216,7 @@ TEST_F(SearchMetadataTest, SearchMetadata_RegularFiles) {
scoped_ptr<MetadataSearchResultVector> result; scoped_ptr<MetadataSearchResultVector> result;
SearchMetadata( SearchMetadata(
base::MessageLoopProxy::current(), resource_metadata_.get(), "SubDir", base::ThreadTaskRunnerHandle::Get(), resource_metadata_.get(), "SubDir",
base::Bind(&MatchesType, SEARCH_METADATA_ALL), kDefaultAtMostNumMatches, base::Bind(&MatchesType, SEARCH_METADATA_ALL), kDefaultAtMostNumMatches,
google_apis::test_util::CreateCopyResultCallback(&error, &result)); google_apis::test_util::CreateCopyResultCallback(&error, &result));
base::RunLoop().RunUntilIdle(); base::RunLoop().RunUntilIdle();
...@@ -238,7 +239,7 @@ TEST_F(SearchMetadataTest, SearchMetadata_AtMostOneFile) { ...@@ -238,7 +239,7 @@ TEST_F(SearchMetadataTest, SearchMetadata_AtMostOneFile) {
// There are two files matching "SubDir" but only one file should be // There are two files matching "SubDir" but only one file should be
// returned. // returned.
SearchMetadata( SearchMetadata(
base::MessageLoopProxy::current(), resource_metadata_.get(), "SubDir", base::ThreadTaskRunnerHandle::Get(), resource_metadata_.get(), "SubDir",
base::Bind(&MatchesType, SEARCH_METADATA_ALL), base::Bind(&MatchesType, SEARCH_METADATA_ALL),
1, // at_most_num_matches 1, // at_most_num_matches
google_apis::test_util::CreateCopyResultCallback(&error, &result)); google_apis::test_util::CreateCopyResultCallback(&error, &result));
...@@ -255,7 +256,7 @@ TEST_F(SearchMetadataTest, SearchMetadata_Directory) { ...@@ -255,7 +256,7 @@ TEST_F(SearchMetadataTest, SearchMetadata_Directory) {
scoped_ptr<MetadataSearchResultVector> result; scoped_ptr<MetadataSearchResultVector> result;
SearchMetadata( SearchMetadata(
base::MessageLoopProxy::current(), resource_metadata_.get(), base::ThreadTaskRunnerHandle::Get(), resource_metadata_.get(),
"Directory 1", base::Bind(&MatchesType, SEARCH_METADATA_ALL), "Directory 1", base::Bind(&MatchesType, SEARCH_METADATA_ALL),
kDefaultAtMostNumMatches, kDefaultAtMostNumMatches,
google_apis::test_util::CreateCopyResultCallback(&error, &result)); google_apis::test_util::CreateCopyResultCallback(&error, &result));
...@@ -271,7 +272,7 @@ TEST_F(SearchMetadataTest, SearchMetadata_HostedDocument) { ...@@ -271,7 +272,7 @@ TEST_F(SearchMetadataTest, SearchMetadata_HostedDocument) {
scoped_ptr<MetadataSearchResultVector> result; scoped_ptr<MetadataSearchResultVector> result;
SearchMetadata( SearchMetadata(
base::MessageLoopProxy::current(), resource_metadata_.get(), "Document", base::ThreadTaskRunnerHandle::Get(), resource_metadata_.get(), "Document",
base::Bind(&MatchesType, SEARCH_METADATA_ALL), kDefaultAtMostNumMatches, base::Bind(&MatchesType, SEARCH_METADATA_ALL), kDefaultAtMostNumMatches,
google_apis::test_util::CreateCopyResultCallback(&error, &result)); google_apis::test_util::CreateCopyResultCallback(&error, &result));
base::RunLoop().RunUntilIdle(); base::RunLoop().RunUntilIdle();
...@@ -288,7 +289,7 @@ TEST_F(SearchMetadataTest, SearchMetadata_ExcludeHostedDocument) { ...@@ -288,7 +289,7 @@ TEST_F(SearchMetadataTest, SearchMetadata_ExcludeHostedDocument) {
scoped_ptr<MetadataSearchResultVector> result; scoped_ptr<MetadataSearchResultVector> result;
SearchMetadata( SearchMetadata(
base::MessageLoopProxy::current(), resource_metadata_.get(), "Document", base::ThreadTaskRunnerHandle::Get(), resource_metadata_.get(), "Document",
base::Bind(&MatchesType, SEARCH_METADATA_EXCLUDE_HOSTED_DOCUMENTS), base::Bind(&MatchesType, SEARCH_METADATA_EXCLUDE_HOSTED_DOCUMENTS),
kDefaultAtMostNumMatches, kDefaultAtMostNumMatches,
google_apis::test_util::CreateCopyResultCallback(&error, &result)); google_apis::test_util::CreateCopyResultCallback(&error, &result));
...@@ -303,7 +304,7 @@ TEST_F(SearchMetadataTest, SearchMetadata_SharedWithMe) { ...@@ -303,7 +304,7 @@ TEST_F(SearchMetadataTest, SearchMetadata_SharedWithMe) {
scoped_ptr<MetadataSearchResultVector> result; scoped_ptr<MetadataSearchResultVector> result;
SearchMetadata( SearchMetadata(
base::MessageLoopProxy::current(), resource_metadata_.get(), "", base::ThreadTaskRunnerHandle::Get(), resource_metadata_.get(), "",
base::Bind(&MatchesType, SEARCH_METADATA_SHARED_WITH_ME), base::Bind(&MatchesType, SEARCH_METADATA_SHARED_WITH_ME),
kDefaultAtMostNumMatches, kDefaultAtMostNumMatches,
google_apis::test_util::CreateCopyResultCallback(&error, &result)); google_apis::test_util::CreateCopyResultCallback(&error, &result));
...@@ -320,7 +321,7 @@ TEST_F(SearchMetadataTest, SearchMetadata_FileAndDirectory) { ...@@ -320,7 +321,7 @@ TEST_F(SearchMetadataTest, SearchMetadata_FileAndDirectory) {
scoped_ptr<MetadataSearchResultVector> result; scoped_ptr<MetadataSearchResultVector> result;
SearchMetadata( SearchMetadata(
base::MessageLoopProxy::current(), resource_metadata_.get(), base::ThreadTaskRunnerHandle::Get(), resource_metadata_.get(),
"excludeDir-test", base::Bind(&MatchesType, SEARCH_METADATA_ALL), "excludeDir-test", base::Bind(&MatchesType, SEARCH_METADATA_ALL),
kDefaultAtMostNumMatches, kDefaultAtMostNumMatches,
google_apis::test_util::CreateCopyResultCallback(&error, &result)); google_apis::test_util::CreateCopyResultCallback(&error, &result));
...@@ -341,7 +342,7 @@ TEST_F(SearchMetadataTest, SearchMetadata_ExcludeDirectory) { ...@@ -341,7 +342,7 @@ TEST_F(SearchMetadataTest, SearchMetadata_ExcludeDirectory) {
scoped_ptr<MetadataSearchResultVector> result; scoped_ptr<MetadataSearchResultVector> result;
SearchMetadata( SearchMetadata(
base::MessageLoopProxy::current(), resource_metadata_.get(), base::ThreadTaskRunnerHandle::Get(), resource_metadata_.get(),
"excludeDir-test", "excludeDir-test",
base::Bind(&MatchesType, SEARCH_METADATA_EXCLUDE_DIRECTORIES), base::Bind(&MatchesType, SEARCH_METADATA_EXCLUDE_DIRECTORIES),
kDefaultAtMostNumMatches, kDefaultAtMostNumMatches,
...@@ -365,7 +366,7 @@ TEST_F(SearchMetadataTest, SearchMetadata_ExcludeSpecialDirectories) { ...@@ -365,7 +366,7 @@ TEST_F(SearchMetadataTest, SearchMetadata_ExcludeSpecialDirectories) {
const std::string query = kQueries[i]; const std::string query = kQueries[i];
SearchMetadata( SearchMetadata(
base::MessageLoopProxy::current(), resource_metadata_.get(), query, base::ThreadTaskRunnerHandle::Get(), resource_metadata_.get(), query,
base::Bind(&MatchesType, SEARCH_METADATA_ALL), kDefaultAtMostNumMatches, base::Bind(&MatchesType, SEARCH_METADATA_ALL), kDefaultAtMostNumMatches,
google_apis::test_util::CreateCopyResultCallback(&error, &result)); google_apis::test_util::CreateCopyResultCallback(&error, &result));
...@@ -381,7 +382,7 @@ TEST_F(SearchMetadataTest, SearchMetadata_Offline) { ...@@ -381,7 +382,7 @@ TEST_F(SearchMetadataTest, SearchMetadata_Offline) {
scoped_ptr<MetadataSearchResultVector> result; scoped_ptr<MetadataSearchResultVector> result;
SearchMetadata( SearchMetadata(
base::MessageLoopProxy::current(), resource_metadata_.get(), "", base::ThreadTaskRunnerHandle::Get(), resource_metadata_.get(), "",
base::Bind(&MatchesType, SEARCH_METADATA_OFFLINE), base::Bind(&MatchesType, SEARCH_METADATA_OFFLINE),
kDefaultAtMostNumMatches, kDefaultAtMostNumMatches,
google_apis::test_util::CreateCopyResultCallback(&error, &result)); google_apis::test_util::CreateCopyResultCallback(&error, &result));
......
...@@ -7,7 +7,7 @@ ...@@ -7,7 +7,7 @@
#include <vector> #include <vector>
#include "base/bind.h" #include "base/bind.h"
#include "base/message_loop/message_loop_proxy.h" #include "base/thread_task_runner_handle.h"
#include "chrome/browser/chromeos/drive/drive.pb.h" #include "chrome/browser/chromeos/drive/drive.pb.h"
#include "chrome/browser/chromeos/drive/file_cache.h" #include "chrome/browser/chromeos/drive/file_cache.h"
#include "chrome/browser/chromeos/drive/file_system/download_operation.h" #include "chrome/browser/chromeos/drive/file_system/download_operation.h"
...@@ -323,7 +323,7 @@ void SyncClient::AddTask(const SyncTasks::key_type& key, ...@@ -323,7 +323,7 @@ void SyncClient::AddTask(const SyncTasks::key_type& key,
tasks_[key] = task; tasks_[key] = task;
} }
DCHECK_EQ(PENDING, task.state); DCHECK_EQ(PENDING, task.state);
base::MessageLoopProxy::current()->PostDelayedTask( base::ThreadTaskRunnerHandle::Get()->PostDelayedTask(
FROM_HERE, FROM_HERE,
base::Bind(&SyncClient::StartTask, weak_ptr_factory_.GetWeakPtr(), key), base::Bind(&SyncClient::StartTask, weak_ptr_factory_.GetWeakPtr(), key),
delay); delay);
...@@ -454,7 +454,7 @@ void SyncClient::OnTaskComplete(SyncType type, ...@@ -454,7 +454,7 @@ void SyncClient::OnTaskComplete(SyncType type,
} }
for (size_t i = 0; i < it->second.waiting_callbacks.size(); ++i) { for (size_t i = 0; i < it->second.waiting_callbacks.size(); ++i) {
base::MessageLoopProxy::current()->PostTask( base::ThreadTaskRunnerHandle::Get()->PostTask(
FROM_HERE, base::Bind(it->second.waiting_callbacks[i], error)); FROM_HERE, base::Bind(it->second.waiting_callbacks[i], error));
} }
it->second.waiting_callbacks.clear(); it->second.waiting_callbacks.clear();
...@@ -463,7 +463,7 @@ void SyncClient::OnTaskComplete(SyncType type, ...@@ -463,7 +463,7 @@ void SyncClient::OnTaskComplete(SyncType type,
DVLOG(1) << "Running again: type = " << type << ", id = " << local_id; DVLOG(1) << "Running again: type = " << type << ", id = " << local_id;
it->second.state = PENDING; it->second.state = PENDING;
it->second.should_run_again = false; it->second.should_run_again = false;
base::MessageLoopProxy::current()->PostDelayedTask( base::ThreadTaskRunnerHandle::Get()->PostDelayedTask(
FROM_HERE, FROM_HERE,
base::Bind(&SyncClient::StartTask, weak_ptr_factory_.GetWeakPtr(), key), base::Bind(&SyncClient::StartTask, weak_ptr_factory_.GetWeakPtr(), key),
retry_delay); retry_delay);
......
...@@ -10,7 +10,9 @@ ...@@ -10,7 +10,9 @@
#include "base/memory/scoped_ptr.h" #include "base/memory/scoped_ptr.h"
#include "base/prefs/testing_pref_service.h" #include "base/prefs/testing_pref_service.h"
#include "base/run_loop.h" #include "base/run_loop.h"
#include "base/single_thread_task_runner.h"
#include "base/test/test_timeouts.h" #include "base/test/test_timeouts.h"
#include "base/thread_task_runner_handle.h"
#include "chrome/browser/chromeos/drive/change_list_loader.h" #include "chrome/browser/chromeos/drive/change_list_loader.h"
#include "chrome/browser/chromeos/drive/drive.pb.h" #include "chrome/browser/chromeos/drive/drive.pb.h"
#include "chrome/browser/chromeos/drive/fake_free_disk_space_getter.h" #include "chrome/browser/chromeos/drive/fake_free_disk_space_getter.h"
...@@ -57,7 +59,7 @@ class SyncClientTestDriveService : public ::drive::FakeDriveService { ...@@ -57,7 +59,7 @@ class SyncClientTestDriveService : public ::drive::FakeDriveService {
const google_apis::ProgressCallback& progress_callback) override { const google_apis::ProgressCallback& progress_callback) override {
++download_file_count_; ++download_file_count_;
if (resource_id == resource_id_to_be_cancelled_) { if (resource_id == resource_id_to_be_cancelled_) {
base::MessageLoopProxy::current()->PostTask( base::ThreadTaskRunnerHandle::Get()->PostTask(
FROM_HERE, FROM_HERE,
base::Bind(download_action_callback, base::Bind(download_action_callback,
google_apis::DRIVE_CANCELLED, google_apis::DRIVE_CANCELLED,
...@@ -113,38 +115,39 @@ class SyncClientTest : public testing::Test { ...@@ -113,38 +115,39 @@ class SyncClientTest : public testing::Test {
drive_service_.reset(new SyncClientTestDriveService); drive_service_.reset(new SyncClientTestDriveService);
scheduler_.reset(new JobScheduler(pref_service_.get(), scheduler_.reset(new JobScheduler(
logger_.get(), pref_service_.get(),
drive_service_.get(), logger_.get(),
base::MessageLoopProxy::current().get())); drive_service_.get(),
base::ThreadTaskRunnerHandle::Get().get()));
metadata_storage_.reset(new ResourceMetadataStorage( metadata_storage_.reset(new ResourceMetadataStorage(
temp_dir_.path(), base::MessageLoopProxy::current().get())); temp_dir_.path(), base::ThreadTaskRunnerHandle::Get().get()));
ASSERT_TRUE(metadata_storage_->Initialize()); ASSERT_TRUE(metadata_storage_->Initialize());
cache_.reset(new FileCache(metadata_storage_.get(), cache_.reset(new FileCache(metadata_storage_.get(),
temp_dir_.path(), temp_dir_.path(),
base::MessageLoopProxy::current().get(), base::ThreadTaskRunnerHandle::Get().get(),
NULL /* free_disk_space_getter */)); NULL /* free_disk_space_getter */));
ASSERT_TRUE(cache_->Initialize()); ASSERT_TRUE(cache_->Initialize());
metadata_.reset(new internal::ResourceMetadata( metadata_.reset(new internal::ResourceMetadata(
metadata_storage_.get(), cache_.get(), metadata_storage_.get(), cache_.get(),
base::MessageLoopProxy::current())); base::ThreadTaskRunnerHandle::Get()));
ASSERT_EQ(FILE_ERROR_OK, metadata_->Initialize()); ASSERT_EQ(FILE_ERROR_OK, metadata_->Initialize());
about_resource_loader_.reset(new AboutResourceLoader(scheduler_.get())); about_resource_loader_.reset(new AboutResourceLoader(scheduler_.get()));
loader_controller_.reset(new LoaderController); loader_controller_.reset(new LoaderController);
change_list_loader_.reset(new ChangeListLoader( change_list_loader_.reset(new ChangeListLoader(
logger_.get(), logger_.get(),
base::MessageLoopProxy::current().get(), base::ThreadTaskRunnerHandle::Get().get(),
metadata_.get(), metadata_.get(),
scheduler_.get(), scheduler_.get(),
about_resource_loader_.get(), about_resource_loader_.get(),
loader_controller_.get())); loader_controller_.get()));
ASSERT_NO_FATAL_FAILURE(SetUpTestData()); ASSERT_NO_FATAL_FAILURE(SetUpTestData());
sync_client_.reset(new SyncClient(base::MessageLoopProxy::current().get(), sync_client_.reset(new SyncClient(base::ThreadTaskRunnerHandle::Get().get(),
&delegate_, &delegate_,
scheduler_.get(), scheduler_.get(),
metadata_.get(), metadata_.get(),
...@@ -217,7 +220,7 @@ class SyncClientTest : public testing::Test { ...@@ -217,7 +220,7 @@ class SyncClientTest : public testing::Test {
// Prepare a removed file. // Prepare a removed file.
file_system::RemoveOperation remove_operation( file_system::RemoveOperation remove_operation(
base::MessageLoopProxy::current().get(), &delegate_, metadata_.get(), base::ThreadTaskRunnerHandle::Get().get(), &delegate_, metadata_.get(),
cache_.get()); cache_.get());
remove_operation.Remove( remove_operation.Remove(
util::GetDriveMyDriveRootPath().AppendASCII("removed"), util::GetDriveMyDriveRootPath().AppendASCII("removed"),
...@@ -228,7 +231,7 @@ class SyncClientTest : public testing::Test { ...@@ -228,7 +231,7 @@ class SyncClientTest : public testing::Test {
// Prepare a moved file. // Prepare a moved file.
file_system::MoveOperation move_operation( file_system::MoveOperation move_operation(
base::MessageLoopProxy::current().get(), &delegate_, metadata_.get()); base::ThreadTaskRunnerHandle::Get().get(), &delegate_, metadata_.get());
move_operation.Move( move_operation.Move(
util::GetDriveMyDriveRootPath().AppendASCII("moved"), util::GetDriveMyDriveRootPath().AppendASCII("moved"),
util::GetDriveMyDriveRootPath().AppendASCII("moved_new_title"), util::GetDriveMyDriveRootPath().AppendASCII("moved_new_title"),
...@@ -387,7 +390,7 @@ TEST_F(SyncClientTest, RetryOnDisconnection) { ...@@ -387,7 +390,7 @@ TEST_F(SyncClientTest, RetryOnDisconnection) {
// FILE_ERROR_NO_CONNECTION is handled by SyncClient correctly. // FILE_ERROR_NO_CONNECTION is handled by SyncClient correctly.
// Without this delay, JobScheduler will keep the jobs unrun and SyncClient // Without this delay, JobScheduler will keep the jobs unrun and SyncClient
// will receive no error. // will receive no error.
base::MessageLoopProxy::current()->PostDelayedTask( base::ThreadTaskRunnerHandle::Get()->PostDelayedTask(
FROM_HERE, FROM_HERE,
base::Bind(&test_util::FakeNetworkChangeNotifier::SetConnectionType, base::Bind(&test_util::FakeNetworkChangeNotifier::SetConnectionType,
base::Unretained(fake_network_change_notifier_.get()), base::Unretained(fake_network_change_notifier_.get()),
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment