Add callback list to PrefModelAssociator after sync data is loaded

components/content_settings/core/browser/cookie_settings_unittest.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "components/content_settings/core/browser/cookie_settings.h"
 
 #include "components/content_settings/core/browser/host_content_settings_map.h"
 #include "components/content_settings/core/common/content_settings_pattern.h"
 #include "components/content_settings/core/common/pref_names.h"
 #include "components/pref_registry/testing_pref_service_syncable.h"
@@ skipping 11 matching lines @@
         kAllowedSite("http://good.allays.com"),
         kFirstPartySite("http://cool.things.com"),
         kChromeURL("chrome://foo"),
         kExtensionURL("chrome-extension://deadbeef"),
         kHttpSite("http://example.com"),
         kHttpsSite("https://example.com"),
         kAllHttpsSitesPattern(ContentSettingsPattern::FromString("https://*")) {
     CookieSettings::RegisterProfilePrefs(prefs_.registry());
     HostContentSettingsMap::RegisterProfilePrefs(prefs_.registry());
     settings_map_ = new HostContentSettingsMap(
-        &prefs_, false /* incognito_profile */, false /* guest_profile */);
+        &prefs_, false /* incognito_profile */, false /* guest_profile */,
+        nullptr /* PrefServiceSyncable* */);

[raymes, 2016/06/29 03:50:01]

nit: same here

[lshang, 2016/06/30 05:03:35]

We've changed to not pass in the pointer, so no need to address this.

     cookie_settings_ =
         new CookieSettings(settings_map_.get(), &prefs_, "chrome-extension");
   }
 
   ~CookieSettingsTest() override { settings_map_->ShutdownOnUIThread(); }
 
  protected:
   user_prefs::TestingPrefServiceSyncable prefs_;
   scoped_refptr<HostContentSettingsMap> settings_map_;
   scoped_refptr<CookieSettings> cookie_settings_;
@@ skipping 186 matching lines @@
 
   // XHRs stemming from extensions are exempt from third-party cookie blocking
   // rules (as the first party is always the extension's security origin).
   EXPECT_TRUE(
       cookie_settings_->IsSettingCookieAllowed(kBlockedSite, kExtensionURL));
 }
 
 }  // namespace
 
 }  // namespace content_settings