As a regression from the patch in bug 68756, lost focus isn't correctly being sent to plugins. Reason: Typo: We should use NS_BLUR_CONTENT instead of NS_LOSTFOCUS in the switch statement.

peter, is 78414 not the same problem ?

Part of it. Creating dependancy.

I'm attaching a patch which should do this correctly. It's in Mac only code as I believe on Windows and Linux focus is handled through child windows. Do you you think I should move the code up to where the DOM calls focus? I added a public weak ref to the PresContext in nsObjectFrame. I don't see any reason why it's any better to use a get/setter in this situation. In there another way I can get the presshell from either an nsIContent (nsHTMLObjectElement) or the frame (nsHTMLContainerFrame)? cc:ing some people for reviews

[s]r=attinasi - patch looks good, and I'm assuming you have tested that it works correctly.

Tested on Mac and Windows. Windows isn't much of an issue because this code path is Mac only. Don, can I get a code review?

Moving to m0.9.2

r=dcone

Patch checked in. Marking FIXED. /cvsroot/mozilla/layout/html/base/src/nsObjectFrame.cpp,v <-- nsObjectFrame.cpp new revision: 1.215; previous revision: 1.214 done Checking in nsObjectFrame.h; /cvsroot/mozilla/layout/html/base/src/nsObjectFrame.h,v <-- nsObjectFrame.h new revision: 1.5; previous revision: 1.4

Marking FIXED for real this time.

Verified on mac 20010517. Was able to use arrow key and command keys after I clicked outside the plugin area.

Please see bug 247140. I'm not sure if that problem (which has actually been around for ages) is a regression of this one - but I'm sure it's related.