chrome/browser: Update calls from RunAllPending() to RunUntilIdle().

chrome/browser/download/chrome_download_manager_delegate_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 "base/file_path.h"
 #include "base/file_util.h"
 #include "base/files/scoped_temp_dir.h"
 #include "base/message_loop.h"
 #include "base/observer_list.h"
 #include "base/stl_util.h"
@@ skipping 294 matching lines @@
   delegate_ = new TestChromeDownloadManagerDelegate(profile());
   delegate_->SetDownloadManager(download_manager_.get());
   pref_service_ = profile()->GetTestingPrefService();
   web_contents()->SetDelegate(&web_contents_delegate_);
 
   ASSERT_TRUE(test_download_dir_.CreateUniqueTempDir());
   SetDefaultDownloadPath(test_download_dir_.path());
 }
 
 void ChromeDownloadManagerDelegateTest::TearDown() {
-  message_loop_.RunAllPending();
+  message_loop_.RunUntilIdle();
   delegate_->Shutdown();
   ChromeRenderViewHostTestHarness::TearDown();
 }
 
 void ChromeDownloadManagerDelegateTest::VerifyAndClearExpectations() {
   ::testing::Mock::VerifyAndClearExpectations(delegate_);
   ::testing::Mock::VerifyAndClearExpectations(download_manager_);
   EXPECT_CALL(*download_manager_, RemoveObserver(_)).WillRepeatedly(Return());
   EXPECT_CALL(*download_manager_, GetAllDownloads(_))
       .WillRepeatedly(Return());
@@ skipping 114 matching lines @@
                 ChooseDownloadPath(item, expected_prompt_path, _))
         .WillOnce(WithArg<2>(ScheduleCallback(expected_target_path)));
   }
 
   // Kick off the test.
   base::WeakPtrFactory<ChromeDownloadManagerDelegateTest> factory(this);
   EXPECT_TRUE(delegate_->DetermineDownloadTarget(
       item,
       base::Bind(&ChromeDownloadManagerDelegateTest::DownloadTargetVerifier,
                  factory.GetWeakPtr(), base::Unretained(&test_case))));
-  message_loop_.RunAllPending();
+  message_loop_.RunUntilIdle();
   VerifyAndClearExpectations();
 }
 
 void ChromeDownloadManagerDelegateTest::RunTestCases(
     const DownloadTestCase test_cases[],
     size_t test_case_count) {
   for (size_t i = 0; i < test_case_count; ++i) {
     scoped_ptr<content::MockDownloadItem> item(CreateActiveDownloadItem(i));
     SCOPED_TRACE(testing::Message() << "Running test case " << i);
     RunTestCaseWithDownloadItem(test_cases[i], item.get());
@@ skipping 447 matching lines @@
   // Run the first test case again. Since the last download path was cleared,
   // this test case should behave identically to the first time it was run.
   RunTestCases(kLastSavePathTestCases, 1);
 }
 
 // TODO(asanka): Add more tests.
 // * Default download path is not writable.
 // * Download path doesn't exist.
 // * IsDangerousFile().
 // * Filename generation.