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

Unified Diff: third_party/WebKit/Source/web/WebRemoteFrameImpl.cpp

Issue 2565203002: Add a requestFullscreen variant with a default (prefixed) type (Closed)
Patch Set: Created 4 years 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: third_party/WebKit/Source/web/WebRemoteFrameImpl.cpp
diff --git a/third_party/WebKit/Source/web/WebRemoteFrameImpl.cpp b/third_party/WebKit/Source/web/WebRemoteFrameImpl.cpp
index 7cff1d54dd6f473c7644a9cd2d31287ad251c98c..562ce9f5cfdb0822a297f14d4eadaa4f0c3ca007 100644
--- a/third_party/WebKit/Source/web/WebRemoteFrameImpl.cpp
+++ b/third_party/WebKit/Source/web/WebRemoteFrameImpl.cpp
@@ -524,9 +524,9 @@ void WebRemoteFrameImpl::willEnterFullscreen() {
// TODO(alexmos): currently, this assumes prefixed requests, but in the
// future, this should plumb in information about which request type
// (prefixed or unprefixed) to use for firing fullscreen events.
- Fullscreen::from(ownerElement->document())
- .requestFullscreen(*ownerElement, Fullscreen::PrefixedRequest,
- true /* forCrossProcessAncestor */);
+ Fullscreen::requestFullscreen(*ownerElement,
+ Fullscreen::RequestType::Prefixed,
+ true /* forCrossProcessAncestor */);
}
WebRemoteFrameImpl::WebRemoteFrameImpl(WebTreeScopeType scope,
« no previous file with comments | « third_party/WebKit/Source/web/WebPluginContainerImpl.cpp ('k') | third_party/WebKit/Source/web/tests/WebFrameTest.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698