Index: chrome/browser/sync_file_system/drive_backend/metadata_database.cc |
diff --git a/chrome/browser/sync_file_system/drive_backend/metadata_database.cc b/chrome/browser/sync_file_system/drive_backend/metadata_database.cc |
index 02561d0494c1d345f1fe1445542f1aad580716e6..cf97ac886898fbbdcf2283b244c85454a028c996 100644 |
--- a/chrome/browser/sync_file_system/drive_backend/metadata_database.cc |
+++ b/chrome/browser/sync_file_system/drive_backend/metadata_database.cc |
@@ -6,6 +6,7 @@ |
#include <algorithm> |
#include <stack> |
+#include <utility> |
#include "base/bind.h" |
#include "base/command_line.h" |
@@ -123,11 +124,11 @@ scoped_ptr<FileMetadata> CreateFileMetadataFromFileResource( |
if (resource.labels().is_trashed()) { |
details->set_missing(true); |
- return file.Pass(); |
+ return file; |
} |
PopulateFileDetailsByFileResource(resource, details); |
- return file.Pass(); |
+ return file; |
} |
scoped_ptr<FileMetadata> CreateFileMetadataFromChangeResource( |
@@ -140,11 +141,11 @@ scoped_ptr<FileMetadata> CreateFileMetadataFromChangeResource( |
if (change.is_deleted()) { |
details->set_missing(true); |
- return file.Pass(); |
+ return file; |
} |
PopulateFileDetailsByFileResource(*change.file(), details); |
- return file.Pass(); |
+ return file; |
} |
scoped_ptr<FileMetadata> CreateDeletedFileMetadata(int64_t change_id, |
@@ -155,7 +156,7 @@ scoped_ptr<FileMetadata> CreateDeletedFileMetadata(int64_t change_id, |
FileDetails* details = file->mutable_details(); |
details->set_change_id(change_id); |
details->set_missing(true); |
- return file.Pass(); |
+ return file; |
} |
scoped_ptr<FileTracker> CreateSyncRootTracker( |
@@ -170,7 +171,7 @@ scoped_ptr<FileTracker> CreateSyncRootTracker( |
sync_root_tracker->set_active(true); |
sync_root_tracker->set_needs_folder_listing(false); |
*sync_root_tracker->mutable_synced_details() = sync_root_metadata.details(); |
- return sync_root_tracker.Pass(); |
+ return sync_root_tracker; |
} |
scoped_ptr<FileTracker> CreateInitialAppRootTracker( |
@@ -186,7 +187,7 @@ scoped_ptr<FileTracker> CreateInitialAppRootTracker( |
app_root_tracker->set_active(false); |
app_root_tracker->set_needs_folder_listing(false); |
*app_root_tracker->mutable_synced_details() = app_root_metadata.details(); |
- return app_root_tracker.Pass(); |
+ return app_root_tracker; |
} |
scoped_ptr<FileTracker> CloneFileTracker(const FileTracker* obj) { |
@@ -290,7 +291,7 @@ void MarkTrackerSetDirty(const TrackerIDSet& trackers, |
if (tracker->dirty()) |
continue; |
tracker->set_dirty(true); |
- index->StoreFileTracker(tracker.Pass()); |
+ index->StoreFileTracker(std::move(tracker)); |
} |
} |
@@ -322,7 +323,7 @@ void MarkTrackersDirtyRecursively(int64_t root_tracker_id, |
index->GetFileTracker(tracker_id, tracker.get()); |
tracker->set_dirty(true); |
- index->StoreFileTracker(tracker.Pass()); |
+ index->StoreFileTracker(std::move(tracker)); |
} |
} |
@@ -470,7 +471,7 @@ void ActivateFileTracker(int64_t tracker_id, |
tracker->set_needs_folder_listing(false); |
} |
- index->StoreFileTracker(tracker.Pass()); |
+ index->StoreFileTracker(std::move(tracker)); |
} |
void DeactivateFileTracker(int64_t tracker_id, |
@@ -490,7 +491,7 @@ void DeactivateFileTracker(int64_t tracker_id, |
tracker->set_dirty(dirtying_options & MARK_ITSELF_DIRTY); |
tracker->set_active(false); |
- index->StoreFileTracker(tracker.Pass()); |
+ index->StoreFileTracker(std::move(tracker)); |
} |
void RemoveFileTracker(int64_t tracker_id, |
@@ -561,7 +562,7 @@ scoped_ptr<MetadataDatabase> MetadataDatabase::CreateInternal( |
metadata_database.reset(); |
*status_out = status; |
- return metadata_database.Pass(); |
+ return metadata_database; |
} |
// static |
@@ -573,10 +574,10 @@ SyncStatusCode MetadataDatabase::CreateForTesting( |
new MetadataDatabase(base::FilePath(), |
enable_on_disk_index, |
nullptr)); |
- metadata_database->db_ = db.Pass(); |
+ metadata_database->db_ = std::move(db); |
SyncStatusCode status = metadata_database->Initialize(); |
if (status == SYNC_STATUS_OK) |
- *metadata_database_out = metadata_database.Pass(); |
+ *metadata_database_out = std::move(metadata_database); |
return status; |
} |
@@ -688,7 +689,7 @@ SyncStatusCode MetadataDatabase::RegisterApp(const std::string& app_id, |
tracker->set_needs_folder_listing(true); |
tracker->set_dirty(true); |
- index_->StoreFileTracker(tracker.Pass()); |
+ index_->StoreFileTracker(std::move(tracker)); |
return WriteToDatabase(); |
} |
@@ -710,7 +711,7 @@ SyncStatusCode MetadataDatabase::DisableApp(const std::string& app_id) { |
// other conflicting trackers won't become active. |
tracker->set_tracker_kind(TRACKER_KIND_DISABLED_APP_ROOT); |
- index_->StoreFileTracker(tracker.Pass()); |
+ index_->StoreFileTracker(std::move(tracker)); |
return WriteToDatabase(); |
} |
@@ -729,7 +730,7 @@ SyncStatusCode MetadataDatabase::EnableApp(const std::string& app_id) { |
DCHECK(tracker->active()); |
tracker->set_tracker_kind(TRACKER_KIND_APP_ROOT); |
- index_->StoreFileTracker(tracker.Pass()); |
+ index_->StoreFileTracker(std::move(tracker)); |
MarkTrackersDirtyRecursively(tracker_id, index_.get()); |
return WriteToDatabase(); |
@@ -750,7 +751,7 @@ SyncStatusCode MetadataDatabase::UnregisterApp(const std::string& app_id) { |
tracker->set_active(false); |
tracker->set_dirty(true); |
- index_->StoreFileTracker(tracker.Pass()); |
+ index_->StoreFileTracker(std::move(tracker)); |
return WriteToDatabase(); |
} |
@@ -900,7 +901,7 @@ SyncStatusCode MetadataDatabase::UpdateByChangeList( |
scoped_ptr<FileMetadata> metadata( |
CreateFileMetadataFromChangeResource(change)); |
- UpdateByFileMetadata(FROM_HERE, metadata.Pass(), |
+ UpdateByFileMetadata(FROM_HERE, std::move(metadata), |
UPDATE_TRACKER_FOR_UNSYNCED_FILE); |
} |
@@ -914,7 +915,7 @@ SyncStatusCode MetadataDatabase::UpdateByFileResource( |
scoped_ptr<FileMetadata> metadata( |
CreateFileMetadataFromFileResource( |
GetLargestKnownChangeID(), resource)); |
- UpdateByFileMetadata(FROM_HERE, metadata.Pass(), |
+ UpdateByFileMetadata(FROM_HERE, std::move(metadata), |
UPDATE_TRACKER_FOR_UNSYNCED_FILE); |
return WriteToDatabase(); |
} |
@@ -925,7 +926,7 @@ SyncStatusCode MetadataDatabase::UpdateByFileResourceList( |
scoped_ptr<FileMetadata> metadata( |
CreateFileMetadataFromFileResource( |
GetLargestKnownChangeID(), *resources[i])); |
- UpdateByFileMetadata(FROM_HERE, metadata.Pass(), |
+ UpdateByFileMetadata(FROM_HERE, std::move(metadata), |
UPDATE_TRACKER_FOR_UNSYNCED_FILE); |
} |
return WriteToDatabase(); |
@@ -935,7 +936,7 @@ SyncStatusCode MetadataDatabase::UpdateByDeletedRemoteFile( |
const std::string& file_id) { |
scoped_ptr<FileMetadata> metadata( |
CreateDeletedFileMetadata(GetLargestKnownChangeID(), file_id)); |
- UpdateByFileMetadata(FROM_HERE, metadata.Pass(), |
+ UpdateByFileMetadata(FROM_HERE, std::move(metadata), |
UPDATE_TRACKER_FOR_UNSYNCED_FILE); |
return WriteToDatabase(); |
} |
@@ -946,7 +947,7 @@ SyncStatusCode MetadataDatabase::UpdateByDeletedRemoteFileList( |
itr != file_ids.end(); ++itr) { |
scoped_ptr<FileMetadata> metadata( |
CreateDeletedFileMetadata(GetLargestKnownChangeID(), *itr)); |
- UpdateByFileMetadata(FROM_HERE, metadata.Pass(), |
+ UpdateByFileMetadata(FROM_HERE, std::move(metadata), |
UPDATE_TRACKER_FOR_UNSYNCED_FILE); |
} |
return WriteToDatabase(); |
@@ -1025,7 +1026,7 @@ SyncStatusCode MetadataDatabase::PopulateFolderByChildList( |
folder_tracker->set_needs_folder_listing(false); |
if (folder_tracker->dirty() && !ShouldKeepDirty(*folder_tracker)) |
folder_tracker->set_dirty(false); |
- index_->StoreFileTracker(folder_tracker.Pass()); |
+ index_->StoreFileTracker(std::move(folder_tracker)); |
return WriteToDatabase(); |
} |
@@ -1118,7 +1119,7 @@ SyncStatusCode MetadataDatabase::UpdateTracker( |
} else if (tracker.dirty() && !ShouldKeepDirty(tracker)) { |
updated_tracker->set_dirty(false); |
} |
- index_->StoreFileTracker(updated_tracker.Pass()); |
+ index_->StoreFileTracker(std::move(updated_tracker)); |
if (should_promote) |
index_->PromoteDemotedDirtyTracker(tracker_id); |
@@ -1166,7 +1167,7 @@ MetadataDatabase::ActivationStatus MetadataDatabase::TryActivateTracker( |
tracker_to_be_deactivated.get())) { |
const std::string file_id = tracker_to_be_deactivated->file_id(); |
tracker_to_be_deactivated->set_active(false); |
- index_->StoreFileTracker(tracker_to_be_deactivated.Pass()); |
+ index_->StoreFileTracker(std::move(tracker_to_be_deactivated)); |
MarkTrackersDirtyByFileID(file_id, index_.get()); |
} else { |
@@ -1184,7 +1185,7 @@ MetadataDatabase::ActivationStatus MetadataDatabase::TryActivateTracker( |
} |
tracker_to_be_activated->set_dirty(false); |
- index_->StoreFileTracker(tracker_to_be_activated.Pass()); |
+ index_->StoreFileTracker(std::move(tracker_to_be_activated)); |
*status_out = WriteToDatabase(); |
return ACTIVATION_PENDING; |
@@ -1302,7 +1303,7 @@ SyncStatusCode MetadataDatabase::SweepDirtyTrackers( |
!CanClearDirty(*tracker)) |
continue; |
tracker->set_dirty(false); |
- index_->StoreFileTracker(tracker.Pass()); |
+ index_->StoreFileTracker(std::move(tracker)); |
} |
return WriteToDatabase(); |
@@ -1399,7 +1400,7 @@ void MetadataDatabase::CreateTrackerInternal(const FileTracker& parent_tracker, |
tracker->mutable_synced_details()->clear_md5(); |
} |
} |
- index_->StoreFileTracker(tracker.Pass()); |
+ index_->StoreFileTracker(std::move(tracker)); |
} |
void MetadataDatabase::MaybeAddTrackersForNewFile( |
@@ -1571,7 +1572,7 @@ void MetadataDatabase::UpdateByFileMetadata( |
TrackerIDSet trackers = index_->GetFileTrackerIDsByFileID(file_id); |
if (!trackers.empty()) { |
- index_->StoreFileMetadata(metadata.Pass()); |
+ index_->StoreFileMetadata(std::move(metadata)); |
if (option != UPDATE_TRACKER_FOR_SYNCED_FILE) |
MarkTrackerSetDirty(trackers, index_.get()); |
@@ -1589,7 +1590,7 @@ scoped_ptr<base::ListValue> MetadataDatabase::DumpFiles( |
FileTracker app_root_tracker; |
if (!FindAppRootTracker(app_id, &app_root_tracker)) |
- return files.Pass(); |
+ return files; |
std::vector<int64_t> stack; |
AppendContents( |
@@ -1627,14 +1628,14 @@ scoped_ptr<base::ListValue> MetadataDatabase::DumpFiles( |
files->Append(file); |
} |
- return files.Pass(); |
+ return files; |
} |
scoped_ptr<base::ListValue> MetadataDatabase::DumpDatabase() { |
scoped_ptr<base::ListValue> list(new base::ListValue); |
list->Append(DumpTrackers().release()); |
list->Append(DumpMetadata().release()); |
- return list.Pass(); |
+ return list; |
} |
bool MetadataDatabase::HasNewerFileMetadata(const std::string& file_id, |
@@ -1706,7 +1707,7 @@ scoped_ptr<base::ListValue> MetadataDatabase::DumpTrackers() { |
} |
trackers->Append(dict); |
} |
- return trackers.Pass(); |
+ return trackers; |
} |
scoped_ptr<base::ListValue> MetadataDatabase::DumpMetadata() { |
@@ -1755,7 +1756,7 @@ scoped_ptr<base::ListValue> MetadataDatabase::DumpMetadata() { |
} |
files->Append(dict); |
} |
- return files.Pass(); |
+ return files; |
} |
void MetadataDatabase::AttachSyncRoot( |
@@ -1767,8 +1768,8 @@ void MetadataDatabase::AttachSyncRoot( |
CreateSyncRootTracker(IncrementTrackerID(), *sync_root_metadata); |
index_->SetSyncRootTrackerID(sync_root_tracker->tracker_id()); |
- index_->StoreFileMetadata(sync_root_metadata.Pass()); |
- index_->StoreFileTracker(sync_root_tracker.Pass()); |
+ index_->StoreFileMetadata(std::move(sync_root_metadata)); |
+ index_->StoreFileTracker(std::move(sync_root_tracker)); |
} |
void MetadataDatabase::AttachInitialAppRoot( |
@@ -1781,8 +1782,8 @@ void MetadataDatabase::AttachInitialAppRoot( |
GetSyncRootTrackerID(), |
*app_root_metadata); |
- index_->StoreFileMetadata(app_root_metadata.Pass()); |
- index_->StoreFileTracker(app_root_tracker.Pass()); |
+ index_->StoreFileMetadata(std::move(app_root_metadata)); |
+ index_->StoreFileTracker(std::move(app_root_tracker)); |
} |
bool MetadataDatabase::CanClearDirty(const FileTracker& tracker) { |