commit 435c5666f334eef2a59b6cc28532ca4af0c62b74
author Ignacio Solla <igsolla@google.com> Mon Nov 17 22:01:57 2014 +0000
committer Ignacio Solla <igsolla@google.com> Fri Nov 21 20:32:32 2014 +0000
tree a3e9f625d9b0b028629300444741cdf04c97a3f9
parent eb33a44eef586f62300547e65a8f666276540b64

[WebView] Set back button to exit fullscreen.

This patch fixes a regression from K to L. Tapping the back button
while in fullscreen should exit fullscreen instead of the current
activity.

BUG:17985005
Change-Id: I7a5eb4edebb54ce1e7d6a2b578a87dd7e7de9dbf

android_webview/java/src/org/chromium/android_webview/AwContentViewClient.java

diff --git a/android_webview/java/src/org/chromium/android_webview/AwContentViewClient.java b/android_webview/java/src/org/chromium/android_webview/AwContentViewClient.java
index a091d77..cdccba9 100644
--- a/android_webview/java/src/org/chromium/android_webview/AwContentViewClient.java
+++ b/android_webview/java/src/org/chromium/android_webview/AwContentViewClient.java
@@ -38,6 +38,21 @@
                     }
                     View fullscreenView = mAwContents.enterFullScreen();
                     if (fullscreenView != null) {
+                        fullscreenView.setOnKeyListener(new View.OnKeyListener() {
+                            @Override
+                            public boolean onKey(View v, int keyCode, KeyEvent event) {
+                                if (event.getKeyCode() == KeyEvent.KEYCODE_BACK
+                                        && event.getAction() == KeyEvent.ACTION_UP
+                                        && mAwContents.isFullScreen()) {
+                                    exitFullscreen();
+                                    return true;
+                                }
+                                return false;
+                            }
+                        });
+                        fullscreenView.setFocusable(true);
+                        fullscreenView.setFocusableInTouchMode(true);
+                        fullscreenView.requestFocus();
                         viewGroup.addView(fullscreenView);
                     }
                 }
@@ -45,15 +60,19 @@
             WebChromeClient.CustomViewCallback cb = new WebChromeClient.CustomViewCallback() {
                 @Override
                 public void onCustomViewHidden() {
-                    ContentVideoView contentVideoView = ContentVideoView.getContentVideoView();
-                    if (contentVideoView != null)
-                        contentVideoView.exitFullscreen(false);
+                    exitFullscreen();
                 }
             };
             mAwContentsClient.onShowCustomView(viewGroup, cb);
             return true;
         }
 
+        private void exitFullscreen() {
+            ContentVideoView contentVideoView = ContentVideoView.getContentVideoView();
+            if (contentVideoView != null)
+                contentVideoView.exitFullscreen(false);
+        }
+
         @Override
         public void onDestroyContentVideoView() {
             mAwContents.exitFullScreen();