Skip to content

Adding an event handler in an event handler can lead to infinite looping #511

Description

@jcmoore

Example here:

https://gist.github.com/jcmoore/38135abefad78dc41e79#file-examlple-html-L14-L30

Not sure what the implications would be, but a solution might be as simple as using the "initial" length of the listeners array in the loop condition (instead of using the "current" length) for ShadowDOM/src/wrappers/events.js' invoke() method:

https://gist.github.com/jcmoore/38135abefad78dc41e79#file-shadowdom-src-wrappers-events-js-L385-L387

Activity

  1. arv commented on May 21, 2014

    @arv
    Contributor

    Yeah. Native does not add the functions to the current listeners. I'll update.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions