Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(83)

Unified Diff: chrome/browser/sync_file_system/drive_backend/metadata_database.cc

Issue 1545283002: Convert Pass()→std::move() in //chrome/browser/ui (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 5 years ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
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) {

Powered by Google App Engine
This is Rietveld 408576698