760417 - When selecting "Pop out" for HTML5 videos, the window is opened in full screen mode and the "Full screen" button is still available on its right bottom side

Status: RESOLVED INVALID

(Core :: Audio/Video, defect)

Description

[Ioana (away)]

Attachment: Fullscreen HTML5 pop out

Mozilla/5.0 (X11; Linux i686; rv:13.0) Gecko/20100101 Firefox/13.0
Mozilla/5.0 (X11; Linux x86_64; rv:13.0) Gecko/20100101 Firefox/13.0

STR:
1. Launch Firefox.
2. Load http://www.youtube.com/html5 in the browser and join the HTML5 trial.
3. Load http://www.youtube.com/watch?v=kkGeOWYOFoA in the browser (or any other HTML5 youtube video).
4. Click on the "Full screen" button.
5. Right-click the video and select "Pop out".

The pop out window is opened directly in full screen mode but the "Full screen" button is still available on its right bottom side (issue visible in the attachment).

This issue reproduces the same way in the latest Aurora and Nightly builds (2012-05-31).

Comment 1

[u279076]

Can you please check if this is a regression in Firefox 13 or if it happens in previous builds?

Comment 2

[Ioana (away)]

Last good nightly: 2011-11-08
First bad nightly: 2011-11-09

Pushlog:
http://hg.mozilla.org/mozilla-central/pushloghtml?fromchange=81dedcc49ac0&tochange=6d42793c4807

Comment 3

[u279076]

Thanks Ioana, nothing stands out there as a bug which caused this, but someone from Engineering likely knows better than I.

Comment 4

[Ioana (away)]

This issue also reproduces on Firefox 10.0.5 ESR:
Mozilla/5.0 (X11; Linux x86_64; rv:10.0.5) Gecko/20100101 Firefox/10.0.5

Comment 5

[cajbir (:cajbir)]

In the regression window there's a merge from inbound to mozilla-central containing:

afca83588ca2	Chris Pearce — Bug 699885 part 3 - Update full-screen mochitests wait for focus. r=roc
de9859b56d6e	Chris Pearce — Bug 699885 part 2 - Exit DOM full-screen if we've changed tab or app by the time the request is granted. r=dao
bea7ecf9084e	Chris Pearce — Bug 699885 part 1 - Ensure we dispatch 'deactivate' event to chrome window when we lose focus while changing to full-screen mode. r=roc

These all involve full screen mode which seems related to the bug, or am I misreading the merge/regression window?

Comment 6

[cajbir (:cajbir)]

There's also:

f7355dc166cd	Chris Pearce — Bug 700151 - Enable DOM full-screen API by default in desktop Firefox. r=roc

Is it possible earlier builds don't see the bug due to the fact they didn't support the full screen API at all and some other code path was being taken?

Comment 7

[Chris Pearce [:cpearce (Not reading bugmail)]]

"Pop out" while in fullscreen seems to work as expected for me in
Mozilla/5.0 (Windows NT 6.1; WOW64; rv:16.0) Gecko/16.0 Firefox/16.0a1

Can you test in the latest nightly please?

Comment 8

[Ioana (away)]

Reproducible on:
Mozilla/5.0 (X11; Linux i686; rv:16.0) Gecko/16.0 Firefox/16.0a1 (20120611030526)

Chris, this bug didn't reproduce on older versions either on Windows. It only reproduces on Linux.

Comment 9

[Chris Peterson [:cpeterson]]

This bug is no longer relevant because YouTube's HTML5 player no longer has a "Pop out" content menu item.

Comment 10

[u279076]

(In reply to Chris Peterson [:cpeterson] from comment #9)
> This bug is no longer relevant because YouTube's HTML5 player no longer has
> a "Pop out" content menu item.

I think INVALID would be more appropriate in this case. WORKSFORME would suggest that the "Pop out" context menu still exists and is working without a fix on our end.