Status: New
Owner: ----
Labels: Type-Defect Priority-Medium jQuery

New issue 1792 by [email protected]: TameWindow does not support full suite of event names
http://code.google.com/p/google-caja/issues/detail?id=1792

Calling addEventListener on a TameWindow with an event name of "unload" does the following:

* Enters TameWindow.addEventListener(). This only supports "load" and "DOMContentLoaded" so it delegates to tameDocument.addEventListener().

* Implemented by TameHTMLDocument.addEventListener(). This only supports "DOMContentLoaded" so it delegates to privates.tameContainerNode.addEventListener().

* Implemented by generic TameBackedNode which has no addEventListener() and is basically a hollow stub object.

We should implement "unload" and other event names. If we cannot do the correct behavior, we should throw a non-incidental error message (to the effect that the tame window does not support event E) rather than allow the exception to bubble up from the underlying implementation.

I would suspect that this might fit into the work being done to support embedded iframes, as per Issue 1199.

--
You received this message because this project is configured to send all issue notifications to this address.
You may adjust your notification preferences at:
https://code.google.com/hosting/settings

--

--- You received this message because you are subscribed to the Google Groups "Google Caja Discuss" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to [email protected].
For more options, visit https://groups.google.com/groups/opt_out.


Reply via email to