925026 - [B2G][Browser]Enabling full screen mode on Youtube videos does not display properly and prevents user from exiting tab or video normally

Status: RESOLVED DUPLICATE of bug 950993

(Core :: Graphics, defect)

Description

[Montoya Clemmons (MClemmons)]

Description: When user taps youtube video to play in full screen mode, unexpected behavior occurs. Video orientation in landscape and portrait is not aligned properly. User is prevented from selecting the video again to return mode from fullscreen and must exit app by selecting home button

Repro Steps:
1) Update Buri to Build ID: 20131009055629
2) Tap Browser App
3) Go to Youtube.com
4) Conduct search on youtube.com for video
5) Select video
6) Play video
7) While video plays tap on video 
8) Tap Full screen button
9) Tap either share or don't share


Actual: Full screen mode showing the video misaligned and no option to return from full screen mode


Expected:
Full screen mode enabled and aligned properly and user able to select video to return to non full screen mode

Environmental Variables
Gaia   907a8656e49599d334203c22dcbdb3f98c1d7864
Gecko c22969eec61d
BuildID 20131009055629
Version 27.0a1


Notes:
Repro frequency: (100 percent, 10/10)
See attached: screenshots
Link to Test Case: https://moztrap.mozilla.org/manage/case/6073/
Other notes: Tested on numerous videos – A-Rod crushes eephus pitch (1 minute 26 seconds, Yankees-Red Sox brawl 2013 (2 minutes 17 seconds), How to Download Mozilla Firefox (4 minutes 37 seconds)

Comment 1

[Montoya Clemmons (MClemmons)]

Attachment: Screenshot of Youtube issue

Comment 2

[Jason Smith [:jsmith]]

Does this reproduce on 1.1?

Comment 3

[Jason Smith [:jsmith]]

(In reply to Jason Smith [:jsmith] from comment #2)
> Does this reproduce on 1.1?

Actually meant to ask - does this reproduce on 1.2?

Comment 4

[Lucas Aliaga]

This does not repro on Buri Aurora 1.2. The video aligns correctly and, after tapping the video in fullscreen mode, the user is given the option to return the video to regular in-browser mode. The video then renders correctly.

Environment:
Gaia   672c47bf94b69a329e0aacb9228a6aa16ade6226
SourceStamp e5edb484e2a9
BuildID 20131009004001
Version 26.0a2

Comment 5

[Montoya Clemmons (MClemmons)]

On today's 10/10/2013 Master/M-c build, actual results reported in Comment 0 did not reproduce - video properly aligned and user provided option to exit full screen mode.

Performance issues demonstrated as alignment was not smooth after user selected full screen mode, but not at rate as described in Comment 0

Environmental Variables
Gaia   959ac2f692d85072ffdc3d16a041b5bf4735ae59
Gecko aa986b6ce882
BuildID 20131010040202
Version 27.0a1

Comment 6

[Jason Smith [:jsmith]]

Sounds like this is a works for me then.

Comment 8

[Jason Smith [:jsmith]]

Reopening per the dupe.

Can we retest this to confirm what branches this reproduces on? I'm confused on the results this bug shows vs. the dupe.

Comment 9

[Sarah Parsons]

This issue only reproduces on the 1.3 branch. 

Most currently the Buri Master Build ID: 20131015040202

Gaia   17e871ae1f82699793e3cd28acda805ba724a8b6
SourceStamp febfe3c7732b
BuildID 20131015040202
Version 27.0a1


This issue does not reproduce on 1.2 or 1.1

Comment 10

[Jason Smith [:jsmith]]

Ben thinks this is a gfx regression.

Comment 11

[Milan Sreckovic [:milan] (needinfo for best results)]

We've had other YouTube related things - Sotaro, is this covered by some of the recent work?

Comment 12

[Sotaro Ikeda [:sotaro]]

The recent work is related to a/v sync. This bug is not covered by it.

Comment 14

[Milan Sreckovic [:milan] (needinfo for best results)]

Kats, thoughts?

Comment 15

[Kartikaya Gupta (email:[e-mail])]

According to https://bugzilla.mozilla.org/show_bug.cgi?id=950488#c8 there are general issues with full-screen and fixed-position content. Marking bug 950993 as a dependency since supposedly that will help matters.

Comment 16

[Sarah Parsons]

This issue started to occur on the Buri 1.3 Build ID: 20131004040203

Gaia   1ac6749e36424124493a1b4c7534f298789bdffd
SourceStamp 8f08240128c8
BuildID 20131004040203
Version 27.0a1

Last working Buri 1.3 Build ID: 20131003040204

Gaia   dc68f530ca7d1b182deef0a3787cfdd8f0778612
SourceStamp 0e26e6f12ad9
BuildID 20131003040204
Version 27.0a1

Comment 17

[Milan Sreckovic [:milan] (needinfo for best results)]

Kats, do we need help from layout here?

Comment 18

[Kartikaya Gupta (email:[e-mail])]

My impression is that this is related to the other fixed-position bugs, and is more of a layers problem than layout. BenWa will be looking at that this week and hopefully Chris Lord can also look at it next week when he's back from PTO.

Comment 19

[Chris Lord [:cwiiis]]

So there are two distinct bugs I'm able to reproduce.

1- If you scroll down as far as possible, then fullscreen the video, the video controls are cut off at the bottom. This is bug 950488 and BenWa is working on it
2- If you scroll down so that the top bar is hidden, the controls are placed too far down and end up being cut off slightly (because they go off-screen). I'm not sure where this bug lies.

I don't see what's shown in the screenshot though, which I guess could possibly have been fixed by bug 950993.

I think we should wait for bug 950993 and bug 950488 to land, then re-test this. I'm not sure that the originally reported bug exists anymore.

Comment 20

[Jean Gong :jgong]

PM triage this bug and believes this is a blocker.

Comment 21

[Benoit Girard (:BenWa)]

I just tried with and without the patch from bug 950993. I can reproduce this bug and the patch from bug 950993 does indeed fix it. It fixes both 1) and 2) from Comment 19.

I'm officially calling this a duplicate of bug 950993.