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

chrome/browser/devtools/devtools_sanity_browsertest.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/bind.h"
 #include "base/cancelable_callback.h"
 #include "base/command_line.h"
 #include "base/compiler_specific.h"
 #include "base/location.h"
 #include "base/macros.h"
@@ skipping 298 matching lines @@
   }
 };
 
 class DevToolsUnresponsiveBeforeUnloadTest: public DevToolsBeforeUnloadTest {
  public:
   void SetUpCommandLine(base::CommandLine* command_line) override {}
 };
 
 void TimeoutCallback(const std::string& timeout_message) {
   ADD_FAILURE() << timeout_message;
-  base::MessageLoop::current()->Quit();
+  base::MessageLoop::current()->QuitWhenIdle();
 }
 
 // Base class for DevTools tests that test devtools functionality for
 // extensions and content scripts.
 class DevToolsExtensionTest : public DevToolsSanityTest,
                               public content::NotificationObserver {
  public:
   DevToolsExtensionTest() : DevToolsSanityTest() {
     PathService::Get(chrome::DIR_TEST_DATA, &test_extensions_dir_);
     test_extensions_dir_ = test_extensions_dir_.AppendASCII("devtools");
@@ skipping 63 matching lines @@
     timeout.Cancel();
     return true;
   }
 
   void Observe(int type,
                const content::NotificationSource& source,
                const content::NotificationDetails& details) override {
     switch (type) {
       case extensions::NOTIFICATION_EXTENSION_LOADED_DEPRECATED:
       case extensions::NOTIFICATION_EXTENSION_HOST_DID_STOP_FIRST_LOAD:
-        base::MessageLoopForUI::current()->Quit();
+        base::MessageLoopForUI::current()->QuitWhenIdle();
         break;
       default:
         NOTREACHED();
         break;
     }
   }
 
   base::FilePath test_extensions_dir_;
 };
 
@@ skipping 33 matching lines @@
     void WorkerCreated(const GURL& url,
                        const base::string16& name,
                        int process_id,
                        int route_id) override {
       if (url.path().rfind(path_) == std::string::npos)
         return;
       worker_data_->worker_process_id = process_id;
       worker_data_->worker_route_id = route_id;
       WorkerService::GetInstance()->RemoveObserver(this);
       BrowserThread::PostTask(BrowserThread::UI, FROM_HERE,
-          base::MessageLoop::QuitClosure());
+                              base::MessageLoop::QuitWhenIdleClosure());
       delete this;
     }
     std::string path_;
     scoped_refptr<WorkerData> worker_data_;
   };
 
   class WorkerTerminationObserver : public WorkerServiceObserver {
    public:
     explicit WorkerTerminationObserver(WorkerData* worker_data)
         : worker_data_(worker_data) {
     }
 
    private:
     ~WorkerTerminationObserver() override {}
 
     void WorkerDestroyed(int process_id, int route_id) override {
       ASSERT_EQ(worker_data_->worker_process_id, process_id);
       ASSERT_EQ(worker_data_->worker_route_id, route_id);
       WorkerService::GetInstance()->RemoveObserver(this);
       BrowserThread::PostTask(BrowserThread::UI, FROM_HERE,
-          base::MessageLoop::QuitClosure());
+                              base::MessageLoop::QuitWhenIdleClosure());
       delete this;
     }
     scoped_refptr<WorkerData> worker_data_;
   };
 
   void RunTest(const char* test_name,
                const char* test_page,
                const char* worker_path) {
     ASSERT_TRUE(test_server()->Start());
     GURL url = test_server()->GetURL(test_page);
@@ skipping 25 matching lines @@
       const std::string& path,
       scoped_refptr<WorkerData> worker_data) {
     std::vector<WorkerService::WorkerInfo> worker_info =
         WorkerService::GetInstance()->GetWorkers();
     for (size_t i = 0; i < worker_info.size(); i++) {
       if (worker_info[i].url.path().rfind(path) == std::string::npos)
         continue;
       worker_data->worker_process_id = worker_info[0].process_id;
       worker_data->worker_route_id = worker_info[0].route_id;
       BrowserThread::PostTask(BrowserThread::UI, FROM_HERE,
-          base::MessageLoop::QuitClosure());
+                              base::MessageLoop::QuitWhenIdleClosure());
       return;
     }
 
     WorkerService::GetInstance()->AddObserver(
         new WorkerCreationObserver(path, worker_data.get()));
   }
 
   static scoped_refptr<WorkerData> WaitForFirstSharedWorker(const char* path) {
     scoped_refptr<WorkerData> worker_data(new WorkerData());
     BrowserThread::PostTask(
@@ skipping 502 matching lines @@
     (defined(OS_CHROMEOS) && defined(OFFICIAL_BUILD))
 #define MAYBE_TestScreenshotRecording DISABLED_TestScreenshotRecording
 #else
 #define MAYBE_TestScreenshotRecording TestScreenshotRecording
 #endif
 // Tests raw headers text.
 IN_PROC_BROWSER_TEST_F(DevToolsPixelOutputTests,
                        MAYBE_TestScreenshotRecording) {
   RunTest("testScreenshotRecording", std::string());
 }