884754 - Promise.fulfill()

Status: RESOLVED INVALID

(Core :: DOM: Core & HTML, defect)

Comment 1

[Andrea Marchesini [:baku]]

Attachment: patch

Comment 2

[Mounir Lamouri (:mounir)]

Do we need that for the moment? Can't we simply keep our implementation as is and wait for a consensus to be formed?

Comment 3

[Mounir Lamouri (:mounir)]

Comment on attachment 764682 [details] [diff] [review]
patch

Clearing review based on comment 2.

Comment 4

[Domenic Denicola]

Unfortunately according to reports from the latest TC39 meeting `fulfill` is necessary for consensus, i.e. promises will not be acceptable to TC39 without it. In the same spirit as my comments on 879245, it seems important for Firefox not to fork the existing efforts and specs, even if this time it would be in a direction I support.

Comment 5

[Andrea Marchesini [:baku]]

Comment on attachment 764682 [details] [diff] [review]
patch

Asking for a review based on comment 4.

Comment 6

[Mounir Lamouri (:mounir)]

(In reply to Domenic Denicola from comment #4)
> Unfortunately according to reports from the latest TC39 meeting `fulfill` is
> necessary for consensus, i.e. promises will not be acceptable to TC39
> without it. In the same spirit as my comments on 879245, it seems important
> for Firefox not to fork the existing efforts and specs, even if this time it
> would be in a direction I support.

As said on bug 879245, we are not going to fork anything, our implementation is simply going to be an experiment behind a flag as long as people are fighting in three different mailing list and one GH repository to know what should be standardized...

Do you have a link to discussions regarding fulfill in TC39? (Other places clearly didn't reach a consensus on that topic but I'm not reading es-discuss.)

Comment 7

[Mounir Lamouri (:mounir)]

Comment on attachment 764682 [details] [diff] [review]
patch

Given that we only need this if we have thenable support that Jonas and I believe is not needed for the moment, I will cancel the review again.

Comment 8

[Andrea Marchesini [:baku]]

Attachment: patch

rebased

Comment 9

[Anne (:annevk)]

This is not coming back as such.