Bug 33861 - Document is not notified about event listener changes
: Document is not notified about event listener changes
Status: NEW
: WebKit
XML DOM
: 528+ (Nightly build)
: All All
: P2 Major
Assigned To:
:
:
:
:
  Show dependency treegraph
 
Reported: 2010-01-19 13:19 PST by
Modified: 2012-08-28 09:45 PST (History)


Attachments


Note

You need to log in before you can comment on or make changes to this bug.


Description From 2010-01-19 13:19:51 PST
Document keeps track wheter a certain event listener is available in the DOM, we only dispatch certain events if document()->hasListenerType(..) returns true.
In Node::addEventListener, we're adding the listener type, but in Node::removeEventListener we're not notifying the Document that something changed.

We need to switch to a counting based tracking scheme, counting how many event listeners are registered for each event type.
This bug can introduce performance problems, when adding a mutation event listener once for ie. <body> and then removing it. We will never stop dispatching mutation events.
------- Comment #1 From 2012-08-28 09:45:32 PST -------
Does this happen on real web sites? Counting listeners would add a toll on every web site, while benefits seem unlikely to materialize.