New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 709367 link

Starred by 1 user

Issue metadata

Status: WontFix
Owner:
Closed: Nov 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux , Windows , Mac
Pri: 1
Type: Bug-Regression

Blocked on:
issue 402376



Sign in to add a comment

Regression: Video doesn't render properly in fullscreen mode on 'THE TIMES OF INDIA' page.

Reported by dchau...@etouch.net, Apr 7 2017

Issue description

Chrome Version: 59.0.3065.0 (Official Build) (64-bit) (cohort: Stable) 1d73b1aebe5f3a978a5a87c7502689b49ba1b134-refs/heads/master@{#462712} 32/64-bit
OS: Windows (7,8,10), Mac (10.11.6, 10.12.1), Linux (Ubuntu 14.04 LTS)

What steps will reproduce the problem?
1. Launch chrome and go to http://timesofindia.indiatimes.com/
2. Scroll down the page and play any video under 'VIDEOS' section.
3. Now, click on full screen button of video and observe.

Video doesn't render properly in fullscreen mode.
Video should render properly in fullscreen mode.

This is a regression issue, broken in M-59 series, below is manual regression range.

Good build: 59.0.3056.0
Bad build: 59.0.3057.0 

Kindly review the attached screen-cast for reference.
 
Actual behavior.mp4
2.6 MB View Download
Expected behavior.mp4
2.5 MB View Download
Labels: hasbisect-per-revision ReleaseBlock-Stable
Owner: erikc...@chromium.org
Status: Assigned (was: Unconfirmed)
Using the per-revision bisect providing the bisect results,
Good build: 59.0.3056.0 (Revision: 460603).
Bad build : 59.0.3057.0 (Revision: 460966).

You are probably looking for a change made after 460645 (known good), but no later than 460646 (first known bad).
CHANGELOG URL:
  https://chromium.googlesource.com/chromium/src/+log/174eb44312202a5132de900553f7c302da0feef6..85bab14c835935e79a0b63f24b47f671840373de

@erikchen: Could you please look into the issue, pardon me if it has nothing to do with your changes and if possible please assign it to concern owner.
Adding RB Label as this is a recent Regression. Please remove if not required.
Thank You.
Cc: mlamouri@chromium.org
Status: Untriaged (was: Assigned)
Components: Blink>Layout
Components: Blink>Fullscreen

Comment 6 Deleted

Comment 7 Deleted

Still we are able to reproduce the issue on Mac-10.12.4 using chrome version 60.0.3073.0.
@erikchen could you please look into this issue.

Thanks.
erikchen@ Since this issue is marked as RB-Stable can we get any latest update available on this issue?

Thanks!
Labels: Needs-triage-Mobile
Components: -Blink>Media>Video
Tested this issue on Android 7.0.99; Build/MRA20 using chrome version #59.0.3065.0 by following steps mentioned below.

1.Launched chrome and navigated to http://timesofindia.indiatimes.com/
2.Scrolled down till the video page and clicked on video
3.Observed the video buffers for longtime and no video is displayed to check this issue

Tested the same on mobile site m.timesofindia.com and observed able to play the videos in full screen with no issues.
Cc: -foolip@chromium.org erikc...@chromium.org
Owner: foolip@chromium.org
foolip: This seems likely to be the same fullscreen issue that we observed before? What's the status of the blink fullscreen refactor?
The feature in question is still experimental, so this shouldn't affect Chrome stable. Have you observed otherwise?
Owner: erikc...@chromium.org
#13, I'm currently working on  issue 402376 , specifically on updating https://fullscreen.spec.whatwg.org/ with a bunch of changes into something that would allow it to be relanded without causing the same regressions as last time.

Since my changes weren't in the regressions range, assigning back.
Reminder that M59 Stable is launch is coming soon (less than 2 weeks)! Your bug is labelled as Stable ReleaseBlock, pls make sure to land the fix and get it merged into the release branch ASAP so it gets enough baking time in Beta (before Stable promotion). Thank you!
erikchen@, foolip@ - If this is part of an experiment, can we remove RB-Stable label? (we're about a week away from M59 stable) 
Blockedon: 402376
Labels: -ReleaseBlock-Stable
The CL that causes this issue is for a feature that has status=experimental
https://codereview.chromium.org/2743053003/diff/280001/third_party/WebKit/Source/platform/RuntimeEnabledFeatures.json5

So it shouldn't be RB-S. This problem [should] be fixed by the new fullscreen implementation.
Labels: -Needs-triage-Mobile

Comment 20 by e...@chromium.org, Nov 30 2017

Can we close this now that 402376 has been resolved?
Status: WontFix (was: Assigned)
Yes, closing as Invalid.

Sign in to add a comment