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

chrome/browser/history/android/sqlite_cursor.h

 // 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.
 
 #ifndef CHROME_BROWSER_HISTORY_ANDROID_SQLITE_CURSOR_H_
 #define CHROME_BROWSER_HISTORY_ANDROID_SQLITE_CURSOR_H_
 
 #include <jni.h>
 #include <vector>
 
@@ skipping 27 matching lines @@
   };
 
   // This class is intended to be used only in unit tests.
   //
   // There are 2 threads in unit test, one is the UI thread, another is the DB
   // thread, after the task posted into UI thread, the MessageLoop needs to run
   // to execute the task. The OnPostMoveToTask() and the OnPostGetFaviconTask()
   // give unit tests a chance to run the message loop before event_.Wait is
   // invoked, The OnGetMoveToResult() and OnGetFaviconResult() is used to notify
   // the test observer in the UI thread when the task's result comes back, it
-  // calls MessageLoop::Quit() to exit the loop, then the event.Wait() is
-  // called. Basically, Two threads are used to simulate 3 threads' behavior
+  // calls MessageLoop::QuitWhenIdle() to exit the loop, then the event.Wait()
+  // is called. Basically, Two threads are used to simulate 3 threads' behavior
   // here.
   // The whole observer design is only for test purpose and should only be used
   // in unit test.
   class TestObserver {
    public:
     TestObserver();
 
     // Notify the MoveTo task has been posted to UI thread.
     virtual void OnPostMoveToTask() = 0;
     // Notify the MoveTo result has been gotten in UI thread.
@@ skipping 122 matching lines @@
 
   // The favicon image.
   favicon_base::FaviconRawBitmapResult favicon_bitmap_result_;
 
   TestObserver* test_observer_;
 
   DISALLOW_COPY_AND_ASSIGN(SQLiteCursor);
 };
 
 #endif  // CHROME_BROWSER_HISTORY_ANDROID_SQLITE_CURSOR_H_