[Sync] Don't call start_callback_ if null in SMTP.

components/sync/model_impl/shared_model_type_processor.cc

Index: components/sync/model_impl/shared_model_type_processor.cc
diff --git a/components/sync/model_impl/shared_model_type_processor.cc b/components/sync/model_impl/shared_model_type_processor.cc
index 4ad248b519120bf024848120928ac708adbcad88..d3e9f6f0613652ae7122b636cf20b1e18e937718 100644
--- a/components/sync/model_impl/shared_model_type_processor.cc
+++ b/components/sync/model_impl/shared_model_type_processor.cc
@@ -102,17 +102,15 @@ void SharedModelTypeProcessor::ConnectIfReady() {
   if (start_error_) {
     error_handler_.Run(start_error_.value());
     start_error_.reset();
-    start_callback_.Reset();
-    return;
+  } else if (start_callback_) {
+    auto activation_context = base::MakeUnique<ActivationContext>();
+    activation_context->model_type_state = model_type_state_;
+    activation_context->type_processor =
+        base::MakeUnique<ModelTypeProcessorProxy>(
+            weak_ptr_factory_.GetWeakPtr(),
+            base::ThreadTaskRunnerHandle::Get());
+    start_callback_.Run(std::move(activation_context));
   }
-
-  auto activation_context = base::MakeUnique<ActivationContext>();
-  activation_context->model_type_state = model_type_state_;
-  activation_context->type_processor =
-      base::MakeUnique<ModelTypeProcessorProxy>(
-          weak_ptr_factory_.GetWeakPtr(), base::ThreadTaskRunnerHandle::Get());
-
-  start_callback_.Run(std::move(activation_context));
   start_callback_.Reset();
 }