Revert r39951 - Broke Valgrind - "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"

Revert r39997 - Attempted Fix - "Fix a conditional jump depending on an uninitialized value by setting it to false in the ctor."

TBR=dimich
BUG=none
TEST=Valgrind

Review URL: http://codereview.chromium.org/661068

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@39999 0039d316-1c4b-4281-b951-d872f2087c98

1 files changed, 2 insertions, 4 deletions

diff --git a/chrome/common/child_thread.cc b/chrome/common/child_thread.cc
index c7b488a..5d61738 100644
--- a/chrome/common/child_thread.cc
+++ b/chrome/common/child_thread.cc
@@ -24,8 +24,7 @@ ChildThread::ChildThread() {
 }
 
 ChildThread::ChildThread(const std::string& channel_name)
-    : channel_name_(channel_name),
-      on_channel_error_called_(false) {
+    : channel_name_(channel_name) {
   Init();
 }
 
@@ -72,7 +71,6 @@ ChildThread::~ChildThread() {
 }
 
 void ChildThread::OnChannelError() {
-  set_on_channel_error_called(true);
   MessageLoop::current()->Quit();
 }
 
@@ -162,7 +160,7 @@ ChildThread* ChildThread::current() {
 }
 
 void ChildThread::OnProcessFinalRelease() {
-  if (on_channel_error_called_ || !check_with_browser_before_shutdown_) {
+  if (!check_with_browser_before_shutdown_) {
     MessageLoop::current()->Quit();
     return;
   }