Fix race condition in WebstoreInstallHelper

chrome/browser/bitmap_fetcher/bitmap_fetcher.cc

Index: chrome/browser/bitmap_fetcher/bitmap_fetcher.cc
diff --git a/chrome/browser/bitmap_fetcher/bitmap_fetcher.cc b/chrome/browser/bitmap_fetcher/bitmap_fetcher.cc
index f99f3c20f56d889b94fee0598e5b2d58e9cf48f5..ae909bab1130fb0f31d31c5bd43967c2b797c6f6 100644
--- a/chrome/browser/bitmap_fetcher/bitmap_fetcher.cc
+++ b/chrome/browser/bitmap_fetcher/bitmap_fetcher.cc
@@ -37,8 +37,6 @@ void BitmapFetcher::Start(net::URLRequestContextGetter* request_context,
 // Methods inherited from URLFetcherDelegate.
 
 void BitmapFetcher::OnURLFetchComplete(const net::URLFetcher* source) {
-  DCHECK_CURRENTLY_ON(content::BrowserThread::UI);

[Marc Treib, 2015/05/27 08:23:04]

I don't think this DCHECK should be here. It's the only thread check in the
whole file, and I see no reason why this should only be allowed on the UI
thread.

-
   if (source->GetStatus().status() != net::URLRequestStatus::SUCCESS) {
     ReportFailure();
     return;