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

Issue 645000 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Sep 2016
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: All
Pri: 1
Type: Bug-Regression

Blocking:
issue 628968



Sign in to add a comment

Video poster is not visible

Project Member Reported by klo...@chromium.org, Sep 8 2016

Issue description

Load https://mobile.twitter.com/Apple/status/773604633008680964 in Chrome Dev (M54) on Android.

Noticed the big white space in the middle of the page.

If you tap it, it will load video.

If you load the same page on Chrome Stable (M52), the poster image shows up.

Didn't try Chrome Beta.
 
Cc: dalecur...@chromium.org foolip@chromium.org
Owner: mlamouri@chromium.org
over to mlamouri@ and cc:foolip@ for any potential poster changes. First frame of the video is black, so it shouldn't be the first frame showing up.
So this is a regression between M52 and M54? I don't recall any recent changes to poster, but a bisect should quickly find the truth :)
Labels: -OS-Android -Type-Bug OS-All Type-Bug-Regression
This isn't Android specific. I can reproduce the same problem on desktop Chrome (Linux).

Working on M53 (53.0.2785.101)
Not working on M54 (54.0.2816.0)
Blocking: 628968
Cc: ramy...@samsung.com mlamouri@chromium.org
Components: -Blink>Media>Video Blink>Layout
Labels: ReleaseBlock-Beta
Owner: timloh@chromium.org
It seems to be a regression from https://codereview.chromium.org/2169343002 according to `git bisect` and reverting commit b76fc4469acf4c44336dd8d42d1e127c8823d08b on top of tree fixes the issue.

Twitter doesn't use a poster and instead shows an image. I guess, they were relying on the previous behaviour?
Can we revert the patch ASAP, we are planning a Beta Release soon.
Cc: ligim...@chromium.org bustamante@chromium.org
Labels: hasbisect-per-revison
This is broken in M54 and works fine in M53. Please revert the patch ASAP. This is blocking the scheduled Beta Release tomorrow.

You are probably looking for a change made after 407434 (known good), but no later than 407435 (first known bad).

CHANGELOG URL:
==============  https://chromium.googlesource.com/chromium/src/+log/48c3ff1de4496dc724db0b604c6ed5cc1b20c60c..b76fc4469acf4c44336dd8d42d1e127c8823d08b
Labels: -ReleaseBlock-Beta
Working as intended in - 54.0.2840.27 (Official Build) beta (64-Bit), MAC 10.11

Reverted patch: https://codereview.chromium.org/2332373003/

Comment 8 by timloh@chromium.org, Sep 21 2016

Status: Fixed (was: Assigned)
Has been reverted/merged, see 628968.
This was fixed by Twitter, fwiw:

.InlineMedia {
	border-radius:.4rem;
	-webkit-mask-image:url(data:image/png;base64,R0lGODlhAQABAIAAAP///////yH5BAEKAAEALAAAAAABAAEAAAICRAEAOw==);
	-webkit-mask-repeat:repeat;
	overflow:hidden
}

Gecko ran into similar issues and we asked twitter to update (and they did very quickly).

Comment 10 by c...@mozilla.com, Feb 8 2017

issue no on w3c github: 
https://github.com/w3c/csswg-drafts/issues/599


Sign in to add a comment