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

Unified Diff: content/public/browser/browser_plugin_guest_manager_delegate.h

Issue 261013005: BrowserPlugin: Move CreateGuest to chrome (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@guestview_manager_simplify_api
Patch Set: Created 6 years, 8 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: content/public/browser/browser_plugin_guest_manager_delegate.h
diff --git a/content/public/browser/browser_plugin_guest_manager_delegate.h b/content/public/browser/browser_plugin_guest_manager_delegate.h
index 28a8fa60864b3c80c9430780b6ab72e231a59afe..5880ef454d9ea8eb07d2bcbbc07ff0c45ee4776d 100644
--- a/content/public/browser/browser_plugin_guest_manager_delegate.h
+++ b/content/public/browser/browser_plugin_guest_manager_delegate.h
@@ -10,15 +10,30 @@
class GURL;
+namespace base {
+class DictionaryValue;
+} // namespace base
+
namespace content {
class SiteInstance;
class WebContents;
+struct StorageInfo {
+ bool persist;
+ std::string partition_id;
+};
+
class CONTENT_EXPORT BrowserPluginGuestManagerDelegate {
public:
virtual ~BrowserPluginGuestManagerDelegate() {}
+ virtual content::WebContents* CreateGuest(
+ content::SiteInstance* embedder_site_instance,
+ int instance_id,
+ const StorageInfo& storage_info,
+ scoped_ptr<base::DictionaryValue> extra_params);
+
// Return a new instance ID.
virtual int GetNextInstanceID();
@@ -39,12 +54,6 @@ class CONTENT_EXPORT BrowserPluginGuestManagerDelegate {
int embedder_render_process_id,
const GuestByInstanceIDCallback& callback) {}
- // Returns an existing SiteInstance if the current profile has a guest of the
- // given |guest_site|.
- // TODO(fsamuel): Remove this. Once guest WebContents allocation
- // moves outside of content, this API will be unnecessary.
- virtual content::SiteInstance* GetGuestSiteInstance(const GURL& guest_site);
-
// Iterates over all WebContents belonging to a given |embedder_web_contents|,
// calling |callback| for each. If one of the callbacks returns true, then
// the iteration exits early.

Powered by Google App Engine
This is Rietveld 408576698