Closed Bug 730601 Opened 8 years ago Closed 8 years ago
Don't use Get
Listener Manager(false) to check existence of ELM, but Has Listener Manager
No description provided.
Attachment #600698 - Flags: review?(jst)
We should consider adding a |bool HasListenerManager() const| on nodes.
Ooops, I realized why I had used GetListenerManager(). it works with document nodes.
Can we set the bit for documents as neeeded?
Actually the previous patch would have worked just fine, since that code was dealing with nsIContents. But since I started to look at this ... https://tbpl.mozilla.org/?tree=Try&rev=5b2add087030
Attachment #600698 - Attachment is obsolete: true
Summary: Don't use GetListenerManager(false) to check existence of ELM, but HasFlag(NODE_HAS_LISTENERMANAGER) → Don't use GetListenerManager(false) to check existence of ELM, but HasListenerManager
Comment on attachment 600782 [details] [diff] [review] v2 Need to have separate handling for non-document objects and document objects, but HasListenerManager() itself is better API.
Attachment #600782 - Flags: review?(jst)
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.