Add callback list to PrefModelAssociator after sync data is loaded

chrome/browser/download/download_request_limiter_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 "chrome/browser/download/download_request_limiter.h"
 
 #include "base/bind.h"
 #include "base/command_line.h"
 #include "base/run_loop.h"
 #include "build/build_config.h"
@@ skipping 133 matching lines @@
     ChromeRenderViewHostTestHarness::SetUp();
     profile_.reset(new TestingProfile());
     testing_delegate_.SetUp(web_contents());
 
     UpdateExpectations(ACCEPT);
     cancel_count_ = continue_count_ = 0;
     download_request_limiter_ = new DownloadRequestLimiter();
 
     content_settings_ = new HostContentSettingsMap(
         profile_->GetPrefs(), false /* incognito_profile */,
-        false /* guest_profile */);
+        false /* guest_profile */, nullptr /* PrefServiceSyncable* */);

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

nit: usually just put the name of the variable in the comment, rather than the
type

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

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

     DownloadRequestLimiter::SetContentSettingsForTesting(
         content_settings_.get());
   }
 
   void TearDown() override {
     content_settings_->ShutdownOnUIThread();
     content_settings_ = nullptr;
     testing_delegate_.TearDown();
     ChromeRenderViewHostTestHarness::TearDown();
   }
@@ skipping 551 matching lines @@
   // changing the content setting).
   UpdateContentSettings(web_contents(), CONTENT_SETTING_ASK);
   ASSERT_EQ(DownloadRequestLimiter::PROMPT_BEFORE_DOWNLOAD,
             download_request_limiter_->GetDownloadStatus(web_contents()));
   UpdateExpectations(WAIT);
   CanDownload();
   ExpectAndResetCounts(0, 0, 1, __LINE__);
   ASSERT_EQ(DownloadRequestLimiter::PROMPT_BEFORE_DOWNLOAD,
             download_request_limiter_->GetDownloadStatus(web_contents()));
 }