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

Unified Diff: extensions/renderer/resources/guest_view/web_view/web_view_events.js

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/renderer/resources/guest_view/web_view/web_view_events.js
diff --git a/extensions/renderer/resources/guest_view/web_view/web_view_events.js b/extensions/renderer/resources/guest_view/web_view/web_view_events.js
index 470b49a22b6ba64db505fedaa884dd21998f9b40..6e35d49c13b972e262451253565a3ad0099f499a 100644
--- a/extensions/renderer/resources/guest_view/web_view/web_view_events.js
+++ b/extensions/renderer/resources/guest_view/web_view/web_view_events.js
@@ -55,6 +55,11 @@ WebViewEvents.EVENTS = {
evt: CreateEvent('webViewInternal.onExit'),
fields: ['processId', 'reason']
},
+ 'exitfullscreen': {
+ evt: CreateEvent('webViewInternal.onExitFullscreen'),
+ fields: ['url'],
+ handler: 'handleFullscreenExitEvent'
Fady Samuel 2015/03/13 01:43:07 add: internal: true
lazyboy 2015/03/13 16:27:02 Done.
+ },
'findupdate': {
evt: CreateEvent('webViewInternal.onFindReply'),
fields: [
@@ -219,6 +224,10 @@ WebViewEvents.prototype.handleFrameNameChangedEvent = function(event) {
this.view.onFrameNameChanged(event.name);
};
+WebViewEvents.prototype.handleFullscreenExitEvent = function(event, eventName) {
+ document.webkitCancelFullScreen();
+};
+
WebViewEvents.prototype.handleLoadAbortEvent = function(event, eventName) {
var showWarningMessage = function(reason) {
var WARNING_MSG_LOAD_ABORTED = '<webview>: ' +
@@ -249,7 +258,12 @@ WebViewEvents.prototype.handleNewWindowEvent = function(event, eventName) {
WebViewEvents.prototype.handlePermissionEvent = function(event, eventName) {
var webViewEvent = this.makeDomEvent(event, eventName);
- new WebViewActionRequests.PermissionRequest(this.view, event, webViewEvent);
+ if (event.permission === 'fullscreen') {
+ new WebViewActionRequests.FullscreenPermissionRequest(
+ this.view, event, webViewEvent);
+ } else {
+ new WebViewActionRequests.PermissionRequest(this.view, event, webViewEvent);
+ }
};
WebViewEvents.prototype.handleSizeChangedEvent = function(event, eventName) {

Powered by Google App Engine
This is Rietveld 408576698