Add callback list to PrefModelAssociator after sync data is loaded

components/syncable_prefs/pref_service_syncable_unittest.cc

Index: components/syncable_prefs/pref_service_syncable_unittest.cc
diff --git a/components/syncable_prefs/pref_service_syncable_unittest.cc b/components/syncable_prefs/pref_service_syncable_unittest.cc
index 5aaa09616db396c3a53545ddeb412651e6ea4e59..126caa2ed3865478c3400da628cfb630bebb1eab 100644
--- a/components/syncable_prefs/pref_service_syncable_unittest.cc
+++ b/components/syncable_prefs/pref_service_syncable_unittest.cc
@@ -46,6 +46,10 @@ const char kDefaultCharsetPrefName[] = "default_charset";
 const char kNonDefaultCharsetValue[] = "foo";
 const char kDefaultCharsetValue[] = "utf-8";
 
+void CallbackFunc(int* num) {

[Bernhard Bauer, 2016/07/14 10:29:17]

Maybe call this method "increment"?

[lshang, 2016/07/15 01:09:07]

Done.

+  (*num)++;
+}
+
 class TestPrefModelAssociatorClient : public PrefModelAssociatorClient {
  public:
   TestPrefModelAssociatorClient() {}
@@ -121,7 +125,6 @@ class PrefServiceSyncableTest : public testing::Test {
     pref_sync_service_ = reinterpret_cast<PrefModelAssociator*>(
         prefs_.GetSyncableService(syncer::PREFERENCES));
     ASSERT_TRUE(pref_sync_service_);
-    next_pref_remote_sync_node_id_ = 0;
   }
 
   syncer::SyncChange MakeRemoteChange(int64_t id,
@@ -555,6 +558,17 @@ TEST_F(PrefServiceSyncableTest, DeletePreference) {
   EXPECT_TRUE(pref->IsDefaultValue());
 }
 
+TEST_F(PrefServiceSyncableTest, RegisterMergeDataFinishedCallback) {
+  int num_callbacks = 0;
+
+  prefs_.RegisterMergeDataFinishedCallback(
+      base::Bind(&CallbackFunc, &num_callbacks));
+  EXPECT_EQ(0, num_callbacks);
+
+  InitWithNoSyncData();
+  EXPECT_EQ(1, num_callbacks);
+}
+
 }  // namespace
 
 }  // namespace syncable_prefs