Commit 3b6b0c7d authored by ananta@chromium.org's avatar ananta@chromium.org

Revert 107995 - Fix a ChromeFrame crash which occurs in the background worker...

Revert 107995 - Fix a ChromeFrame crash which occurs in the background worker thread while dereferencing a NULL automation client.
This crash occurs when the active document is in the process of shutting down while there are active background
requests still pending.

Fix is to ensure that the background thread has been stopped before returning from the UrlmonUrlRequestManager::StopAll
function which gets called during CF shutdown.

Fixes bug http://code.google.com/p/chromium/issues/detail?id=102393

BUG=102393
Review URL: http://codereview.chromium.org/8429011

TBR=ananta@chromium.org
Review URL: http://codereview.chromium.org/8431011

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@107997 0039d316-1c4b-4281-b951-d872f2087c98
parent 9dc4eb0f
......@@ -1261,8 +1261,6 @@ void UrlmonUrlRequestManager::StopAll() {
NewRunnableMethod(
this, &UrlmonUrlRequestManager::StopAllRequestsHelper,
&background_request_map_, &background_resource_map_lock_));
// background_thread_->Stop();
// background_thread_.reset();
}
}
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment