mac branch build 0710 this seems to have regressed due to last night's checkin... Keyboard events are not getting registered... try faceball under games|sports... blocks all games that involve keyboard interaction..
I think this is a regression, I'll try to get a patch made up right away.
Status: NEW → ASSIGNED
Priority: -- → P1
Target Milestone: --- → mozilla0.9.3
Mark PDT+ to get on radar. This seems like a stop ship for plugins.
Hardware: PC → Macintosh
Rephrasing summary. Without focus events being sent to Shockwave, key events won't work. Shrirang, just to be clear on the old summary, keyboard do work on other plugins on Mac like Flash, right? For example, at http://www.emailtrack.com
Summary: keyboard events are not working on mac → Focus events not being sent to Shockwave
Whiteboard: [PDT+] → [PDT+][ETA Wednesday]
Andrei, can you review this patch? It seems to fix it for the Mac. For some reason, a focus event isn't a nsGUIEvent. This was making OS X crash. It doesn't matter because all we care about is that it's a focus event and then we do something special with that in ProccessEvent(). The memset() ensures we don't crash doing any casts later as we have checks for null.
Whiteboard: [PDT+][ETA Wednesday] → [PDT+][ETA Wednesday][seeking reviews]
OK, as long as you are sure that memset on the structure is sufficient to initialize it to a safe state, sr=attinasi
Patch is in the branch. I'll guess I'll get the trunk tomorrow when it opens. /cvsroot/mozilla/layout/html/base/src/nsObjectFrame.cpp,v new revision: 18.104.22.168; previous revision: 22.214.171.124
Whiteboard: [PDT+][ETA Wednesday][seeking reviews] → [PDT+][vbranch][patch in branch]
Patch in the trunk, marking FIXED.
Status: ASSIGNED → RESOLVED
Last Resolved: 17 years ago
Resolution: --- → FIXED
Whiteboard: [PDT+][vbranch][patch in branch] → [PDT+][vbranch]
Thanks, pls verify when you get a chance, shrir. Thanks!
Whiteboard: [PDT+][vbranch] → [PDT+]
Shrir will be back on Wednesday.
verified this one on mac branch 0718. things look good , focus events are working now. Added 'vtrunk' for trunk verification later. marking VERIF
Status: RESOLVED → VERIFIED
verif on mac trunk 0801, removing 'vtrunk'
You need to log in before you can comment on or make changes to this bug.