arni2033 Reporter
	Description • 9 years ago

STR:   (Win7, 42.0a1 (2015-08-09))
1.   Open any video. E.g. this one: https://www.youtube.com/watch?v=xjbi5WWt5ak
2.A) Click fullscreen button in video controls to enter DOM fullscreen mode
2.B) Press F11 to enter fullscreen mode
3.   Press and release F10 (or Alt), then "Down" button
4.   Press F11 several times

Result:
No visible changes in steps 3 and 4.

Expectations:
Browser should switch to normal mode after the first time I press F11 in Step 4

Note:
If in step 4 I press Esc before pressing F11, then browser switches to normal mode eventually.
That happens because F10 (as well as Alt) opens Menu bar "in the background", and "Down" button opens "File" menu
So Currently user is able to do some actions without even knowing about it.

Also, part of this refers to Bug 1121280 - Fix full-screen UX issues for Flash and HTML5 video

	Hossain Al Ikram [:ikram] (QA Contact)
	Comment 1 • 9 years ago

I could Replicate this issue on Latest Nightly and Latest Release Version!

After going to watch any video in Yotube, If i press F10(Which happens to open or Hide Menu), then the F11 Key Doesn't work, unless I press the F10 key again or click on some buttons like play or pause.

Thanks Arni for finding this issue! I would like to wait for sometime and get some more feedback on this :)

	Xidorn Quan [:xidorn] UTC+11 Assignee
	Comment 2 • 9 years ago

If we destroy them, they won't be focused anymore. This patch should completely eliminate focusing invisible target in fullscreen.

	Justin Dolske [:Dolske]
	Comment 3 • 9 years ago

Comment on attachment 8650807 [details] [diff] [review]
patch

Makes sense to me, but I'd rather have Dao consider if there are any other surprising implications.

	Dão Gottwald [:dao]
	Comment 4 • 9 years ago

Comment on attachment 8650807 [details] [diff] [review]
patch

This will probably have side effects for code trying to interact with the hidden nodes. E.g. when I set #navigator-toolbox to display:none, trying to open and close tabs seems to break things. Not sure if this is because of missing XBL bindings or missing layout stuff.

We do exit DOM FS when opening or closing tabs, and maybe we do this timely enough that the above case isn't a problem, but it was just an example anyway.

By the way, this patch wouldn't fix it for (non-DOM) FS.

	Xidorn Quan [:xidorn] UTC+11 Assignee
	Comment 5 • 9 years ago

(In reply to Dão Gottwald [:dao] from comment #4)
> By the way, this patch wouldn't fix it for (non-DOM) FS.

It fixes it for fullscreen mode for me, at least on Windows.

	Dão Gottwald [:dao]
	Comment 6 • 9 years ago

(In reply to Xidorn Quan [:xidorn] (UTC+12) from comment #5)
> (In reply to Dão Gottwald [:dao] from comment #4)
> > By the way, this patch wouldn't fix it for (non-DOM) FS.
> 
> It fixes it for fullscreen mode for me, at least on Windows.

Oh right, I overlooked this line:

 #main-window[inFullscreen]:not([OSXLionFullscreen]) toolbar:not([fullscreentoolbar=true]),

	Xidorn Quan [:xidorn] UTC+11 Assignee
	Comment 7 • 9 years ago

Hmmm, so the listener of the keys are set in nsMenuBarFrame, and performed in nsMenuBarListener.

	Pulsebot
	Comment 9 • 9 years ago

https://hg.mozilla.org/integration/mozilla-inbound/rev/312566100da6

	Ryan VanderMeulen [:RyanVM]
	Comment 10 • 9 years ago

https://hg.mozilla.org/mozilla-central/rev/312566100da6