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

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

Issue 984963004: <webview>: Implement fullscreen permission for html5 element.requestFullscreen() (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@tmptmptmp
Patch Set: Clean up for review. Created 5 years, 9 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: extensions/browser/guest_view/guest_view_base.h
diff --git a/extensions/browser/guest_view/guest_view_base.h b/extensions/browser/guest_view/guest_view_base.h
index 231275f9cc1a7b499edb2cdb5494611222914e7a..a1bcc9c4012d936395ab184a8b73fdad03da8f34 100644
--- a/extensions/browser/guest_view/guest_view_base.h
+++ b/extensions/browser/guest_view/guest_view_base.h
@@ -124,6 +124,12 @@ class GuestViewBase : public content::BrowserPluginGuestDelegate,
// work.
virtual void GuestReady() {}
+ // This method is called when emebedder WebContents's fullscreen is toggled.
Fady Samuel 2015/03/13 01:43:06 typo: embedder.
lazyboy 2015/03/13 16:27:02 Done.
+ //
+ // If the guest asked the embedder to go fullscreen, guests use this signal to
+ // get out of fullscreen state.
+ virtual void EmbedderFullscreenToggled(bool entered_fullscreen) {}
+
// This method is invoked when the contents auto-resized to give the container
// an opportunity to match it if it wishes.
//
@@ -347,7 +353,7 @@ class GuestViewBase : public content::BrowserPluginGuestDelegate,
void UpdateTargetURL(content::WebContents* source, const GURL& url) override;
private:
- class OwnerLifetimeObserver;
+ class OwnerContentsObserver;
class OpenerLifetimeObserver;
@@ -414,7 +420,7 @@ class GuestViewBase : public content::BrowserPluginGuestDelegate,
// This observer ensures that this guest self-destructs if the embedder goes
// away.
- scoped_ptr<OwnerLifetimeObserver> owner_lifetime_observer_;
+ scoped_ptr<OwnerContentsObserver> owner_contents_observer_;
// This observer ensures that if the guest is unattached and its opener goes
// away then this guest also self-destructs.

Powered by Google App Engine
This is Rietveld 408576698