Re: [battery] Assess compatibility risk of using [SecureContext] (#15)

Oh, it looks like [Blink doesn't implement the spec's secure context restriction at all](;l=29-30;drc=6531b4f2fda5515c3315e50d1cfb709e607d53e6), wow. So there are three possible behaviors:

1. The current implementation: works fine in a non-secure context
2. The current spec: returns a rejected promise in a non-secure context
3. The ideal spec: throws an exception (because the method doesn't exist at all because it's guarded with `[SecureContext]`) in a non-secure context

That's... tricky.

I guess my preference would be to remove the secure context restriction from the spec entirely, add a warning box pointing to this issue, and then as a separate effort, try to move the spec and implementation toward (3), if we think it's worth the engineering effort?

GitHub Notification of comment by domenic
Please view or discuss this issue at using your GitHub account

Sent via github-notify-ml as configured in

Received on Thursday, 25 February 2021 21:02:39 UTC