Be a little more precise in the wording of comments for process death.

content/public/browser/web_contents_observer.h

 // 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.
 
 #ifndef CONTENT_PUBLIC_BROWSER_WEB_CONTENTS_OBSERVER_H_
 #define CONTENT_PUBLIC_BROWSER_WEB_CONTENTS_OBSERVER_H_
 
 #include "base/process/kill.h"
 #include "base/process/process_handle.h"
 #include "content/common/content_export.h"
@@ skipping 79 matching lines @@
   // This method is invoked when the RenderView of the current RenderViewHost
   // is ready, e.g. because we recreated it after a crash.
   virtual void RenderViewReady() {}
 
   // This method is invoked when a RenderViewHost of the WebContents is
   // deleted. Note that this does not always happen when the WebContents starts
   // to use a different RenderViewHost, as the old RenderViewHost might get
   // 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 RenderView exits

[nasko, 2015/07/10 07:48:54]

"the process for the current main RenderFrameHost". Also, can we go with
s/RenderViewHost/RenderFrameHost/ in this entire comment, since you are updating
it anyway?

[Avi (use Gerrit), 2015/07/10 15:29:20]

Done.

+  // (usually by crashing, though possibly by other means). 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.
   //
-  // 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, consider whether that API might

[nasko, 2015/07/10 07:48:55]

I'd word it a bit more strongly. Unless they have to be WCO, they are better off
monitoring for process crashing through the process observer :).

[Avi (use Gerrit), 2015/07/10 15:29:20]

Done.

+  // be a better choice.
   virtual void RenderProcessGone(base::TerminationStatus status) {}
 
   // This method is invoked when a WebContents swaps its visible RenderViewHost
   // with another one, possibly changing processes. The RenderViewHost that has
   // been replaced is in |old_host|, which is nullptr if the old RVH was shut
   // down.
   virtual void RenderViewHostChanged(RenderViewHost* old_host,
                                      RenderViewHost* new_host) {}
 
   // This method is invoked after the WebContents decides which RenderFrameHost
@@ skipping 290 matching lines @@
   void ResetWebContents();
 
   WebContentsImpl* web_contents_;
 
   DISALLOW_COPY_AND_ASSIGN(WebContentsObserver);
 };
 
 }  // namespace content
 
 #endif  // CONTENT_PUBLIC_BROWSER_WEB_CONTENTS_OBSERVER_H_