This is an archived snapshot of W3C's public bugzilla bug tracker, decommissioned in April 2019. Please see the home page for more details.

Bug 26457 - Replace instances of Promise<any> with Promise<void>
Summary: Replace instances of Promise<any> with Promise<void>
Status: RESOLVED FIXED
Alias: None
Product: HTML WG
Classification: Unclassified
Component: Encrypted Media Extensions (show other bugs)
Version: unspecified
Hardware: PC All
: P2 normal
Target Milestone: ---
Assignee: David Dorwin
QA Contact: HTML WG Bugzilla archive list
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2014-07-29 18:53 UTC by Boris Zbarsky
Modified: 2014-08-12 00:58 UTC (History)
3 users (show)

See Also:

Attachments

Description Boris Zbarsky 2014-07-29 18:53:02 UTC 
There's no reason for this to be a promise<any>, since it's only resolved with undefined.
Comment 1 Boris Zbarsky 2014-07-29 18:54:30 UTC 
Same thing for the promise returned from update().
Comment 2 Boris Zbarsky 2014-07-29 18:55:03 UTC 
And close() and remove().
Comment 3 Boris Zbarsky 2014-07-29 18:58:16 UTC 
And setServerCertificate.
Comment 4 Boris Zbarsky 2014-07-29 19:26:14 UTC 
And setMediaKeys.
Comment 5 David Dorwin 2014-07-30 18:25:10 UTC 
When these were added, void was not acceptable in the grammar. This has changed [1][2], so I will make the change.

[1] http://heycam.github.io/webidl/#prod-ReturnType
[2] http://heycam.github.io/webidl/#changes
Comment 6 David Dorwin 2014-08-11 23:10:36 UTC 
Changed in https://dvcs.w3.org/hg/html-media/rev/7b3dd5a1f571.
Comment 7 David Dorwin 2014-08-12 00:58:03 UTC 
https://dvcs.w3.org/hg/html-media/rev/673780061087 updates the algorithms to not "Resolve promise with undefined." "Resolve promise with void" doesn't sound right, so I just said "Resolve promise."