Bug 58291 – REGRESSION: Vimeo fullscreen video displays incorrectly

• Component: Media
• Status: RESOLVED FIXED

[Jer Noble]

4/9/11 8:36 AM Simon Fraser:
If I go to http://vimeo.com/21864555 (in HTML5 mode), start playing the video, and then take the video fullscreen, the fullscreen presentation is totally messed up. The video is unscaled, and there are bits of the page scattered around.

WebKit nightly r83321 on 10.6.7

[Jer Noble]

<rdar://problem/9261869>

[Jer Noble]

Created attachment 89160 [details]
Patch

[Simon Fraser (smfr)]

I'm concerned that we're adding a lot of special magic here. Is this z-index issue discussed in the fullscreen spec?

[Jer Noble]

(In reply to comment #3)
> I'm concerned that we're adding a lot of special magic here. Is this z-index issue discussed in the fullscreen spec?

The spec specifically suggests setting the zIndex to 2^31-1.  So there is no "special magic" here.

https://wiki.mozilla.org/Gecko:FullScreenAPI

[Simon Fraser (smfr)]

Comment on attachment 89160 [details]
Patch

Is there a way to test the z-index issue? Do pixel tests work for the fullscreen window?

[Jer Noble]

Pixel tests of fullscreen content should work in DRT; it doesn't use WebKit's full screen window, and it doesn't do any animation, but the test shouldn't require that.  I'll add one.

Thanks!

[Jer Noble]

Created attachment 89242 [details]
Patch

[Simon Fraser (smfr)]

Comment on attachment 89242 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=89242&action=review

> Source/WebCore/css/fullscreen.css:16
> +    background-color: black ! important;
> +    position: static ! important;
> +    margin: 0 ! important;
> +    height: 100% ! important;
> +    width: 100% ! important;
> +    -webkit-box-flex: 1 ! important;
> +    display: block ! important;

We don't put a space after the ! normally.

> Source/WebCore/html/HTMLMediaElement.cpp:1495
> +    // always show controls when in full screen mode.

Should use sentence case.

[Jer Noble]

Committed r83654: <http://trac.webkit.org/changeset/83654>

Created attachment 89619 [details]
Fullscreen works, but scrollbar fails to hide and controls are not centered.

Attached screenshot of vimeo video fullscreen on release r83831.

[Edward Sabol]

Should this bug be reopened due to the issues described in comment #10 and comment #11 or is there a follow-up bug?

[Edward Sabol]

Created attachment 90888 [details]
Screenshot with r84622

Shortly after launching the latest Webkit Nightly (r84622), I went to a Vimeo page and clicked on the fullscreen button prior to playing the video. This is what I see. If I start playing the video and click the fullscreen button a couple times, it will eventually go fullscreen. (Also, the scroll bar and non-centered controls noted in comment #10 and comment #11 appear to no longer be an issue with r84622.)

Created attachment 90890 [details]
Fullscreen Working Properly on r84622

Fullscreen is working properly for me ( MBPRO 13 2009 ) on  r84622.

[Edward Sabol]

(In reply to comment #14)
> Created an attachment (id=90890) [details]
> Fullscreen Working Properly on r84622
> 
> Fullscreen is working properly for me ( MBPRO 13 2009 ) on  r84622.

Ah, but you clicked play first. I said I clicked the fullscreen button first.

(In reply to comment #15)
> (In reply to comment #14)
> > Created an attachment (id=90890) [details] [details]
> > Fullscreen Working Properly on r84622
> > 
> > Fullscreen is working properly for me ( MBPRO 13 2009 ) on  r84622.
> 
> Ah, but you clicked play first. I said I clicked the fullscreen button first.

My fullscreen issues with the visible scrollbar and player alignment have been resolved.

I tried to press the fullscreen button prior to play, and had the same results.  I can confirm your issue.

[Jer Noble]

(In reply to comment #16)
> (In reply to comment #15)
> > (In reply to comment #14)
> > > Created an attachment (id=90890) [details] [details] [details]
> > > Fullscreen Working Properly on r84622
> > > 
> > > Fullscreen is working properly for me ( MBPRO 13 2009 ) on  r84622.
> > 
> > Ah, but you clicked play first. I said I clicked the fullscreen button first.
> 
> My fullscreen issues with the visible scrollbar and player alignment have been resolved.
> 
> I tried to press the fullscreen button prior to play, and had the same results.  I can confirm your issue.

Can either of you verify that Vimeo actually entered the new full screen mode?  Both they and YouTube have a "pseudo" full screen mode which just takes up the current window.

You can verify this by using the Inspector and checking the output of document.webkitCurrentFullScreenElement.

Created attachment 90893 [details]
document.webkitCurrentFullScreenElement Output

document.webkitCurrentFullScreenElement: null
document.webkitIsFullScreen: false

Created attachment 90894 [details]
Youtube fails to go fullscreen

document.webkitCurrentFullScreenElement: null
document.webkitIsFullScreen: false

It looks like youtube is using the pseudo fullscreen.