Index: chrome/browser/chromeos/file_system_provider/fileapi/provider_async_file_util_unittest.cc |
diff --git a/chrome/browser/chromeos/file_system_provider/fileapi/provider_async_file_util_unittest.cc b/chrome/browser/chromeos/file_system_provider/fileapi/provider_async_file_util_unittest.cc |
index 30e97ee4862f3b0ad19ded429db1ef193d653452..08e5f4cb1d4564548ea53127a0fecbb3e2c2667e 100644 |
--- a/chrome/browser/chromeos/file_system_provider/fileapi/provider_async_file_util_unittest.cc |
+++ b/chrome/browser/chromeos/file_system_provider/fileapi/provider_async_file_util_unittest.cc |
@@ -45,30 +45,30 @@ class EventLogger { |
virtual ~EventLogger() {} |
void OnStatus(base::File::Error error) { |
- error_.reset(new base::File::Error(error)); |
+ result_.reset(new base::File::Error(error)); |
} |
void OnCreateOrOpen(base::File file, |
const base::Closure& on_close_callback) { |
if (file.IsValid()) |
- error_.reset(new base::File::Error(base::File::FILE_OK)); |
+ result_.reset(new base::File::Error(base::File::FILE_OK)); |
- error_.reset(new base::File::Error(file.error_details())); |
+ result_.reset(new base::File::Error(file.error_details())); |
} |
void OnEnsureFileExists(base::File::Error error, bool created) { |
- error_.reset(new base::File::Error(error)); |
+ result_.reset(new base::File::Error(error)); |
} |
void OnGetFileInfo(base::File::Error error, |
const base::File::Info& file_info) { |
- error_.reset(new base::File::Error(error)); |
+ result_.reset(new base::File::Error(error)); |
} |
void OnReadDirectory(base::File::Error error, |
const fileapi::AsyncFileUtil::EntryList& file_list, |
bool has_more) { |
- error_.reset(new base::File::Error(error)); |
+ result_.reset(new base::File::Error(error)); |
} |
void OnCreateSnapshotFile( |
@@ -76,7 +76,7 @@ class EventLogger { |
const base::File::Info& file_info, |
const base::FilePath& platform_path, |
const scoped_refptr<webkit_blob::ShareableFileReference>& file_ref) { |
- error_.reset(new base::File::Error(error)); |
+ result_.reset(new base::File::Error(error)); |
} |
void OnCopyFileProgress(int64 size) {} |
@@ -85,10 +85,10 @@ class EventLogger { |
return weak_ptr_factory_.GetWeakPtr(); |
} |
- base::File::Error* error() { return error_.get(); } |
+ base::File::Error* result() { return result_.get(); } |
private: |
- scoped_ptr<base::File::Error> error_; |
+ scoped_ptr<base::File::Error> result_; |
base::WeakPtrFactory<EventLogger> weak_ptr_factory_; |
DISALLOW_COPY_AND_ASSIGN(EventLogger); |
@@ -191,8 +191,8 @@ TEST_F(FileSystemProviderProviderAsyncFileUtilTest, CreateOrOpen_Create) { |
base::File::FLAG_CREATE, |
base::Bind(&EventLogger::OnCreateOrOpen, logger.GetWeakPtr())); |
- ASSERT_TRUE(logger.error()); |
- EXPECT_EQ(base::File::FILE_ERROR_ACCESS_DENIED, *logger.error()); |
+ ASSERT_TRUE(logger.result()); |
+ EXPECT_EQ(base::File::FILE_ERROR_ACCESS_DENIED, *logger.result()); |
} |
TEST_F(FileSystemProviderProviderAsyncFileUtilTest, CreateOrOpen_CreateAlways) { |
@@ -204,8 +204,8 @@ TEST_F(FileSystemProviderProviderAsyncFileUtilTest, CreateOrOpen_CreateAlways) { |
base::File::FLAG_CREATE_ALWAYS, |
base::Bind(&EventLogger::OnCreateOrOpen, logger.GetWeakPtr())); |
- ASSERT_TRUE(logger.error()); |
- EXPECT_EQ(base::File::FILE_ERROR_ACCESS_DENIED, *logger.error()); |
+ ASSERT_TRUE(logger.result()); |
+ EXPECT_EQ(base::File::FILE_ERROR_ACCESS_DENIED, *logger.result()); |
} |
TEST_F(FileSystemProviderProviderAsyncFileUtilTest, CreateOrOpen_OpenAlways) { |
@@ -217,8 +217,8 @@ TEST_F(FileSystemProviderProviderAsyncFileUtilTest, CreateOrOpen_OpenAlways) { |
base::File::FLAG_OPEN_ALWAYS, |
base::Bind(&EventLogger::OnCreateOrOpen, logger.GetWeakPtr())); |
- ASSERT_TRUE(logger.error()); |
- EXPECT_EQ(base::File::FILE_ERROR_ACCESS_DENIED, *logger.error()); |
+ ASSERT_TRUE(logger.result()); |
+ EXPECT_EQ(base::File::FILE_ERROR_ACCESS_DENIED, *logger.result()); |
} |
TEST_F(FileSystemProviderProviderAsyncFileUtilTest, |
@@ -231,8 +231,8 @@ TEST_F(FileSystemProviderProviderAsyncFileUtilTest, |
base::File::FLAG_OPEN_TRUNCATED, |
base::Bind(&EventLogger::OnCreateOrOpen, logger.GetWeakPtr())); |
- ASSERT_TRUE(logger.error()); |
- EXPECT_EQ(base::File::FILE_ERROR_ACCESS_DENIED, *logger.error()); |
+ ASSERT_TRUE(logger.result()); |
+ EXPECT_EQ(base::File::FILE_ERROR_ACCESS_DENIED, *logger.result()); |
} |
TEST_F(FileSystemProviderProviderAsyncFileUtilTest, CreateOrOpen_Open) { |
@@ -244,8 +244,8 @@ TEST_F(FileSystemProviderProviderAsyncFileUtilTest, CreateOrOpen_Open) { |
base::File::FLAG_OPEN, |
base::Bind(&EventLogger::OnCreateOrOpen, logger.GetWeakPtr())); |
- ASSERT_TRUE(logger.error()); |
- EXPECT_EQ(base::File::FILE_ERROR_INVALID_OPERATION, *logger.error()); |
+ ASSERT_TRUE(logger.result()); |
+ EXPECT_EQ(base::File::FILE_ERROR_INVALID_OPERATION, *logger.result()); |
} |
TEST_F(FileSystemProviderProviderAsyncFileUtilTest, EnsureFileExists) { |
@@ -256,8 +256,8 @@ TEST_F(FileSystemProviderProviderAsyncFileUtilTest, EnsureFileExists) { |
file_url_, |
base::Bind(&EventLogger::OnEnsureFileExists, logger.GetWeakPtr())); |
- ASSERT_TRUE(logger.error()); |
- EXPECT_EQ(base::File::FILE_ERROR_ACCESS_DENIED, *logger.error()); |
+ ASSERT_TRUE(logger.result()); |
+ EXPECT_EQ(base::File::FILE_ERROR_ACCESS_DENIED, *logger.result()); |
} |
TEST_F(FileSystemProviderProviderAsyncFileUtilTest, CreateDirectory) { |
@@ -269,9 +269,10 @@ TEST_F(FileSystemProviderProviderAsyncFileUtilTest, CreateDirectory) { |
false, // exclusive |
false, // recursive |
base::Bind(&EventLogger::OnStatus, logger.GetWeakPtr())); |
+ base::RunLoop().RunUntilIdle(); |
- ASSERT_TRUE(logger.error()); |
- EXPECT_EQ(base::File::FILE_ERROR_ACCESS_DENIED, *logger.error()); |
+ ASSERT_TRUE(logger.result()); |
+ EXPECT_EQ(base::File::FILE_OK, *logger.result()); |
} |
TEST_F(FileSystemProviderProviderAsyncFileUtilTest, GetFileInfo) { |
@@ -283,8 +284,8 @@ TEST_F(FileSystemProviderProviderAsyncFileUtilTest, GetFileInfo) { |
base::Bind(&EventLogger::OnGetFileInfo, logger.GetWeakPtr())); |
base::RunLoop().RunUntilIdle(); |
- ASSERT_TRUE(logger.error()); |
- EXPECT_EQ(base::File::FILE_OK, *logger.error()); |
+ ASSERT_TRUE(logger.result()); |
+ EXPECT_EQ(base::File::FILE_OK, *logger.result()); |
} |
TEST_F(FileSystemProviderProviderAsyncFileUtilTest, ReadDirectory) { |
@@ -296,22 +297,22 @@ TEST_F(FileSystemProviderProviderAsyncFileUtilTest, ReadDirectory) { |
base::Bind(&EventLogger::OnReadDirectory, logger.GetWeakPtr())); |
base::RunLoop().RunUntilIdle(); |
- ASSERT_TRUE(logger.error()); |
- EXPECT_EQ(base::File::FILE_OK, *logger.error()); |
+ ASSERT_TRUE(logger.result()); |
+ EXPECT_EQ(base::File::FILE_OK, *logger.result()); |
} |
TEST_F(FileSystemProviderProviderAsyncFileUtilTest, Touch) { |
EventLogger logger; |
- async_file_util_->CreateDirectory( |
+ async_file_util_->Touch( |
CreateOperationContext(), |
file_url_, |
- false, // exclusive |
- false, // recursive |
+ base::Time(), // last_modified_time |
+ base::Time(), // last_access_time |
base::Bind(&EventLogger::OnStatus, logger.GetWeakPtr())); |
- ASSERT_TRUE(logger.error()); |
- EXPECT_EQ(base::File::FILE_ERROR_ACCESS_DENIED, *logger.error()); |
+ ASSERT_TRUE(logger.result()); |
+ EXPECT_EQ(base::File::FILE_ERROR_ACCESS_DENIED, *logger.result()); |
} |
TEST_F(FileSystemProviderProviderAsyncFileUtilTest, Truncate) { |
@@ -323,8 +324,8 @@ TEST_F(FileSystemProviderProviderAsyncFileUtilTest, Truncate) { |
0, // length |
base::Bind(&EventLogger::OnStatus, logger.GetWeakPtr())); |
- ASSERT_TRUE(logger.error()); |
- EXPECT_EQ(base::File::FILE_ERROR_ACCESS_DENIED, *logger.error()); |
+ ASSERT_TRUE(logger.result()); |
+ EXPECT_EQ(base::File::FILE_ERROR_ACCESS_DENIED, *logger.result()); |
} |
TEST_F(FileSystemProviderProviderAsyncFileUtilTest, CopyFileLocal) { |
@@ -338,8 +339,8 @@ TEST_F(FileSystemProviderProviderAsyncFileUtilTest, CopyFileLocal) { |
base::Bind(&EventLogger::OnCopyFileProgress, logger.GetWeakPtr()), |
base::Bind(&EventLogger::OnStatus, logger.GetWeakPtr())); |
- ASSERT_TRUE(logger.error()); |
- EXPECT_EQ(base::File::FILE_ERROR_ACCESS_DENIED, *logger.error()); |
+ ASSERT_TRUE(logger.result()); |
+ EXPECT_EQ(base::File::FILE_ERROR_ACCESS_DENIED, *logger.result()); |
} |
TEST_F(FileSystemProviderProviderAsyncFileUtilTest, MoveFileLocal) { |
@@ -352,8 +353,8 @@ TEST_F(FileSystemProviderProviderAsyncFileUtilTest, MoveFileLocal) { |
fileapi::FileSystemOperation::OPTION_NONE, |
base::Bind(&EventLogger::OnStatus, logger.GetWeakPtr())); |
- ASSERT_TRUE(logger.error()); |
- EXPECT_EQ(base::File::FILE_ERROR_ACCESS_DENIED, *logger.error()); |
+ ASSERT_TRUE(logger.result()); |
+ EXPECT_EQ(base::File::FILE_ERROR_ACCESS_DENIED, *logger.result()); |
} |
TEST_F(FileSystemProviderProviderAsyncFileUtilTest, CopyInForeignFile) { |
@@ -365,8 +366,8 @@ TEST_F(FileSystemProviderProviderAsyncFileUtilTest, CopyInForeignFile) { |
file_url_, // dst_url |
base::Bind(&EventLogger::OnStatus, logger.GetWeakPtr())); |
- ASSERT_TRUE(logger.error()); |
- EXPECT_EQ(base::File::FILE_ERROR_ACCESS_DENIED, *logger.error()); |
+ ASSERT_TRUE(logger.result()); |
+ EXPECT_EQ(base::File::FILE_ERROR_ACCESS_DENIED, *logger.result()); |
} |
TEST_F(FileSystemProviderProviderAsyncFileUtilTest, DeleteFile) { |
@@ -377,8 +378,8 @@ TEST_F(FileSystemProviderProviderAsyncFileUtilTest, DeleteFile) { |
file_url_, |
base::Bind(&EventLogger::OnStatus, logger.GetWeakPtr())); |
- ASSERT_TRUE(logger.error()); |
- EXPECT_EQ(base::File::FILE_ERROR_ACCESS_DENIED, *logger.error()); |
+ ASSERT_TRUE(logger.result()); |
+ EXPECT_EQ(base::File::FILE_ERROR_ACCESS_DENIED, *logger.result()); |
} |
TEST_F(FileSystemProviderProviderAsyncFileUtilTest, DeleteDirectory) { |
@@ -389,8 +390,8 @@ TEST_F(FileSystemProviderProviderAsyncFileUtilTest, DeleteDirectory) { |
directory_url_, |
base::Bind(&EventLogger::OnStatus, logger.GetWeakPtr())); |
- ASSERT_TRUE(logger.error()); |
- EXPECT_EQ(base::File::FILE_ERROR_ACCESS_DENIED, *logger.error()); |
+ ASSERT_TRUE(logger.result()); |
+ EXPECT_EQ(base::File::FILE_ERROR_ACCESS_DENIED, *logger.result()); |
} |
TEST_F(FileSystemProviderProviderAsyncFileUtilTest, DeleteRecursively) { |
@@ -401,8 +402,8 @@ TEST_F(FileSystemProviderProviderAsyncFileUtilTest, DeleteRecursively) { |
directory_url_, |
base::Bind(&EventLogger::OnStatus, logger.GetWeakPtr())); |
- ASSERT_TRUE(logger.error()); |
- EXPECT_EQ(base::File::FILE_ERROR_ACCESS_DENIED, *logger.error()); |
+ ASSERT_TRUE(logger.result()); |
+ EXPECT_EQ(base::File::FILE_ERROR_ACCESS_DENIED, *logger.result()); |
} |
TEST_F(FileSystemProviderProviderAsyncFileUtilTest, CreateSnapshotFile) { |
@@ -413,8 +414,8 @@ TEST_F(FileSystemProviderProviderAsyncFileUtilTest, CreateSnapshotFile) { |
file_url_, |
base::Bind(&EventLogger::OnCreateSnapshotFile, logger.GetWeakPtr())); |
- ASSERT_TRUE(logger.error()); |
- EXPECT_EQ(base::File::FILE_ERROR_INVALID_OPERATION, *logger.error()); |
+ ASSERT_TRUE(logger.result()); |
+ EXPECT_EQ(base::File::FILE_ERROR_INVALID_OPERATION, *logger.result()); |
} |
} // namespace file_system_provider |