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

Unified Diff: components/guest_view/browser/guest_view_base.h

Issue 2417693002: Allow MimeHandlerViewGuest be embedded inside OOPIFs (Closed)
Patch Set: Rebased Created 4 years, 1 month 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 | « chrome/test/data/page_with_embedded_pdf.html ('k') | components/guest_view/browser/guest_view_base.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: components/guest_view/browser/guest_view_base.h
diff --git a/components/guest_view/browser/guest_view_base.h b/components/guest_view/browser/guest_view_base.h
index 320a5e4d50b269831aeb861b657a7c3d3f4646df..3e6eac1ede32dc7d27d03dde948b73664a51fedf 100644
--- a/components/guest_view/browser/guest_view_base.h
+++ b/components/guest_view/browser/guest_view_base.h
@@ -200,6 +200,10 @@ class GuestViewBase : public content::BrowserPluginGuestDelegate,
void SetAttachParams(const base::DictionaryValue& params);
void SetOpener(GuestViewBase* opener);
+ // BrowserPluginGuestDelegate implementation.
+ content::RenderWidgetHost* GetOwnerRenderWidgetHost() override;
+ content::SiteInstance* GetOwnerSiteInstance() override;
+
protected:
explicit GuestViewBase(content::WebContents* owner_web_contents);
@@ -208,6 +212,12 @@ class GuestViewBase : public content::BrowserPluginGuestDelegate,
// BrowserPluginGuestDelegate implementation.
void SetContextMenuPosition(const gfx::Point& position) override;
+ // TODO(ekaramad): If a guest is based on BrowserPlugin and is embedded inside
+ // a cross-process frame, we need to notify the destruction of the frame so
+ // that the clean-up on the browser side is done appropriately. Remove this
+ // method when BrowserPlugin is removed (https://crbug.com/535197).
+ virtual void OnRenderFrameHostDeleted(int process_id, int routing_id);
+
// WebContentsDelegate implementation.
void HandleKeyboardEvent(
content::WebContents* source,
« no previous file with comments | « chrome/test/data/page_with_embedded_pdf.html ('k') | components/guest_view/browser/guest_view_base.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698