For debugging, split worker ref count into service worker and shared worker counts

content/browser/site_per_process_browsertest.cc

 // Copyright (c) 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.
 
 #include "content/browser/site_per_process_browsertest.h"
 
 #include <stddef.h>
 #include <stdint.h>
 
 #include <algorithm>
@@ skipping 6920 matching lines @@
       root->child_at(0)->current_frame_host()->GetRoutingID()));
 
   // Prevent b.com process from terminating right away once the subframe
   // navigates away from b.com below.  This is necessary so that the renderer
   // process has time to process the closings of RenderWidget and RenderView,
   // which is where the original bug was triggered.  Incrementing worker
   // RefCount will cause RenderProcessHostImpl::Cleanup to forego process
   // termination.
   RenderProcessHost* subframe_process =
       root->child_at(0)->current_frame_host()->GetProcess();
-  subframe_process->IncrementWorkerRefCount();
+  subframe_process->IncrementSharedWorkerRefCount();
 
   // Navigate the subframe away from b.com.  Since this is the last active
   // frame in the b.com process, this causes the RenderWidget and RenderView to
   // be closed.  If this succeeds without crashing, the renderer will release
   // the process and send a ChildProcessHostMsg_ShutdownRequest to the browser
   // process to ask whether it's ok to terminate.  Thus, wait for this message
   // to ensure that the RenderView and widget were closed without crashing.
   scoped_refptr<ShutdownRequestMessageFilter> filter =
       new ShutdownRequestMessageFilter();
   subframe_process->AddFilter(filter.get());
   NavigateFrameToURL(root->child_at(0),
                      embedded_test_server()->GetURL("a.com", "/title1.html"));
   filter->Wait();
 
   // TODO(alexmos): Navigating the subframe back to b.com at this point would
   // trigger the race in https://crbug.com/535246, where the browser process
   // tries to reuse the b.com process thinking it's still initialized, whereas
   // the process has actually been destroyed by the renderer (but the browser
   // process hasn't heard the OnChannelError yet).  This race will need to be
   // fixed.
 
-  subframe_process->DecrementWorkerRefCount();
+  subframe_process->DecrementSharedWorkerRefCount();
 }
 
 // Tests that an input event targeted to a out-of-process iframe correctly
 // triggers a user interaction notification for WebContentsObservers.
 // This is used for browser features such as download request limiting and
 // launching multiple external protocol handlers, which can block repeated
 // actions from a page when a user is not interacting with the page.
 IN_PROC_BROWSER_TEST_F(SitePerProcessBrowserTest,
                        UserInteractionForChildFrameTest) {
   GURL main_url(embedded_test_server()->GetURL(
@@ skipping 1021 matching lines @@
   child_rfh->OnDispatchLoad();
 
   // In the bug, OnDispatchLoad killed the b.com renderer.  Ensure that this is
   // not the case. Note that the process kill doesn't happen immediately, so
   // IsRenderFrameLive() can't be checked here (yet).  Instead, check that
   // JavaScript can still execute in b.com using the popup.
   EXPECT_TRUE(ExecuteScript(popup_shell->web_contents(), "true"));
 }
 
 }  // namespace content