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

Side by Side Diff: sync/api/model_type_service.cc

Issue 1763953002: [USS] Change the place where SharedModelTypeProcessor got created (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: update base on Max and Sky's comment Created 4 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 unified diff | Download patch
OLDNEW
1 // Copyright 2015 The Chromium Authors. All rights reserved. 1 // Copyright 2015 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 "sync/api/model_type_service.h" 5 #include "sync/api/model_type_service.h"
6 6
7 #include "sync/internal_api/public/shared_model_type_processor.h"
skym 2016/03/24 16:08:52 Replace this with correct include.
Gang Wu 2016/03/24 22:18:11 Done.
8
7 namespace syncer_v2 { 9 namespace syncer_v2 {
8 10
9 ModelTypeService::ModelTypeService() {} 11 ModelTypeService::ModelTypeService(
12 const ChangeProcessorFactory& change_processor_factory)
13 : change_processor_factory_(change_processor_factory) {}
10 14
11 ModelTypeService::~ModelTypeService() {} 15 ModelTypeService::~ModelTypeService() {}
12 16
13 ModelTypeChangeProcessor* ModelTypeService::change_processor() const { 17 ModelTypeChangeProcessor* ModelTypeService::change_processor() const {
14 return change_processor_.get(); 18 return change_processor_.get();
15 } 19 }
16 20
17 void ModelTypeService::set_change_processor( 21 ModelTypeChangeProcessor* ModelTypeService::GetOrCreateChangeProcessor() {
18 scoped_ptr<ModelTypeChangeProcessor> change_processor) { 22 if (!change_processor_.get()) {
skym 2016/03/24 16:08:52 I think this .get() is unnecessary.
Gang Wu 2016/03/24 22:18:11 Done.
19 DCHECK(!change_processor_); 23 change_processor_.reset(
20 change_processor_.swap(change_processor); 24 change_processor_factory_.Run(type(), this).release());
skym 2016/03/24 16:08:52 Is there a reason you're not using std::swap anymo
Gang Wu 2016/03/24 22:18:11 Done.
21 OnChangeProcessorSet(); 25 DCHECK(change_processor_.get());
skym 2016/03/24 16:08:52 I think this .get() in unnecessary.
Gang Wu 2016/03/24 22:18:11 Done.
26 OnChangeProcessorSet();
27 }
28 return change_processor_.get();
22 } 29 }
23 30
24 void ModelTypeService::clear_change_processor() { 31 void ModelTypeService::clear_change_processor() {
25 change_processor_.reset(); 32 change_processor_.reset();
26 } 33 }
27 34
35 ModelTypeChangeProcessor* ModelTypeService::OnSyncStarting(
36 const ModelTypeChangeProcessor::StartCallback& start_callback) {
37 ModelTypeChangeProcessor* processor = GetOrCreateChangeProcessor();
38 DCHECK(processor);
39 processor->OnSyncStarting(start_callback);
40 return processor;
41 }
42
28 } // namespace syncer_v2 43 } // namespace syncer_v2
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698