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

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: Removed AddGuest/RemoveGuest from content API! w00t! 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..9cd7325392930e39884d62a76f954681a005007d 100644
--- a/content/public/browser/browser_plugin_guest_manager_delegate.h
+++ b/content/public/browser/browser_plugin_guest_manager_delegate.h
@@ -10,41 +10,39 @@
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();
- // Adds a new WebContents |guest_web_contents| as a guest.
- // TODO(fsamuel): Remove this. Once guest WebContents allocation
- // moves outside of content, this API will be unnecessary.
- virtual void AddGuest(int guest_instance_id,
- WebContents* guest_web_contents) {}
-
- // Removes a |guest_instance_id| as a valid guest.
- // TODO(fsamuel): Remove this. Once guest WebContents allocation
- // moves outside of content, this API will be unnecessary.
- virtual void RemoveGuest(int guest_instance_id) {}
-
typedef base::Callback<void(WebContents*)> GuestByInstanceIDCallback;
virtual void MaybeGetGuestByInstanceIDOrKill(
int guest_instance_id,
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