diff options
| author | dimich@google.com <dimich@google.com@0039d316-1c4b-4281-b951-d872f2087c98> | 2010-02-24 23:50:53 +0000 |
|---|---|---|
| committer | dimich@google.com <dimich@google.com@0039d316-1c4b-4281-b951-d872f2087c98> | 2010-02-24 23:50:53 +0000 |
| commit | f11030094e99bcb61555dc3bb4c157703dc1d7ea (patch) | |
| tree | 5f2001f74906cc33a10bb3d6fc3154a26774b47e /chrome/worker/webworker_stub.cc | |
| parent | 0070eab07191eaed49b945da7db9ba8e687612a5 (diff) | |
| download | chromium_src-f11030094e99bcb61555dc3bb4c157703dc1d7ea.zip chromium_src-f11030094e99bcb61555dc3bb4c157703dc1d7ea.tar.gz chromium_src-f11030094e99bcb61555dc3bb4c157703dc1d7ea.tar.bz2 | |
Make sure the workers are given a chance to terminate their thread when the IPC channel fails.
Usually, the ChildThread::OnChannelError() simply kills the message loop. we want to give workers
an opportunity to get out of their threads to avoid crashes when main thread destroys globals.
BUG=35963
TEST=WorkerTest.StressJSExecution
Review URL: http://codereview.chromium.org/647064
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@39951 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome/worker/webworker_stub.cc')
| -rw-r--r-- | chrome/worker/webworker_stub.cc | 4 |
1 files changed, 4 insertions, 0 deletions
diff --git a/chrome/worker/webworker_stub.cc b/chrome/worker/webworker_stub.cc index 33a9b60..142adcf 100644 --- a/chrome/worker/webworker_stub.cc +++ b/chrome/worker/webworker_stub.cc @@ -48,6 +48,10 @@ WebWorkerStub::~WebWorkerStub() { impl_->clientDestroyed(); } +void WebWorkerStub::OnChannelError() { + OnTerminateWorkerContext(); +} + void WebWorkerStub::OnMessageReceived(const IPC::Message& message) { if (!impl_) return; |