[Sync] Not having an autocomplete ModelTypeState is valid.

components/sync/model/recording_model_type_change_processor.h

Index: components/sync/model/recording_model_type_change_processor.h
diff --git a/components/sync/model/recording_model_type_change_processor.h b/components/sync/model/recording_model_type_change_processor.h
index d0789b4e6e5de0d3da03b6d3980e5863e169782e..6fe9180166c8fd4fb35916eab8139cdbeb257b94 100644
--- a/components/sync/model/recording_model_type_change_processor.h
+++ b/components/sync/model/recording_model_type_change_processor.h
@@ -5,6 +5,11 @@
 #ifndef COMPONENTS_SYNC_MODEL_RECORDING_MODEL_TYPE_CHANGE_PROCESSOR_H_
 #define COMPONENTS_SYNC_MODEL_RECORDING_MODEL_TYPE_CHANGE_PROCESSOR_H_
 
+#include <map>
+#include <memory>
+#include <set>
+#include <string>
+
 #include "components/sync/model/fake_model_type_change_processor.h"
 
 namespace syncer {
@@ -34,7 +39,7 @@ class RecordingModelTypeChangeProcessor : public FakeModelTypeChangeProcessor {
 
   const std::set<std::string>& delete_set() const { return delete_set_; }
 
-  const MetadataBatch* metadata() const { return metadata_.get(); }
+  MetadataBatch* metadata() const { return metadata_.get(); }

[Patrick Noland, 2017/01/27 20:49:59]

why this change?

[skym, 2017/01/27 20:55:56]

Because I want to call TakeAllMetadata() on it, which modifies |metadata_|.
There isn't a public const accessor on the interface.

 
  private:
   std::multimap<std::string, std::unique_ptr<EntityData>> put_multimap_;