544544 - chrome event listener gets NS_ERROR_XPC_BAD_OP_ON_WN_PROTO when checking event.isTrusted

Status: RESOLVED FIXED

(Core :: XPConnect, defect)

Description

[Justin Dolske [:Dolske]]

Bug 538910 adds a chrome event listener, which checks the event's .isTrusted property. But when it does this, an exception is thrown:

  uncaught exception: [Exception... "Illegal operation on WrappedNative
  prototype object"  nsresult: "0x8057000c (NS_ERROR_XPC_BAD_OP_ON_WN_PROTO)

I *think* this started happening after adding bug 541076 to my mq, maybe due to using nsIDOMDataContainerEvent or adding NS_EVENT_FLAG_ONLY_CHROME_DISPATCH? Could be coincidence, though.

mrbkap has done some initial debugging.

Comment 1

[Blake Kaplan (:mrbkap) (inactive)]

Attachment: Patch

I'm still confused as to how this works at all (how do we resolve isTrusted on the event?) but we need to make sure that DataContainerEvent advertises that it's nsIDOMEvent and nsIDOMNSEvent to XPConnect.

Comment 2

[Olli Pettay [:smaug][[e-mail]]]

Comment on attachment 425531 [details] [diff] [review]
Patch

So far datacontainerevent has been used only by C++ a11y code, I believe.
So no one noticed this :(

Comment 3

[Blake Kaplan (:mrbkap) (inactive)]

http://hg.mozilla.org/mozilla-central/rev/500068ba0693

Comment 4

[Benjamin Smedberg]

http://hg.mozilla.org/projects/firefox-lorentz/rev/79396e021cb2

Comment 5

[Mike Beltzner [:beltzner, not reading bugmail]]

Blanket approval for Lorentz merge to mozilla-1.9.2
a=beltzner for 1.9.2.4 - please make sure to mark status1.9.2:.4-fixed

Comment 6

[Benjamin Smedberg]

Merged into 1.9.2 at http://hg.mozilla.org/releases/mozilla-1.9.2/rev/84ba4d805430