Exit fullscreen issue on Android

[Xie, William]

Hi

 Does anyone meet this issue?

When exit from full screen video mode, the content video view was still there, see the screen shot below:

 

 

Is it a bug?

 

William

[Aurimas Liutikas]

ChromeShell is meant to be for testing so some UI bugs are supposed to be expected. Can you reproduce this with official Chrome that ships on the play store?

--
--
Chromium Developers mailing list: chromi...@chromium.org
View archives, change email options, or unsubscribe:
http://groups.google.com/a/chromium.org/group/chromium-dev

[Xie, William]

No,

Official Chrome works fine.

 

Should somebody fix the UI bugs in ChromeShell?

 

William

[Anton Vayvod]

It could be a hidden bug that might only rarely present itself in the future Chrome for Android releases.

To unsubscribe from this group and stop receiving emails from it, send an email to chromium-dev...@chromium.org.

[Xie, William]

Following Min’s suggestion, it works for ChromeShell now.

Thank you so much to Min and all.

 

William

 

 

From: qin...@google.com [mailto:qin...@google.com] On Behalf Of Min Qin
Sent: Tuesday, January 06, 2015 11:25 AM
To: ava...@chromium.org
Cc: Xie, William; Aurimas Liutikas; chromi...@chromium.org; trc...@chromium.org
Subject: Re: [chromium-dev] Exit fullscreen issue on Android

 

In official chrome build, FullscreenManager.setPersistentFullscreenMode() will remove the video view when exiting the fullscreen video.

However, for content shell and chrome shell,  FullscreenManager is not created, so the videoview will not be destroyed. 

We need to remove the surfaceview somewhere in ChromeShellTab when toggleFullscreenModeForTab is called.

 

Min

[王荣岗]

Hi,

  I meet other situation.The location of the video is incorrect when clicking the exitfullscreenbutton from fullscreen on YouTube on Android.

  It is incorrect on Android device 6.0 ( 32 bit & 480*854).It is normal when I test on Android device 6.0( 64 bit & 720*1280).Please see the attachments.

Could you help to give some advice? I use the AOSP Browser+ SystemWebView.apk.

在 2015年1月5日星期一 UTC+8下午4:55:33，Xie, William写道：

[Xie, William]

The issues I met before was fixed by https://codereview.chromium.org/833363002/

Please make sure your browser is newer than this CL.

 

Are your issue reproducible on other product like Nexus?

 

Thanks

William

--

--
Chromium Developers mailing list: chromi...@chromium.org
View archives, change email options, or unsubscribe:
http://groups.google.com/a/chromium.org/group/chromium-dev
---

You received this message because you are subscribed to the Google Groups "Chromium-dev" group.

[王荣岗]

The CL maybe fix chrome_shell and content_shell,does it work for android_webview?

The webivew version is 48.0.2531.0, whch should be newer than this CL.

I will test on Nexus and tell you the result. Thank you for your help！

在 2015年12月18日星期五 UTC+8下午6:32:22，Xie, William写道：

[Xie, William]

Are you using android_webview or system_webview?

Which browser are you using?

Is it a bug?

 

William

--
--
Chromium Developers mailing list: chromi...@chromium.org
View archives, change email options, or unsubscribe:
http://groups.google.com/a/chromium.org/group/chromium-dev
---
You received this message because you are subscribed to the Google Groups "Chromium-dev" group.
To unsubscribe from this group and stop receiving emails from it, send an email to chromium-dev...@chromium.org.

[王荣岗]

Dear William:

        I use the SystemWebView.apk with version 48.0.2531.0 & 49.0.2579.2 and the Browser is android 6.0 AOSP Browser. I will try to get the log with the Nexus.

        In additional,it is OK to click the Back key to exit fullscreen. 

        Thank you!

[王荣岗]

 The SystemWebView and AOSP Browser are installed on Nexus 6 (android 6.0).But the Browser always use the AOSP WebView (44.0.2403.117),don't use the SystemWebView(48 or 49).Even though I disable the AOSP WebView in the System Setting,the Browser also use the AOSP WebView. I don't know the reason.

[王荣岗]

Please see the video youtube.mp4.The webview is built by myself base on android system webview 48.0.2531.0. I also test the other Android SystemWebview from site(http://www.apkmirror.com/?s=webview&post_type=apps_post),all can reproduce this on my phone.It is normal to exit fullscreen with pressing the back key,but it is not OK to click the exit_fullscreen_button.
   I found the VideoLayer's parent layer's position is incorrect,but I can't find anymore.

在 2015年12月18日星期五 UTC+8下午7:30:50，Xie, William写道：

[Xie, William]

Try this…

[王荣岗]

Dear William:

        The issue can reproduce with your Browser_5.0_L. In additional, The pending tree is incorrect between 2201ms and 2202 ms in trace_button_exit_full_screen.html.json. please see the cc::LayerTreeHostImpl in the systrace. I can provider other log which you need. Thanks

在 2015年12月30日星期三 UTC+8下午12:00:43，Xie, William写道：

[Xie, William]

Would you please file a bug at crbug.com with reproduce steps?

[王荣岗]

I have submitted it. Please see https://code.google.com/p/chromium/issues/detail?id=570941 & https://code.google.com/p/chromium/issues/detail?id=558045

Thanks!

在 2016年1月4日星期一 UTC+8上午10:08:37，Xie, William写道：