Reland: [Extensions] Change ProcessManager to use RenderFrameHosts

(Reland of https://codereview.chromium.org/1037263004)

RenderViewHosts are going to go away, and things using them are blocking
OOPIs. Make ProcessManager use RenderFrameHosts.
Additionally, this takes out ~130 lines of code.

BUG=466373
(TBRing other reviewers from original patch, since no code there changed)
TBR=avi@chromium.org
TBR=asargent@chromium.org
TBR=atwilson@chromium.org
TBR=jamescook@chromium.org
TBR=dgozman@chromium.org
TBR=nick@chromium.org

Review URL: https://codereview.chromium.org/1056463005

Cr-Commit-Position: refs/heads/master@{#323803}

1 files changed, 7 insertions, 0 deletions

diff --git a/extensions/browser/extension_web_contents_observer.h b/extensions/browser/extension_web_contents_observer.h
index cd48560..993a8fe 100644
--- a/extensions/browser/extension_web_contents_observer.h
+++ b/extensions/browser/extension_web_contents_observer.h
@@ -42,6 +42,13 @@ class ExtensionWebContentsObserver : public content::WebContentsObserver {
 
   void RenderFrameCreated(content::RenderFrameHost* render_frame_host) override;
 
+  // Per the documentation in WebContentsObserver, these two methods are
+  // appropriate to track the set of current RenderFrameHosts.
+  // NOTE: FrameDeleted() != RenderFrameDeleted().
+  void FrameDeleted(content::RenderFrameHost* render_frame_host) override;
+  void RenderFrameHostChanged(content::RenderFrameHost* old_host,
+                              content::RenderFrameHost* new_host) override;
+
   // Returns the extension or app associated with a render view host. Returns
   // NULL if the render view host is not for a valid extension.
   const Extension* GetExtension(content::RenderViewHost* render_view_host);