[USS] Change the place where SharedModelTypeProcessor got created

sync/internal_api/public/test/fake_model_type_service.h

Index: sync/internal_api/public/test/fake_model_type_service.h
diff --git a/sync/internal_api/public/test/fake_model_type_service.h b/sync/internal_api/public/test/fake_model_type_service.h
index 966680a9a796ad36bd90c55b41f0fb03d0330c18..729bf1990690412abd4cacdf7b1022b386a9aa8b 100644
--- a/sync/internal_api/public/test/fake_model_type_service.h
+++ b/sync/internal_api/public/test/fake_model_type_service.h
@@ -42,6 +42,19 @@ class FakeModelTypeService
   std::string GetClientTag(const EntityData& entity_data) override;
 
   void OnChangeProcessorSet() override;
+
+  syncer::ModelType type() const override;
+
+ protected:
+  // The function will create ModelTypeChangeProcessor.
+  virtual ModelTypeChangeProcessor* CreateProcessorForTest(

[skym, 2016/03/24 16:08:52]

I still think overloading the constructor is cleaner than overriding virtual
methods.

[Gang Wu, 2016/03/24 22:18:11]

I can do it in another CL.

[maxbogue, 2016/03/24 23:28:47]

I'm pretty please with this way because you don't need any binds in subclasses,
but either is fine with me.

+      syncer::ModelType type,
+      ModelTypeService* service);
+
+ private:
+  scoped_ptr<ModelTypeChangeProcessor> CreateProcessorForTestWrapper(
+      syncer::ModelType type,
+      ModelTypeService* service);
 };
 
 }  // namespace syncer_v2