fix: default share(data) to empty dictionary

[marcoscaceres]

fixes #109

[marcoscaceres]

We probably don't need the note anymore because now it's clear from the IDL that at least {} is always present. Now the algorithm always makes sense :)

[marcoscaceres]

I can’t recall if the IDL changes are automatically picked up on WPT. Also, any idea if whatwg/webidl#750 is implemented in Blink yet?... I’m guessing not, but we should probably track that.

[ewilligers]

any idea if whatwg/webidl#750 is implemented in Blink yet?

No.

boolean canShare(optional ShareData data = {});
Promise<void> share(optional ShareData data = {});

gives

navigator_share.idl(12) : error: Unexpected "{" after "=".
navigator_share.idl(15) : error: Unexpected "{" after "=".

[marcoscaceres]

WPT also hasn’t updated to the latest version of WebIDL2.js, which includes support for this. Should hopefully happen by end of the week.

Do we have a Chrome bug we can track? Should we file one?

[ewilligers]

Chrome bug for dictionaries with no required members:
https://bugs.chromium.org/p/chromium/issues/detail?id=948139