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

Unified Diff: content/public/browser/web_contents_observer.h

Issue 1228993005: Be a little more precise in the wording of comments for process death. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: updates Created 5 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « content/public/browser/render_process_host_observer.h ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: content/public/browser/web_contents_observer.h
diff --git a/content/public/browser/web_contents_observer.h b/content/public/browser/web_contents_observer.h
index 501b94efb0c95f2c5f357ab95e6d74740f66aabd..15fa138690faf704aa5ef7e98071635220235943 100644
--- a/content/public/browser/web_contents_observer.h
+++ b/content/public/browser/web_contents_observer.h
@@ -97,13 +97,16 @@ class CONTENT_EXPORT WebContentsObserver : public IPC::Listener,
// just swapped out.
virtual void RenderViewDeleted(RenderViewHost* render_view_host) {}
- // This method is invoked when the process for the current RenderView crashes.
- // The WebContents continues to use the RenderViewHost, e.g. when the user
- // reloads the current page. When the RenderViewHost itself is deleted, the
- // RenderViewDeleted method will be invoked.
+ // This method is invoked when the process for the current main
+ // RenderFrameHost exits (usually by crashing, though possibly by other
+ // means). The WebContents continues to use the RenderFrameHost, e.g. when the
+ // user reloads the current page. When the RenderFrameHost itself is deleted,
+ // the RenderFrameDeleted method will be invoked.
//
- // Note that this is equivalent to
- // RenderProcessHostObserver::RenderProcessExited().
+ // Note that this is triggered upstream through
+ // RenderProcessHostObserver::RenderProcessExited(); for code that doesn't
+ // otherwise need to be a WebContentsObserver, that API is probably a better
+ // choice.
virtual void RenderProcessGone(base::TerminationStatus status) {}
// This method is invoked when a WebContents swaps its visible RenderViewHost
« no previous file with comments | « content/public/browser/render_process_host_observer.h ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698