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

Unified Diff: chrome/browser/extensions/api/guest_view/guest_view_internal_api.cc

Issue 458323002: Change GuestViewInternalCreateGuestFunction to create the guest view using the (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: rebaseeeeeee Created 6 years, 4 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
Index: chrome/browser/extensions/api/guest_view/guest_view_internal_api.cc
diff --git a/chrome/browser/extensions/api/guest_view/guest_view_internal_api.cc b/chrome/browser/extensions/api/guest_view/guest_view_internal_api.cc
index f16f353a40eb2879fd157a08d2b2ac8e9b59cd4e..0e898a49a3ab8168de7916394bb02e447ab85941 100644
--- a/chrome/browser/extensions/api/guest_view/guest_view_internal_api.cc
+++ b/chrome/browser/extensions/api/guest_view/guest_view_internal_api.cc
@@ -33,12 +33,19 @@ bool GuestViewInternalCreateGuestFunction::RunAsync() {
GuestViewManager::WebContentsCreatedCallback callback =
base::Bind(&GuestViewInternalCreateGuestFunction::CreateGuestCallback,
this);
+
+ content::WebContents* embedder_web_contents =
+ content::WebContents::FromRenderViewHost(render_view_host());
+ if (!embedder_web_contents) {
+ error_ = "Guest views can only be embedded in web content";
+ return false;
+ }
+
guest_view_manager->CreateGuest(view_type,
extension_id(),
- GetAssociatedWebContents(),
+ embedder_web_contents,
*create_params,
callback);
-
return true;
}

Powered by Google App Engine
This is Rietveld 408576698