Only wait for CancelAllRequests done if the post task succeeds

chrome/browser/history/android/sqlite_cursor.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 "chrome/browser/history/android/sqlite_cursor.h"
 
 #include "base/android/jni_android.h"
 #include "base/android/jni_array.h"
 #include "base/android/jni_string.h"
 #include "base/bind.h"
@@ skipping 138 matching lines @@
       test_observer_(NULL) {
 }
 
 SQLiteCursor::~SQLiteCursor() {
   // Consumer requests were set in the UI thread. They must be cancelled
   // using the same thread.
   if (BrowserThread::CurrentlyOn(BrowserThread::UI)) {
     CancelAllRequests(NULL);
   } else {
     base::WaitableEvent event(false, false);
-    BrowserThread::PostTask(
-        BrowserThread::UI,
-        FROM_HERE,
+    if (BrowserThread::PostTask(BrowserThread::UI, FROM_HERE,
         base::Bind(&SQLiteCursor::CancelAllRequests, base::Unretained(this),
-                   &event));
-    event.Wait();
+                   &event)))

[Yaron, 2013/05/30 18:14:00]

Please add {}

[michaelbai, 2013/05/30 18:33:56]

Done.

+      event.Wait();
   }
 
   BrowserThread::PostTask(
       BrowserThread::UI,
       FROM_HERE,
       base::Bind(&AndroidHistoryProviderService::CloseStatement,
                  base::Unretained(service_), statement_));
 }
 
 bool SQLiteCursor::GetFavicon(chrome::FaviconID id,
@@ skipping 70 matching lines @@
 }
 
 void SQLiteCursor::RunMoveStatementOnUIThread(int pos) {
   DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI));
   if (!consumer_.get())
     consumer_.reset(new CancelableRequestConsumer());
   service_->MoveStatement(
       statement_, position_, pos, consumer_.get(),
       base::Bind(&SQLiteCursor::OnMoved, base::Unretained(this)));
 }