Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Make "inner invoke" use "remove an event listener" #758

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
10 changes: 6 additions & 4 deletions dom.bs
Original file line number Diff line number Diff line change
Expand Up @@ -1568,10 +1568,11 @@ run these steps:
<a for="event listener">capture</a> is true, then <a for=iteration>continue</a>.

<li><p>If <var>listener</var>'s <a for="event listener">once</a> is true, then
<a for=list>remove</a> <var>listener</var> from <var>event</var>'s {{Event/currentTarget}}
attribute value's <a for=EventTarget>event listener list</a>.
<!-- Do this before invocation to avoid reentrancy issues. No need to set removed to true since
each listener in listeners is run once anyway. -->
<a>remove an event listener</a> with the <var>event</var>'s {{Event/currentTarget}}
and <var>listener</var></a>.
<!-- Do this before invocation to avoid reentrancy issues. The listener is only run once in this
this loop, so setting removed to true is unnecessary. Regardless, the remove an event
listener function is used to share an abstraction. -->

<li><p>Let <var>global</var> be <var>listener</var> <a for="event listener">callback</a>'s
<a>associated Realm</a>'s <a for=Realm>global object</a>.
Expand Down Expand Up @@ -10056,6 +10057,7 @@ Mike Champion,
Mike Taylor,
Mike West,
Ojan Vafai,
Oliver Dunk,
Oliver Nightingale,
Olli Pettay,
Ondřej Žára,
Expand Down