| Index: chrome/browser/sync_file_system/local_file_sync_service_unittest.cc
|
| diff --git a/chrome/browser/sync_file_system/local_file_sync_service_unittest.cc b/chrome/browser/sync_file_system/local_file_sync_service_unittest.cc
|
| index fd0dbbeffc39707a4eaa2b6e10ca37f632e7bc66..80b438b106dee09eadbfb28de6b6fb1648afeb2b 100644
|
| --- a/chrome/browser/sync_file_system/local_file_sync_service_unittest.cc
|
| +++ b/chrome/browser/sync_file_system/local_file_sync_service_unittest.cc
|
| @@ -89,12 +89,12 @@ void OnGetFileMetadata(const tracked_objects::Location& where,
|
|
|
| ACTION_P(MockStatusCallback, status) {
|
| base::MessageLoopProxy::current()->PostTask(
|
| - FROM_HERE, base::Bind(arg4, status));
|
| + FROM_HERE, base::Bind(arg3, status));
|
| }
|
|
|
| ACTION_P2(MockStatusCallbackAndRecordChange, status, changes) {
|
| base::MessageLoopProxy::current()->PostTask(
|
| - FROM_HERE, base::Bind(arg4, status));
|
| + FROM_HERE, base::Bind(arg3, status));
|
| changes->push_back(arg0);
|
| }
|
|
|
| @@ -329,27 +329,19 @@ TEST_F(LocalFileSyncServiceTest, ProcessLocalChange_CreateFile) {
|
| EXPECT_EQ(kTestFileDataSize,
|
| file_system_->WriteString(kFile, std::string(kTestFileData)));
|
|
|
| - // Retrieve the file metadata to set up the expected values.
|
| + // Retrieve the expected platform_path.
|
| base::PlatformFileInfo info;
|
| FilePath platform_path;
|
| EXPECT_EQ(base::PLATFORM_FILE_OK,
|
| file_system_->GetMetadata(kFile, &info, &platform_path));
|
|
|
| - ASSERT_FALSE(info.is_directory);
|
| - ASSERT_EQ(kTestFileDataSize, info.size);
|
| -
|
| - fileapi::SyncFileMetadata metadata;
|
| - metadata.file_type = fileapi::SYNC_FILE_TYPE_FILE;
|
| - metadata.size = info.size;
|
| - metadata.last_modified = info.last_modified;
|
| -
|
| // The local_change_processor's ApplyLocalChange should be called once
|
| // with ADD_OR_UPDATE change for TYPE_FILE.
|
| StrictMock<MockLocalChangeProcessor> local_change_processor;
|
| const FileChange change(FileChange::FILE_CHANGE_ADD_OR_UPDATE,
|
| fileapi::SYNC_FILE_TYPE_FILE);
|
| EXPECT_CALL(local_change_processor,
|
| - ApplyLocalChange(change, platform_path, metadata, kFile, _))
|
| + ApplyLocalChange(change, platform_path, kFile, _))
|
| .WillOnce(MockStatusCallback(fileapi::SYNC_STATUS_OK));
|
|
|
| local_service_->ProcessLocalChange(
|
| @@ -386,7 +378,7 @@ TEST_F(LocalFileSyncServiceTest, ProcessLocalChange_CreateAndRemoveFile) {
|
| StrictMock<MockLocalChangeProcessor> local_change_processor;
|
| const FileChange change(FileChange::FILE_CHANGE_DELETE,
|
| fileapi::SYNC_FILE_TYPE_FILE);
|
| - EXPECT_CALL(local_change_processor, ApplyLocalChange(change, _, _, kFile, _))
|
| + EXPECT_CALL(local_change_processor, ApplyLocalChange(change, _, kFile, _))
|
| .WillOnce(MockStatusCallback(fileapi::SYNC_FILE_ERROR_NOT_FOUND));
|
|
|
| // The sync should succeed anyway.
|
| @@ -457,7 +449,7 @@ TEST_F(LocalFileSyncServiceTest, ProcessLocalChange_MultipleChanges) {
|
| // twice for FILE_TYPE and FILE_DIRECTORY.
|
| StrictMock<MockLocalChangeProcessor> local_change_processor;
|
| std::vector<FileChange> changes;
|
| - EXPECT_CALL(local_change_processor, ApplyLocalChange(_, _, _, kPath, _))
|
| + EXPECT_CALL(local_change_processor, ApplyLocalChange(_, _, kPath, _))
|
| .Times(2)
|
| .WillOnce(MockStatusCallbackAndRecordChange(fileapi::SYNC_STATUS_OK,
|
| &changes))
|
| @@ -546,7 +538,7 @@ TEST_F(LocalFileSyncServiceTest, RecordFakeChange) {
|
| // Next local sync should pick up the recorded change.
|
| StrictMock<MockLocalChangeProcessor> local_change_processor;
|
| std::vector<FileChange> changes;
|
| - EXPECT_CALL(local_change_processor, ApplyLocalChange(_, _, _, kURL, _))
|
| + EXPECT_CALL(local_change_processor, ApplyLocalChange(_, _, kURL, _))
|
| .WillOnce(MockStatusCallbackAndRecordChange(fileapi::SYNC_STATUS_OK,
|
| &changes));
|
| {
|
|
|