Merge pull request #2021 from brave/fullscreen

enable escape for exit of webview fullscreen

js/components/frame.js

@@ -207,6 +207,17 @@ class Frame extends ImmutableComponent {
       if (this.props.isActive && !prevProps.isActive) {
         this.webview.focus()
       }
+
+      // make sure the webview content updates to
+      // match the fullscreen state of the frame
+      if (prevProps.frame.get('isFullScreen') !==
+         this.props.frame.get('isFullScreen')) {
+        if (this.props.frame.get('isFullScreen')) {
+          this.webview.executeJavaScript('document.webkitRequestFullscreen()')
+        } else {
+          this.webview.executeJavaScript('document.webkitExitFullscreen()')
+        }
+      }
       this.webview.setAudioMuted(this.props.frame.get('audioMuted') || false)
       this.updateAboutDetails()
     }
@@ -526,6 +537,7 @@ class Frame extends ImmutableComponent {
     })
     this.webview.addEventListener('enter-html-full-screen', () => {
       windowActions.setFullScreen(this.props.frame, true, true)
+      // disable the fullscreen warning after 5 seconds
       setTimeout(windowActions.setFullScreen.bind(this, this.props.frame, undefined, false), 5000)
     })
     this.webview.addEventListener('leave-html-full-screen', () => {

js/components/main.js

@@ -78,6 +78,9 @@ class Main extends ImmutableComponent {
     const isDarwin = process.platform === 'darwin'
     document.addEventListener('keydown', (e) => {
       switch (e.which) {
+        case keyCodes.ESC:
+          this.exitFullScreen()
+          break
         case keyCodes.F12:
           if (!isDarwin) {
             ipc.emit(messages.SHORTCUT_ACTIVE_FRAME_TOGGLE_DEV_TOOLS)
@@ -96,6 +99,14 @@ class Main extends ImmutableComponent {
       }
     })
   }
+
+  exitFullScreen () {
+    const activeFrame = FrameStateUtil.getActiveFrame(this.props.windowState)
+    if (activeFrame && activeFrame.get('isFullScreen')) {
+      windowActions.setFullScreen(activeFrame, false)
+    }
+  }
+
   registerSwipeListener () {
     // Navigates back/forward on OS X two-finger swipe
     var trackingFingers = false
@@ -129,12 +140,7 @@ class Main extends ImmutableComponent {
       deltaY = 0
       startTime = 0
     })
-    ipc.on(messages.LEAVE_FULL_SCREEN, () => {
-      const activeFrame = FrameStateUtil.getActiveFrame(this.props.windowState)
-      if (activeFrame && activeFrame.get('isFullScreen')) {
-        windowActions.setFullScreen(activeFrame, false)
-      }
-    })
+    ipc.on(messages.LEAVE_FULL_SCREEN, this.exitFullScreen.bind(this))
   }
 
   loadOpenSearch () {