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

Unified Diff: content/browser/web_contents/web_contents_impl.cc

Issue 14221009: Rename RenderViewHostDelegate::RenderViewGone -> RenderViewTerminated (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: compile fix Created 7 years, 8 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 side-by-side diff with in-line comments
Download patch
Index: content/browser/web_contents/web_contents_impl.cc
diff --git a/content/browser/web_contents/web_contents_impl.cc b/content/browser/web_contents/web_contents_impl.cc
index 8612ef126abb81164f22fdf96cad838f4a05f1db..2e03f22b59a90438f247125af93a154bddd4fed7 100644
--- a/content/browser/web_contents/web_contents_impl.cc
+++ b/content/browser/web_contents/web_contents_impl.cc
@@ -2641,9 +2641,9 @@ void WebContentsImpl::RenderViewReady(RenderViewHost* rvh) {
FOR_EACH_OBSERVER(WebContentsObserver, observers_, RenderViewReady());
}
-void WebContentsImpl::RenderViewGone(RenderViewHost* rvh,
- base::TerminationStatus status,
- int error_code) {
+void WebContentsImpl::RenderViewTerminated(RenderViewHost* rvh,
+ base::TerminationStatus status,
+ int error_code) {
if (rvh != GetRenderViewHost()) {
// The pending page's RenderViewHost is gone.
return;
@@ -3240,7 +3240,7 @@ void WebContentsImpl::BeforeUnloadFiredFromRenderManager(
void WebContentsImpl::RenderViewGoneFromRenderManager(
RenderViewHost* render_view_host) {
DCHECK(crashed_status_ != base::TERMINATION_STATUS_STILL_RUNNING);
- RenderViewGone(render_view_host, crashed_status_, crashed_error_code_);
+ RenderViewTerminated(render_view_host, crashed_status_, crashed_error_code_);
}
void WebContentsImpl::UpdateRenderViewSizeForRenderManager() {
« no previous file with comments | « content/browser/web_contents/web_contents_impl.h ('k') | content/browser/web_contents/web_contents_impl_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698