Ticket #2164 (closed defect: fixed)

Opened 2 years ago

Last modified 22 months ago

Mistake in BUFakeNode.js, BUFakeNode.find_listener_()

Reported by: james@… Owned by: dylan
Priority: normal Milestone: 0.9
Component: TestFramework Version: 0.4.1
Severity: normal Keywords:
Cc:

Description

The BUFakeNode.find_listener_() method in trunk/testtools/JsFakeDom/BUFakeDom.js contains a mistake, which means it never finds a listener and always returns -1. This problem causes both BUFakeNode.addEventListener() and BUFakeNode.removeEventListener() to function improperly.

The fix is simple. Line 319

return burst.Alg.find(this.listeners_, function(o) {o[0] == eventType && o[1] === listener && o[2] == useCapture});

should be

return burst.Alg.find_if(this.listeners_, function(o) {
    return o[0] == eventType && o[1] === listener && o[2] == useCapture
});

Change History

Changed 22 months ago by dylan

  • owner changed from alex to dylan
  • status changed from new to assigned
  • milestone set to 0.9

Changed 22 months ago by dylan

  • status changed from assigned to closed
  • resolution set to fixed

(In [7151]) fixes #2164, Mistake in BUFakeNode.js, BUFakeNode.find_listener_(), thanks James

Note: See TracTickets for help on using tickets.