Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(115)

Side by Side Diff: content/browser/renderer_host/render_process_host_impl.h

Issue 2133813002: Check that all RenderProcessHost(s) are terminated. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: added more specific checks Created 4 years, 5 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #ifndef CONTENT_BROWSER_RENDERER_HOST_RENDER_PROCESS_HOST_IMPL_H_ 5 #ifndef CONTENT_BROWSER_RENDERER_HOST_RENDER_PROCESS_HOST_IMPL_H_
6 #define CONTENT_BROWSER_RENDERER_HOST_RENDER_PROCESS_HOST_IMPL_H_ 6 #define CONTENT_BROWSER_RENDERER_HOST_RENDER_PROCESS_HOST_IMPL_H_
7 7
8 #include <stddef.h> 8 #include <stddef.h>
9 #include <stdint.h> 9 #include <stdint.h>
10 10
(...skipping 186 matching lines...) Expand 10 before | Expand all | Expand 10 after
197 static void ReleaseOnCloseACK( 197 static void ReleaseOnCloseACK(
198 RenderProcessHost* host, 198 RenderProcessHost* host,
199 const SessionStorageNamespaceMap& sessions, 199 const SessionStorageNamespaceMap& sessions,
200 int view_route_id); 200 int view_route_id);
201 201
202 // Register/unregister the host identified by the host id in the global host 202 // Register/unregister the host identified by the host id in the global host
203 // list. 203 // list.
204 static void RegisterHost(int host_id, RenderProcessHost* host); 204 static void RegisterHost(int host_id, RenderProcessHost* host);
205 static void UnregisterHost(int host_id); 205 static void UnregisterHost(int host_id);
206 206
207 // CHECKs that all hosts have terminated.
208 // Added temporarily to diagnose crbug.com/608049.
209 static void CheckAllTerminated();
210
207 // Implementation of FilterURL below that can be shared with the mock class. 211 // Implementation of FilterURL below that can be shared with the mock class.
208 static void FilterURL(RenderProcessHost* rph, bool empty_allowed, GURL* url); 212 static void FilterURL(RenderProcessHost* rph, bool empty_allowed, GURL* url);
209 213
210 // Returns true if |host| is suitable for launching a new view with |site_url| 214 // Returns true if |host| is suitable for launching a new view with |site_url|
211 // in the given |browser_context|. 215 // in the given |browser_context|.
212 static bool IsSuitableHost(RenderProcessHost* host, 216 static bool IsSuitableHost(RenderProcessHost* host,
213 BrowserContext* browser_context, 217 BrowserContext* browser_context,
214 const GURL& site_url); 218 const GURL& site_url);
215 219
216 // Returns an existing RenderProcessHost for |url| in |browser_context|, 220 // Returns an existing RenderProcessHost for |url| in |browser_context|,
(...skipping 294 matching lines...) Expand 10 before | Expand all | Expand 10 after
511 mojo::ScopedMessagePipeHandle in_process_renderer_handle_; 515 mojo::ScopedMessagePipeHandle in_process_renderer_handle_;
512 516
513 base::WeakPtrFactory<RenderProcessHostImpl> weak_factory_; 517 base::WeakPtrFactory<RenderProcessHostImpl> weak_factory_;
514 518
515 DISALLOW_COPY_AND_ASSIGN(RenderProcessHostImpl); 519 DISALLOW_COPY_AND_ASSIGN(RenderProcessHostImpl);
516 }; 520 };
517 521
518 } // namespace content 522 } // namespace content
519 523
520 #endif // CONTENT_BROWSER_RENDERER_HOST_RENDER_PROCESS_HOST_IMPL_H_ 524 #endif // CONTENT_BROWSER_RENDERER_HOST_RENDER_PROCESS_HOST_IMPL_H_
OLDNEW
« no previous file with comments | « content/browser/browser_main_loop.cc ('k') | content/browser/renderer_host/render_process_host_impl.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698