Don't use base::MessageLoop::{Quit,QuitClosure} in components/

components/history/core/browser/top_sites_impl_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/history/core/browser/top_sites_impl.h"
 
 #include "base/bind.h"
 #include "base/files/scoped_temp_dir.h"
 #include "base/memory/weak_ptr.h"
 #include "base/message_loop/message_loop.h"
@@ skipping 35 matching lines @@
 
 // Used by WaitForHistory, see it for details.
 class WaitForHistoryTask : public HistoryDBTask {
  public:
   WaitForHistoryTask() {}
 
   bool RunOnDBThread(HistoryBackend* backend, HistoryDatabase* db) override {
     return true;
   }
 
-  void DoneRunOnMainThread() override { base::MessageLoop::current()->Quit(); }
+  void DoneRunOnMainThread() override {
+    base::MessageLoop::current()->QuitWhenIdle();
+  }
 
  private:
   ~WaitForHistoryTask() override {}
 
   DISALLOW_COPY_AND_ASSIGN(WaitForHistoryTask);
 };
 
 // Used for querying top sites. Either runs sequentially, or runs a nested
 // nested message loop until the response is complete. The later is used when
 // TopSites is queried before it finishes loading.
@@ skipping 30 matching lines @@
   const MostVisitedURLList& urls() const { return urls_; }
 
   int number_of_callbacks() const { return number_of_callbacks_; }
 
  private:
   // Callback for TopSitesImpl::GetMostVisitedURLs.
   void OnTopSitesAvailable(const history::MostVisitedURLList& data) {
     urls_ = data;
     number_of_callbacks_++;
     if (waiting_) {
-      base::MessageLoop::current()->Quit();
+      base::MessageLoop::current()->QuitWhenIdle();
       waiting_ = false;
     }
   }
 
   MostVisitedURLList urls_;
   int number_of_callbacks_;
   bool waiting_;
   base::WeakPtrFactory<TopSitesQuerier> weak_ptr_factory_;
 
   DISALLOW_COPY_AND_ASSIGN(TopSitesQuerier);
@@ skipping 86 matching lines @@
     ASSERT_LE(start_index + prepopulate_pages.size(), querier.urls().size());
     for (size_t i = 0; i < prepopulate_pages.size(); ++i) {
       EXPECT_EQ(prepopulate_pages[i].most_visited.url.spec(),
                 querier.urls()[start_index + i].url.spec())
           << " @ index " << i;
     }
   }
 
   // Quit the current message loop when invoked. Useful when running a nested
   // message loop.
-  void QuitCallback() { base::MessageLoop::current()->Quit(); }
+  void QuitCallback() { base::MessageLoop::current()->QuitWhenIdle(); }
 
   // Adds a page to history.
   void AddPageToHistory(const GURL& url) {
     RedirectList redirects;
     redirects.push_back(url);
     history_service()->AddPage(
         url, base::Time::Now(), reinterpret_cast<ContextID>(1), 0, GURL(),
         redirects, ui::PAGE_TRANSITION_TYPED, history::SOURCE_BROWSED, false);
   }
 
@@ skipping 1426 matching lines @@
   EXPECT_EQ("http://forced/5", querier.urls()[5].url.spec());
   EXPECT_EQ(6000u, querier.urls()[5].last_forced_time.ToJsTime());
   scoped_refptr<base::RefCountedMemory> thumbnail_data;
   ASSERT_TRUE(top_sites()->GetPageThumbnail(GURL("http://forced/5"), false,
                                             &thumbnail_data));
   ASSERT_TRUE(
       ThumbnailsAreEqual(orig_thumbnail_data.get(), thumbnail_data.get()));
 }
 
 }  // namespace history