New issue
Advanced search Search tips

Issue 881776 link

Starred by 1 user
Status: Verified
Owner:
dtapu...@chromium.org
Closed: Sep 17
Cc:
foolip@chromium.org
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux , Android , Windows , Chrome , Mac
Pri: 1
Type: Bug-Regression



Sign in to add a comment

CBS Interactive Video player fails to exit fullscreen

Reported by ngo...@etouch.net, Sep 7 
Device name: Pixel 2(8.0.0/OPD1.170816.010), Nexus 5X 7.1.2/(N2G47W), Samsung J7 6.0.1/MMB29M, Samsung J2 5.1.1/LMY47X
WebView version: 71.0.3545.0
Application: Webview Shell Browser
Application version: 1.0

Test URL: https://download.cnet.com/Inpix-BETA-Power-of-Images/3000-2164_4-77606537.html OR https://tinyurl.com/yc2kgycc

Bisect Information: This is a regression issue broken in 'M-71'

Per-Version bisect information:
Good build: 71.0.3544.0
Bad build: 71.0.3545.0

Regression range: https://chromium.googlesource.com/chromium/src/+log/71.0.3544.0..71.0.3545.0?pretty=fuller&n=10000

Steps to reproduce:
1. Navigate to the above mentioned link > Scroll down and play the video.
2. Enter full screen > Exit full screen > Observe.

Actual result: Unable to exit video from full screen mode.

Expected result: Should be able to exit video from full screen mode.

Comment 1 by rmahanandigari@chromium.org, Sep 7 

Please find the log and Video @ http://go/chrome-androidlogs1/8/881776

Comment 2 by rmahanandigari@chromium.org, Sep 7

Labels: -Pri-3 RegressedIn-71 ReleaseBlock-Stable Target-71 M-71 FoundIn-71 hasbisect-per-revision Pri-1 Type-Bug-Regression
Owner: dtapu...@chromium.org
Status: Assigned (was: Unconfirmed)
Per-CL bisect information:
Good commit: 589184
Bad commit: 589185

Suspect CL:
https://chromium.googlesource.com/chromium/src/+/62898a131761fbeeac3f2b2f7c2c0512f7ca14a9


dtapuska@  Might be it looks like this issue is related to your change. please look into once, if its not related to your change please reassign back to me. 

Thanks!

Comment 3 by dtapu...@chromium.org, Sep 7

Cc: foolip@chromium.org
Components: -Mobile>WebView Blink>Fullscreen
Labels: -Restrict-View-Google OS-Chrome OS-Linux OS-Mac OS-Windows
Summary: CBS Interactive Video player fails to exit fullscreen (was: Unable to exit video from full screen mode in the test URL)
The problem is the CBS interactive fullscreen code is broken. This problem occurs on desktop as well.

They are checking the "fullScreenElement" property on the document to determine if they are in fullscreen or not. But they really should be referencing "fullscreenElement"

I'll try to reach out to CBS.

Comment 4 by dtapu...@chromium.org, Sep 10

Status: ExternalDependency (was: Assigned)
I contacted CBSInteractive they were able to fix the issue in development and it will be fixed in the next release of the video player.

Comment 5 by dtapu...@chromium.org, Sep 10

Labels: -ReleaseBlock-Stable
Removing RBS since this is a site issue and has been triaged and fix on customer site is in the works, hopefully will be fixed before 71 goes stable.

Comment 6 by e...@chromium.org, Sep 17

Status: Fixed (was: ExternalDependency)
Closing as a fix has been deployed and nu further action is required from the blink side.

Comment 7 by battun@chromium.org, Sep 17

Status: Verified (was: Fixed)
Fix issue verified on Latest M71: 71.0.3554.0

Tested Devices:Pixel 2(8.0.0/OPD1.170816.010), Nexus 5X 7.1.2/(N2G47W)

Thanks!
►

Sign in to add a comment