Revert 146422 - Merge 144583 - sync: fix reentrancy crash in ModelAssociationManager

chrome/browser/sync/glue/fake_data_type_controller.cc

Index: chrome/browser/sync/glue/fake_data_type_controller.cc
===================================================================
--- chrome/browser/sync/glue/fake_data_type_controller.cc	(revision 146436)
+++ chrome/browser/sync/glue/fake_data_type_controller.cc	(working copy)
@@ -74,13 +74,6 @@
 // * -> NOT_RUNNING
 void FakeDataTypeController::Stop() {
   state_ = NOT_RUNNING;
-  if (!model_load_callback_.is_null()) {
-    // Real data type controllers run the callback and specify "ABORTED" as an
-    // error.  We should probably find a way to use the real code and mock out
-    // unnecessary pieces.
-    SimulateModelLoadFinishing();
-  }
-
   // The DTM still expects |last_start_callback_| to be called back.
   if (!last_start_callback_.is_null()) {
     SyncError error(FROM_HERE, "Fake error", type_);