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

Unified Diff: components/sync/syncable/directory.h

Issue 2563883006: [Sync] Separate purge types step from sync manager configuration. (Closed)
Patch Set: Created 4 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
« no previous file with comments | « components/sync/engine_impl/sync_manager_impl_unittest.cc ('k') | components/sync/syncable/directory.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: components/sync/syncable/directory.h
diff --git a/components/sync/syncable/directory.h b/components/sync/syncable/directory.h
index 461892b9d2af02b65106e9d261a08925b912433e..d3bd42938b6362149a7ecc5ff1acb7e163a5ebfc 100644
--- a/components/sync/syncable/directory.h
+++ b/components/sync/syncable/directory.h
@@ -464,11 +464,9 @@ class Directory {
// Note: "Purge" is just meant to distinguish from "deleting" entries, which
// means something different in the syncable namespace.
// WARNING! This can be real slow, as it iterates over all entries.
- // WARNING! Performs synchronous I/O.
- // Returns: true on success, false if an error was encountered.
- virtual bool PurgeEntriesWithTypeIn(ModelTypeSet disabled_types,
- ModelTypeSet types_to_journal,
- ModelTypeSet types_to_unapply);
+ void PurgeEntriesWithTypeIn(ModelTypeSet disabled_types,
+ ModelTypeSet types_to_journal,
+ ModelTypeSet types_to_unapply);
// Resets the base_versions and server_versions of all synced entities
// associated with |type| to 1.
« no previous file with comments | « components/sync/engine_impl/sync_manager_impl_unittest.cc ('k') | components/sync/syncable/directory.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698