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

Side by Side Diff: components/sync_driver/fake_generic_change_processor.cc

Issue 548373003: Move AttachmentStore ownership to datatype (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Don't create AttachmentService in GenericChangeProcessor if AttachmentStore is NULL Created 6 years, 3 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 unified diff | Download patch
OLDNEW
1 // Copyright 2014 The Chromium Authors. All rights reserved. 1 // Copyright 2014 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "components/sync_driver/fake_generic_change_processor.h" 5 #include "components/sync_driver/fake_generic_change_processor.h"
6 6
7 #include "base/location.h" 7 #include "base/location.h"
8 #include "base/memory/weak_ptr.h" 8 #include "base/memory/weak_ptr.h"
9 #include "sync/api/syncable_service.h" 9 #include "sync/api/syncable_service.h"
10 #include "sync/internal_api/public/attachments/attachment_service_impl.h" 10 #include "sync/internal_api/public/attachments/attachment_service_impl.h"
11 11
12 namespace sync_driver { 12 namespace sync_driver {
13 13
14 FakeGenericChangeProcessor::FakeGenericChangeProcessor( 14 FakeGenericChangeProcessor::FakeGenericChangeProcessor(
15 SyncApiComponentFactory* sync_factory) 15 SyncApiComponentFactory* sync_factory)
16 : GenericChangeProcessor(NULL, 16 : GenericChangeProcessor(NULL,
17 base::WeakPtr<syncer::SyncableService>(), 17 base::WeakPtr<syncer::SyncableService>(),
18 base::WeakPtr<syncer::SyncMergeResult>(), 18 base::WeakPtr<syncer::SyncMergeResult>(),
19 NULL, 19 NULL,
20 sync_factory), 20 sync_factory,
21 scoped_refptr<syncer::AttachmentStore>()),
21 sync_model_has_user_created_nodes_(true), 22 sync_model_has_user_created_nodes_(true),
22 sync_model_has_user_created_nodes_success_(true) {} 23 sync_model_has_user_created_nodes_success_(true) {}
23 24
24 FakeGenericChangeProcessor::~FakeGenericChangeProcessor() {} 25 FakeGenericChangeProcessor::~FakeGenericChangeProcessor() {}
25 26
26 void FakeGenericChangeProcessor::set_sync_model_has_user_created_nodes( 27 void FakeGenericChangeProcessor::set_sync_model_has_user_created_nodes(
27 bool has_nodes) { 28 bool has_nodes) {
28 sync_model_has_user_created_nodes_ = has_nodes; 29 sync_model_has_user_created_nodes_ = has_nodes;
29 } 30 }
30 void FakeGenericChangeProcessor::set_sync_model_has_user_created_nodes_success( 31 void FakeGenericChangeProcessor::set_sync_model_has_user_created_nodes_success(
(...skipping 43 matching lines...) Expand 10 before | Expand all | Expand 10 after
74 FakeGenericChangeProcessorFactory::CreateGenericChangeProcessor( 75 FakeGenericChangeProcessorFactory::CreateGenericChangeProcessor(
75 syncer::UserShare* user_share, 76 syncer::UserShare* user_share,
76 DataTypeErrorHandler* error_handler, 77 DataTypeErrorHandler* error_handler,
77 const base::WeakPtr<syncer::SyncableService>& local_service, 78 const base::WeakPtr<syncer::SyncableService>& local_service,
78 const base::WeakPtr<syncer::SyncMergeResult>& merge_result, 79 const base::WeakPtr<syncer::SyncMergeResult>& merge_result,
79 SyncApiComponentFactory* sync_factory) { 80 SyncApiComponentFactory* sync_factory) {
80 return processor_.PassAs<GenericChangeProcessor>(); 81 return processor_.PassAs<GenericChangeProcessor>();
81 } 82 }
82 83
83 } // namespace sync_driver 84 } // namespace sync_driver
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698