Skip to content
This repository was archived by the owner on Jul 3, 2019. It is now read-only.

Fix check for existing listener - #2453

Open
ivanpopelyshev wants to merge 1 commit into
mozilla:masterfrom
ivanpopelyshev:fix-add-event-listener
Open

Fix check for existing listener#2453
ivanpopelyshev wants to merge 1 commit into
mozilla:masterfrom
ivanpopelyshev:fix-add-event-listener

Conversation

@ivanpopelyshev

@ivanpopelyshev ivanpopelyshev commented Apr 19, 2019

Copy link
Copy Markdown

The following code adds handler two times:

d.addEventListener(Event.COMPLETE, handler);
d.addEventListener(Event.COMPLETE, handler2);
d.addEventListener(Event.COMPLETE, handler);

@danimesq danimesq mentioned this pull request Mar 13, 2022
Merged
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@ivanpopelyshev