Only wait for CancelAllRequests done if the post task succeeds

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 118 matching lines @@
   // should match the sql query's projection name.
   // |statement| is query's statement which bound the variables. This class
   // take the ownership of |statement|.
   SQLiteCursor(const std::vector<std::string>& column_names,
                history::AndroidStatement* statement,
                AndroidHistoryProviderService* service,
                FaviconService* favicon_service);
 
   virtual ~SQLiteCursor();
 
+  void DestroyOnUIThread();

[sky, 2013/06/03 17:48:25]

Add a comment.

[michaelbai, 2013/06/03 22:25:37]

Done.

+
+

[sky, 2013/06/03 17:48:25]

nit: remove one of these newlines.

[michaelbai, 2013/06/03 22:25:37]

Done.

   // This method is for testing only.
   void set_test_observer(TestObserver* test_observer) {
     test_observer_ = test_observer;
   }
 
   // Get Favicon from history backend.
   bool GetFavicon(chrome::FaviconID id,
                   std::vector<unsigned char>* image_data);
 
   void GetFaviconForIDInUIThread(
       chrome::FaviconID id,
       const FaviconService::FaviconRawCallback& callback);
 
   // The callback function of FaviconService::GetLargestRawFaviconForID().
   void OnFaviconData(const chrome::FaviconBitmapResult& bitmap_result);
 
   // The callback function of MoveTo().
   void OnMoved(AndroidHistoryProviderService::Handle handle, int pos);
 
   // Used to cancel all request on the UI thread during shutdown.
-  void CancelAllRequests(base::WaitableEvent* finished);
+  void CancelAllRequests();
 
   JavaColumnType GetColumnTypeInternal(int column);
 
   // Runs the MoveStatement on UI thread.
   void RunMoveStatementOnUIThread(int pos);
 
   // The current row position, '-1' means the position before the first one.
   int position_;
 
   base::WaitableEvent event_;
@@ skipping 17 matching lines @@
 
   // The favicon image.
   chrome::FaviconBitmapResult favicon_bitmap_result_;
 
   TestObserver* test_observer_;
 
   DISALLOW_COPY_AND_ASSIGN(SQLiteCursor);
 };
 
 #endif  // CHROME_BROWSER_HISTORY_ANDROID_SQLITE_CURSOR_H_