New issue
Advanced search Search tips

Issue 621071 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner: ----
Closed: Jun 2016
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Android
Pri: 2
Type: Bug



Sign in to add a comment

Fullscreen mode full white screen

Reported by noah.sei...@huffingtonpost.com, Jun 17 2016

Issue description

Steps to reproduce the problem:
1. Install Huffington Post app
2. Open any article with a video
3. Play video, select standard fullscreen mode once available

What is the expected behavior?
I uninstalled the latest Android System  WebView updates and fullscreen mode works as expected. Re-installing the latest updates causes fullscreen mode to show a completely white screen with no video playback.

What went wrong?
Using an older Android System WebView fullscreen videos work as expected. Using the up to date Android System WebView fullscreen video no longer works.

Did this work before? Yes The previous version of Android System WebView

Chrome version: 51.0.2704.81  Channel: stable
OS Version: 6.0.1
Flash Version: N/A
 
Components: Mobile>WebView
Issue reproducible on Samsung Galaxy S6/MMB29K, webview: 51.0.2704.81

Logs @ http://go/chrome-androidlogs1/6/621071

files added: S6_621071_logcat.txt
             s6_621071_vid.mp4

Comment 3 by boliu@chromium.org, Jun 17 2016

how about nexus device?
boliu@ Also repros on Nexus 6/6.0.1, webview: 51.0.2704.81.


video and logcat added for Nexus6 @ http://go/chrome-androidlogs1/6/621071

Comment 6 by boliu@chromium.org, Jun 17 2016

hmm, local trunk build seems ok..

Comment 7 by boliu@chromium.org, Jun 17 2016

current beta (52.0.2743.41) is ok, so could just wait until m52 goes out to stable

still kinda bugs me why this can happen in the first place though..

Comment 8 by boliu@chromium.org, Jun 17 2016

Cc: tobiasjs@chromium.org
Components: -UI
Status: Fixed (was: Unconfirmed)
Yeah, m52 definitely fixed this. There are a few came together

* mAwGLFunctor.requestDrawGLForCanvas(canvas) is returning false. this causes the fullscreen view to not draw at all, which means it remains white and opaque, not letting the video view below it show through

* Why is requestDrawGLForCanvas failing? It's because mAwGLFunctor is using the *wrong* view. It's still pointing to the webview, not the full screen view. This is fixed already in m52

* Why is requestDrawGLForCanvas failing, because the view shouldn't matter in the first place? Well, we check canInvokeDrawGlFunctor first, and that's returning false for the webview. We should not need to check for invoke GL if we are doing DrawGLForCanvas. But this is not wrong if it wasn't for the issue above.

App can workaround this by not detaching the webview while the full screen view, ie the one from onShowCustomView, is showing. Or wait until m52.

And you probably should start testing beta channel. It's too late for us to fix issues if we already pushed an update to stable.

Sign in to add a comment