Closed Bug 628585 Opened 11 years ago Closed 11 years ago
Pressing ctrl key in Flash doesn't work as expected
I just tried playing Wolfenstein 1-D at the above URL, but when i tried to shoot by pressing the ctrl key, my character instead moved to the left. :mmm and :dolske also experienced this. We all were using the a recent Minefield build on OS X. I tested the game in Google Chrome on OS X, and the shoot command worked correctly as expected upon pressing the ctrl key. :mmm and i also tested this in Minefield on Windows, and it worked correctly.
This is a regression from 3.6. (I just played Wolfenstein 1-D in Firefox 3.6 with the same Flash plugin, and it worked.) Not sure if this is a regression from plugin focus fixes for OS X.
blocking2.0: --- → ?
Cocoa event handling? Not a blocker.
blocking2.0: ? → .x
dupe of bug 616177 ?
> dupe of bug 616177 ? Possibly. I'll need to dig further into both bugs.
Instead of setting keyCode if the event is either NSKeyDown or NSKeyUp, this patch also sets the keyCode if the event is a NSFlagsChanged. Tested this in a build and it seemed to work, the Mac documentation states that this "Raises an NSInternalInconsistencyException if sent to a non-key event." so this seems safe.
Assignee: nobody → mars.martian+bugmail
Status: NEW → ASSIGNED
Attachment #522444 - Flags: review?
Attachment #522444 - Flags: review? → review?(joshmoz)
Comment on attachment 522444 [details] [diff] [review] Always set keyCode. Looks good, thanks!
Attachment #522444 - Flags: review?(joshmoz) → review+
Status: ASSIGNED → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
mehdi++! Would we want to push this to 4.x too?
Yes, we want this for Firefox 4 as soon as possible.
Comment on attachment 522444 [details] [diff] [review] Always set keyCode. This is pretty low risk, only affects Mac users and slightly changes how key events are handled with plugins.
Attachment #522444 - Flags: approval2.0?
Comment on attachment 522444 [details] [diff] [review] Always set keyCode. Approved for mozilla2.0 repository, a=dveditz for release-drivers
Attachment #522444 - Flags: approval2.0? → approval2.0+
WFM on Mozilla/5.0 (Macintosh; Intel Mac OS X 10.6; rv:2.2a1pre) Gecko/20110404 Firefox/4.2a1pre
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.