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

Check whether a promises based API is doable #12

Closed
frivoal opened this issue Sep 27, 2016 · 5 comments
Closed

Check whether a promises based API is doable #12

frivoal opened this issue Sep 27, 2016 · 5 comments
Assignees
Labels

Comments

@frivoal
Copy link
Member

frivoal commented Sep 27, 2016

No description provided.

@frivoal
Copy link
Member Author

frivoal commented Sep 27, 2016

@Rob--W
Copy link

Rob--W commented Sep 27, 2016

Two things:

  • In the minutes it is not clear whether you consider the possibility for promise-based APIs to be added to Chrome. Here is the relevant ticket where Chromium team members express a favorable attitude towards promises: https://crbug.com/328932.
  • Adding promises does not necessarily negate the need for the runtime.lastError attribute, because it is currently the only way to signal errors in things such as events (e.g. port.onDisconnect).

@snoack
Copy link

snoack commented Oct 5, 2016

For reference, there was a brief discussion on the mailing list. To sum it up, it seems very feasible to use promises instead of callback, even without breaking backwards compatibility, like Firefox is already doing.

@kmaglione
Copy link

Adding promises does not necessarily negate the need for the runtime.lastError attribute, because it is currently the only way to signal errors in things such as events (e.g. port.onDisconnect).

It's perfectly possible to signal errors to event listeners by passing them as arguments.

@mikepie1
Copy link
Contributor

mikepie1 commented Feb 2, 2017

Approved Promises per meeting http://browserext.github.io/minutes/2017-02-02.html
Spec updated in Pull Request 44 #44

@mikepie1 mikepie1 closed this as completed Feb 2, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

5 participants