Closed
Bug 704709
Opened 13 years ago
Closed 13 years ago
Hardware volume buttons in full screen mode triggers full screen warning message
Categories
(Core :: DOM: Core & HTML, defect)
Tracking
()
VERIFIED
FIXED
mozilla11
People
(Reporter: asa, Assigned: cpearce)
References
Details
(Whiteboard: [good first bug][mentor=jwein][lang=c++][qa!])
Attachments
(1 file)
(deleted),
patch
|
smaug
:
review+
jimm
:
review+
|
Details | Diff | Splinter Review |
At YouTube's HTML5 video player, http://www.youtube.com/watch?v=JCUdy1nUqrg for example, if you go into fullscreen mode and then hit the hardware volume buttons on your laptop, the "Press ESC to leave Full-screen mode" UI returns. That UI should not return after initially disappearing.
Tested on latest nightly build on a Thinkpad X1 laptop running Win 7 at this URL http://www.youtube.com/watch?v=JCUdy1nUqrg
Assignee | ||
Updated•13 years ago
|
Guess we need to add those keys to the whitelist...
Comment 2•13 years ago
|
||
The changes to the whitelist should be made in layout/base/nsPresShell.cpp within the |IsFullScreenAndRestrictedKeyEvent| function.
We'll need to find out what the keyCode is for the hardware volume buttons.
Whiteboard: [good first bug][mentor=jwein][lang=cpp]
Assignee | ||
Comment 3•13 years ago
|
||
http://pearce.org.nz/full-screen/ displays keycode of pressed keys. On my Win7 Lenovo T510 mute is 173, volume-down is 174, volume up is 175. I'm not sure if the same key codes are used on other platforms or even other hardware vendors on windows.
Comment 4•13 years ago
|
||
I have the same key codes (although my laptop is very similar: Lenovo W520 Win7)
Updated•13 years ago
|
Whiteboard: [good first bug][mentor=jwein][lang=cpp] → [good first bug][mentor=jwein][lang=c++]
Comment 5•13 years ago
|
||
(In reply to Chris Pearce (:cpearce) (Mozilla Corporation) from comment #3)
> http://pearce.org.nz/full-screen/ displays keycode of pressed keys. On my
> Win7 Lenovo T510 mute is 173, volume-down is 174, volume up is 175. I'm not
> sure if the same key codes are used on other platforms or even other
> hardware vendors on windows.
Some on my old Samsung NC10 netbook on Windows XP.
Comment 6•13 years ago
|
||
(In reply to Dão Gottwald [:dao] from comment #5)
> (In reply to Chris Pearce (:cpearce) (Mozilla Corporation) from comment #3)
> > http://pearce.org.nz/full-screen/ displays keycode of pressed keys. On my
> > Win7 Lenovo T510 mute is 173, volume-down is 174, volume up is 175. I'm not
> > sure if the same key codes are used on other platforms or even other
> > hardware vendors on windows.
>
> Some on my old Samsung NC10 netbook on Windows XP.
s/Some/Same/
Assignee | ||
Comment 8•13 years ago
|
||
The hardware volume control keys don't make it through to content on MacOSX and on Fedora Core 15 Linux. I'm not sure about other Linux distros/window managers, so I'll assume this is windows-only for now.
Assignee: nobody → chris
Status: NEW → ASSIGNED
Summary: using hardware volume buttons in full screen mode on YouTube HTML5 video triggers the UI for escaping full screen → Hardware volume buttons in full screen mode triggers full screen warning message
Assignee | ||
Comment 9•13 years ago
|
||
Don't show the full-screen warning when pressing media volume keys on Windows.
Attachment #577814 -
Flags: review?(bugs)
Comment 10•13 years ago
|
||
(In reply to Chris Pearce (:cpearce) (Mozilla Corporation) from comment #8)
> The hardware volume control keys don't make it through to content on MacOSX
> and on Fedora Core 15 Linux.
Yeah, I haven't managed to get events from volume control keys on Fedora.
Updated•13 years ago
|
Attachment #577814 -
Flags: review?(jmathies)
Attachment #577814 -
Flags: review?(bugs)
Attachment #577814 -
Flags: review+
Updated•13 years ago
|
Attachment #577814 -
Flags: review?(jmathies) → review+
Assignee | ||
Comment 11•13 years ago
|
||
Comment 12•13 years ago
|
||
Status: ASSIGNED → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
Whiteboard: [good first bug][mentor=jwein][lang=c++] → [good first bug][mentor=jwein][lang=c++][qa+]
Comment 13•13 years ago
|
||
Verified as fixed using a Windows 7 64-bit MacBook Air with:
Mozilla/5.0 (Windows NT 6.1; WOW64; rv:11.0) Gecko/20100101 Firefox/11.0
The "Press ESC to leave Full-screen mode" message is not displayed anymore when changing the volume from the keyboard.
Status: RESOLVED → VERIFIED
Whiteboard: [good first bug][mentor=jwein][lang=c++][qa+] → [good first bug][mentor=jwein][lang=c++][qa!]
Comment 14•13 years ago
|
||
Also verified as fixed using a Windows 7 64-bit HP Compaq 6715b with:
Mozilla/5.0 (Windows NT 6.1; WOW64; rv:11.0) Gecko/20100101 Firefox/11.0 beta 3.
I changed volume by using the specific volume keys from this laptop but the issue didn't reproduce anymore.
You need to log in
before you can comment on or make changes to this bug.
Description
•