diff options
author | rdevlin.cronin <rdevlin.cronin@chromium.org> | 2015-04-02 14:25:35 -0700 |
---|---|---|
committer | Commit bot <commit-bot@chromium.org> | 2015-04-02 21:25:59 +0000 |
commit | 93a2547e8519ce060379102a611d58f767c72ece (patch) | |
tree | 8ca75b24a9c8f14076765d19bd51c2cda334f8b7 /extensions/browser/extension_web_contents_observer.h | |
parent | ca43dfb7b9f5209ef5d5912522f015d4d617012b (diff) | |
download | chromium_src-93a2547e8519ce060379102a611d58f767c72ece.zip chromium_src-93a2547e8519ce060379102a611d58f767c72ece.tar.gz chromium_src-93a2547e8519ce060379102a611d58f767c72ece.tar.bz2 |
[Extensions] Change ProcessManager to use RenderFrameHosts
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
Review URL: https://codereview.chromium.org/1037263004
Cr-Commit-Position: refs/heads/master@{#323565}
Diffstat (limited to 'extensions/browser/extension_web_contents_observer.h')
-rw-r--r-- | extensions/browser/extension_web_contents_observer.h | 7 |
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); |