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

chrome/browser/sync/glue/model_association_manager.cc

Index: chrome/browser/sync/glue/model_association_manager.cc
===================================================================
--- chrome/browser/sync/glue/model_association_manager.cc	(revision 146436)
+++ chrome/browser/sync/glue/model_association_manager.cc	(working copy)
@@ -101,14 +101,12 @@
 
 void ModelAssociationManager::Initialize(
     syncable::ModelTypeSet desired_types) {
-  // TODO(tim): Bug 134550.  CHECKing to ensure no reentrancy on dev channel.
-  // Remove this.
-  CHECK_EQ(state_, IDLE);
+  DCHECK_EQ(state_, IDLE);
   needs_start_.clear();
   needs_stop_.clear();
   failed_datatypes_info_.clear();
   desired_types_ = desired_types;
-  state_ = INITIALIZED_TO_CONFIGURE;
+  state_ = INITIAILIZED_TO_CONFIGURE;
 
   DVLOG(1) << "ModelAssociationManager: Initializing";
 
@@ -156,14 +154,14 @@
 }
 
 void ModelAssociationManager::StartAssociationAsync() {
-  DCHECK_EQ(state_, INITIALIZED_TO_CONFIGURE);
+  DCHECK_EQ(state_, INITIAILIZED_TO_CONFIGURE);
   state_ = CONFIGURING;
   DVLOG(1) << "ModelAssociationManager: Going to start model association";
   LoadModelForNextType();
 }
 
 void ModelAssociationManager::ResetForReconfiguration() {
-  DCHECK_EQ(state_, INITIALIZED_TO_CONFIGURE);
+  DCHECK_EQ(state_, INITIAILIZED_TO_CONFIGURE);
   state_ = IDLE;
   DVLOG(1) << "ModelAssociationManager: Reseting for reconfiguration";
   needs_start_.clear();
@@ -172,7 +170,7 @@
 }
 
 void ModelAssociationManager::StopDisabledTypes() {
-  DCHECK_EQ(state_, INITIALIZED_TO_CONFIGURE);
+  DCHECK_EQ(state_, INITIAILIZED_TO_CONFIGURE);
   DVLOG(1) << "ModelAssociationManager: Stopping disabled types.";
   // Stop requested data types.
   for (size_t i = 0; i < needs_stop_.size(); ++i) {
@@ -213,7 +211,7 @@
 
   // Now continue stopping any types that have already started.
   DCHECK(state_ == IDLE ||
-         state_ == INITIALIZED_TO_CONFIGURE);
+         state_ == INITIAILIZED_TO_CONFIGURE);
   for (DataTypeController::TypeMap::const_iterator it = controllers_->begin();
        it != controllers_->end(); ++it) {
     DataTypeController* dtc = (*it).second;
@@ -418,18 +416,13 @@
     }
     NOTREACHED();
     return;
-  } else if (state_ == IDLE) {
+  } else {
     DVLOG(1) << "ModelAssociationManager: Models loaded after configure cycle"
             << "Informing DTM";
     // This datatype finished loading after the deadline imposed by the
     // originating configuration cycle. Inform the DataTypeManager that the
     // type has loaded, so that association may begin.
     result_processor_->OnTypesLoaded();
-  } else {
-    // If we're not IDLE or CONFIGURING, we're being invoked as part of an abort
-    // process (possibly a reconfiguration, or disabling of a broken data type).
-    DVLOG(1) << "ModelAssociationManager: ModelLoadCallback occurred while "
-             << "not IDLE or CONFIGURING. Doing nothing.";
   }
 
 }
@@ -509,3 +502,4 @@
 }
 
 }  // namespace browser_sync
+