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

Unified Diff: chrome/browser/sync_file_system/drive_file_sync_service.cc

Issue 15410005: Deprecate DriveMetadataStore.batch_sync_origins (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Small test fixes. Created 7 years, 7 months 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
« no previous file with comments | « no previous file | chrome/browser/sync_file_system/drive_metadata_store.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/sync_file_system/drive_file_sync_service.cc
diff --git a/chrome/browser/sync_file_system/drive_file_sync_service.cc b/chrome/browser/sync_file_system/drive_file_sync_service.cc
index 2f6031b186694432856d4c7a6fafe6ad3637cd7c..e7094d46624373bc4ce0c4d91728d777826c51e1 100644
--- a/chrome/browser/sync_file_system/drive_file_sync_service.cc
+++ b/chrome/browser/sync_file_system/drive_file_sync_service.cc
@@ -381,19 +381,6 @@ void DriveFileSyncService::DidInitializeMetadataStore(
return;
}
- // TODO(calvinlo): Move the code to delete legacy batch_sync_origin keys from
- // DB into DriveMetadataStore.InitializeDBOnFileThread.
- std::vector<GURL> batch_origins_to_delete;
- typedef std::map<GURL, std::string>::const_iterator origin_itr;
- for (origin_itr itr = metadata_store_->batch_sync_origins().begin();
- itr != metadata_store_->batch_sync_origins().end(); ++itr) {
- batch_origins_to_delete.push_back(itr->first);
- }
- for (std::vector<GURL>::const_iterator itr = batch_origins_to_delete.begin();
- itr != batch_origins_to_delete.end(); ++itr) {
- metadata_store_->RemoveOrigin(*itr, base::Bind(&EmptyStatusCallback));
- }
-
DCHECK(pending_batch_sync_origins_.empty());
UpdateRegisteredOrigins();
@@ -615,8 +602,7 @@ void DriveFileSyncService::DoApplyLocalChange(
return;
}
- if (!metadata_store_->IsIncrementalSyncOrigin(url.origin()) &&
- !metadata_store_->IsBatchSyncOrigin(url.origin())) {
+ if (!metadata_store_->IsIncrementalSyncOrigin(url.origin())) {
// We may get called by LocalFileSyncService to sync local changes
// for the origins that are disabled.
DVLOG(1) << "Got request for stray origin: " << url.origin().spec();
@@ -636,7 +622,6 @@ void DriveFileSyncService::UpdateRegisteredOrigins() {
ExtensionService* extension_service =
extensions::ExtensionSystem::Get(profile_)->extension_service();
DCHECK(pending_batch_sync_origins_.empty());
- DCHECK(metadata_store_->batch_sync_origins().empty());
if (!extension_service)
return;
@@ -1653,8 +1638,7 @@ bool DriveFileSyncService::GetOriginForEntry(
}
DCHECK(origin.is_valid());
- if (!metadata_store_->IsBatchSyncOrigin(origin) &&
- !metadata_store_->IsIncrementalSyncOrigin(origin))
+ if (!metadata_store_->IsIncrementalSyncOrigin(origin))
continue;
std::string resource_id(metadata_store_->GetResourceIdForOrigin(origin));
if (resource_id.empty())
« no previous file with comments | « no previous file | chrome/browser/sync_file_system/drive_metadata_store.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698