Revert "Reland of Check that all shared workers are terminated. (patchset #2 id:170001 of https://c…

content/browser/renderer_host/render_process_host_impl.cc

 // Copyright 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.
 
 // Represents the browser side of the browser <--> renderer communication
 // channel. There will be one RenderProcessHost per renderer process.
 
 #include "content/browser/renderer_host/render_process_host_impl.h"
 
 #include <algorithm>
@@ skipping 773 matching lines @@
       NOTREACHED() << "There should be only one RenderProcessHost when running "
                    << "in-process.";
   }
 }
 
 void RenderProcessHostImpl::RegisterRendererMainThreadFactory(
     RendererMainThreadFactoryFunction create) {
   g_renderer_main_thread_factory = create;
 }
 
-// static
-// TODO(alokp): Remove after collecting crash data.
-// Temporary checks to verify that all shared workers are terminated.
-// It is suspected that shared workers prevent render process hosts
-// from shutting down: crbug.com/608049
-void RenderProcessHostImpl::CheckAllWorkersTerminated() {
-  iterator iter(AllHostsIterator());
-  while (!iter.IsAtEnd()) {
-    RenderProcessHostImpl* host =
-        static_cast<RenderProcessHostImpl*>(iter.GetCurrentValue());
-    if (host->worker_ref_count() != 0) {
-      std::string message = base::StringPrintf(
-          "%zu service workers, %zu shared workers",
-          host->service_worker_ref_count_, host->shared_worker_ref_count_);
-      // Use separate CHECKs for better crash report readability.
-      CHECK(host->service_worker_ref_count_ == 0 ||
-            host->shared_worker_ref_count_ == 0)
-          << message;
-      CHECK_EQ(0UL, host->service_worker_ref_count_) << message;
-      CHECK_EQ(0UL, host->shared_worker_ref_count_) << message;
-    }
-
-    iter.Advance();
-  }
-}
-
 RenderProcessHostImpl::~RenderProcessHostImpl() {
   DCHECK_CURRENTLY_ON(BrowserThread::UI);
 #ifndef NDEBUG
   DCHECK(is_self_deleted_)
       << "RenderProcessHostImpl is destroyed by something other than itself";
 #endif
 
   // Make sure to clean up the in-process renderer before the channel, otherwise
   // it may still run and have its IPCs fail, causing asserts.
   in_process_renderer_.reset();
@@ skipping 2136 matching lines @@
   LOG(ERROR) << "Terminating render process for bad Mojo message: " << error;
 
   // The ReceivedBadMessage call below will trigger a DumpWithoutCrashing. Alias
   // enough information here so that we can determine what the bad message was.
   base::debug::Alias(&error);
   bad_message::ReceivedBadMessage(process.get(),
                                   bad_message::RPH_MOJO_PROCESS_ERROR);
 }
 
 }  // namespace content