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

Unified Diff: Source/web/FullscreenController.cpp

Issue 428633004: Webkit setting for embedders that do not support fullscreen. (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@master
Patch Set: Created 6 years, 5 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: Source/web/FullscreenController.cpp
diff --git a/Source/web/FullscreenController.cpp b/Source/web/FullscreenController.cpp
index 6c1aa5fe4a9e3dca85325a4b3473fe0b7566513e..e45046097fbbd38aa33cdbca86658406a88d5bda 100644
--- a/Source/web/FullscreenController.cpp
+++ b/Source/web/FullscreenController.cpp
@@ -153,6 +153,13 @@ void FullscreenController::didExitFullScreen()
m_fullScreenFrame.clear();
}
+bool FullscreenController::fullscreenEnabled(blink::Element*)
+{
+ // TODO(igsolla): add check for disallowFullscreenForNonMediaElements here once
+ // https://codereview.chromium.org/416013002/ is reverted.
+ return m_webViewImpl->settingsImpl()->fullscreenEnabled();
+}
+
void FullscreenController::enterFullScreenForElement(blink::Element* element)
{
// We are already transitioning to fullscreen for a different element.

Powered by Google App Engine
This is Rietveld 408576698