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

Unified Diff: components/sync/driver/glue/sync_backend_host_impl.h

Issue 2284283002: Remove stl_util's STLElementDeleter from sync. (Closed)
Patch Set: Created 4 years, 4 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
Index: components/sync/driver/glue/sync_backend_host_impl.h
diff --git a/components/sync/driver/glue/sync_backend_host_impl.h b/components/sync/driver/glue/sync_backend_host_impl.h
index b7e0253620249b074e903e8ba48d2c9e879fc14e..73b70199c3a4b674bd2364eb531e238ce07cfd36 100644
--- a/components/sync/driver/glue/sync_backend_host_impl.h
+++ b/components/sync/driver/glue/sync_backend_host_impl.h
@@ -143,7 +143,8 @@ class SyncBackendHostImpl : public SyncBackendHost,
void GetAllNodesForTypes(
syncer::ModelTypeSet types,
base::Callback<void(const std::vector<syncer::ModelType>&,
- ScopedVector<base::ListValue>)> type) override;
+ std::vector<std::unique_ptr<base::ListValue>>)> type)
+ override;
base::MessageLoop* GetSyncLoopForTesting() override;
void RefreshTypesForTest(syncer::ModelTypeSet types) override;
void ClearServerData(
@@ -200,7 +201,8 @@ class SyncBackendHostImpl : public SyncBackendHost,
// Forwards a ProtocolEvent to the frontend. Will not be called unless a
// call to SetForwardProtocolEvents() explicitly requested that we start
// forwarding these events.
- void HandleProtocolEventOnFrontendLoop(syncer::ProtocolEvent* event);
+ void HandleProtocolEventOnFrontendLoop(
+ std::unique_ptr<syncer::ProtocolEvent> event);
// Forwards a directory commit counter update to the frontend loop. Will not
// be called unless a call to EnableDirectoryTypeDebugInfoForwarding()
« no previous file with comments | « components/sync/driver/glue/sync_backend_host_core.cc ('k') | components/sync/driver/glue/sync_backend_host_impl.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698