Skip to content
This repository has been archived by the owner on Jun 7, 2018. It is now read-only.

Commit

Permalink
Drop issues about the possibility of cancelable promises
Browse files Browse the repository at this point in the history
According to #141 these are not going to eventuate after all.

This closes #141.
  • Loading branch information
birtles committed Aug 23, 2017
1 parent cd6775e commit a483117
Showing 1 changed file with 0 additions and 14 deletions.
14 changes: 0 additions & 14 deletions Overview.bs
Expand Up @@ -790,10 +790,6 @@ The procedure to <dfn>reset an animation's pending tasks</dfn> for
1. If <var>animation</var> has a <a>pending pause task</a>, cancel that task.
1. <a lt="reject a Promise">Reject</a> <var>animation</var>'s <a>current ready
promise</a> with a DOMException named "AbortError".

Issue: If cancelable promises materialize, we should probably cancel here
instead of rejecting.

1. Let <var>animation</var>'s <a>current ready promise</a> be the result of
<a lt="create a new resolved Promise">creating a new resolved Promise
object</a>.
Expand Down Expand Up @@ -1588,10 +1584,6 @@ as follows:
<var>animation</var>.
1. <a lt="reject a Promise">Reject</a> the <a>current finished promise</a>
with a DOMException named "AbortError".

Issue: If cancelable promises materialize, we should cancel here instead
of rejecting.

1. Let <a>current finished promise</a> be a new (pending) <a>Promise</a>
object.
1. If <var>animation</var> is <em>not</em>
Expand Down Expand Up @@ -4874,14 +4866,8 @@ interface Animation : EventTarget {
:: The <a>play state</a> of this animation.
: <dfn attribute for=Animation>ready</dfn>
:: Returns the <a>current ready promise</a> for this object.

Issue: This should become a cancelable promise when (if) such things exist.

: <dfn attribute for=Animation>finished</dfn>
:: Returns the <a>current finished promise</a> for this object.

Issue: This should become a cancelable promise when (if) such things exist.

: <dfn attribute for=Animation>onfinish</dfn>
:: The event handler for the <a>finish event</a>.
: <dfn attribute for=Animation>oncancel</dfn>
Expand Down

0 comments on commit a483117

Please sign in to comment.