Make Navigator.canShare and Navigator.share methods optional

[rjimenezda]

Hi!

I've recently worked with navigator.share and navigator.canShare and have noticed that the types report them as non-optional.

I saw #837 where it's stated that the w3c says they are non-optional, but the truth is that most desktop browsers do not implement them and are only supported on secure contexts, so they can be undefined in certain cases.

I'm not sure if there's something else that should be changed, let me know.

[github-actions]

Thanks for the PR!

This section of the codebase is owned by @saschanaz - if they write a comment saying "LGTM" then it will be merged.

[chemitaxis]

Good job @rjimenezda LGTM!

[github-actions]

Sorry @chemitaxis, you don't have access to these files:

• /baselines/dom.generated.d.ts
• /inputfiles/overridingTypes.jsonc.

[rjimenezda]

@microsoft-github-policy-service agree company="Genially"

[saschanaz]

As stated in #1486 I'd rather not do this only for this specific API. We could make any not-supported-everywhere things as optional but such breaking change needs agreement from TS team. Discussion can go on in #1486.

[rjimenezda]

Fair enough, I'll close it then.