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

Unified Diff: chrome/browser/sync/glue/generic_change_processor.cc

Issue 213003004: Replace calls to 3-arg SyncData::CreateLocalData with 5-arg version. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@syncapi
Patch Set: Forgot a class keyword. Created 6 years, 9 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: chrome/browser/sync/glue/generic_change_processor.cc
diff --git a/chrome/browser/sync/glue/generic_change_processor.cc b/chrome/browser/sync/glue/generic_change_processor.cc
index f90365d9fb8bd7132b13c78a60d412d861030c23..d7bfeb815dd3b9332659b579d9c820bc118bf64e 100644
--- a/chrome/browser/sync/glue/generic_change_processor.cc
+++ b/chrome/browser/sync/glue/generic_change_processor.cc
@@ -40,7 +40,8 @@ void SetNodeSpecifics(const sync_pb::EntitySpecifics& entity_specifics,
syncer::SyncData BuildRemoteSyncData(
int64 sync_id,
const syncer::BaseNode& read_node,
- const syncer::AttachmentServiceProxy& attachment_service_proxy) {
+ const scoped_refptr<syncer::AttachmentServiceProxy>&
+ attachment_service_proxy) {
const syncer::AttachmentIdList& attachment_ids = read_node.GetAttachmentIds();
// Use the specifics of non-password datatypes directly (encryption has
// already been handled).
@@ -78,7 +79,7 @@ GenericChangeProcessor::GenericChangeProcessor(
share_handle_(user_share),
attachment_service_(attachment_service.Pass()),
attachment_service_weak_ptr_factory_(attachment_service_.get()),
- attachment_service_proxy_(syncer::AttachmentServiceProxy(
+ attachment_service_proxy_(new syncer::AttachmentServiceProxy(
base::MessageLoopProxy::current(),
attachment_service_weak_ptr_factory_.GetWeakPtr())) {
DCHECK(CalledOnValidThread());

Powered by Google App Engine
This is Rietveld 408576698