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

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

Issue 247433004: sync: remove CreateSharedChangeProcessor from ProfileSyncComponentsFactory (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: remove SCPFactory Created 6 years, 8 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/non_ui_data_type_controller.cc
diff --git a/chrome/browser/sync/glue/non_ui_data_type_controller.cc b/chrome/browser/sync/glue/non_ui_data_type_controller.cc
index ba47bcf7d1f4403288b06459d08b400ae3f7ebb2..e14a8fdf949d70d31fc75e4012af798c107b1191 100644
--- a/chrome/browser/sync/glue/non_ui_data_type_controller.cc
+++ b/chrome/browser/sync/glue/non_ui_data_type_controller.cc
@@ -20,6 +20,11 @@ using content::BrowserThread;
namespace browser_sync {
+SharedChangeProcessor*
+NonUIDataTypeController::CreateSharedChangeProcessor() {
+ return new SharedChangeProcessor();
+}
+
NonUIDataTypeController::NonUIDataTypeController(
scoped_refptr<base::MessageLoopProxy> ui_thread,
const base::Closure& error_callback,
@@ -47,12 +52,10 @@ void NonUIDataTypeController::LoadModels(
}
state_ = MODEL_STARTING;
-
// Since we can't be called multiple times before Stop() is called,
// |shared_change_processor_| must be NULL here.
DCHECK(!shared_change_processor_.get());
- shared_change_processor_ =
- profile_sync_factory_->CreateSharedChangeProcessor();
+ shared_change_processor_ = CreateSharedChangeProcessor();
DCHECK(shared_change_processor_.get());
model_load_callback_ = model_load_callback;

Powered by Google App Engine
This is Rietveld 408576698