search

w3c / push-api

Push API
https://w3c.github.io/push-api/
Other
144 stars 40 forks source link

Does PushManager subscribe() need transient activation check? #346

Open youennf opened 2 years ago

youennf commented 2 years ago

https://w3c.github.io/push-api/#dom-pushmanager-subscribe does not mention whether a https://html.spec.whatwg.org/multipage/interaction.html#transient-activation check should be done. AIUI, Chrome and Firefox enforce these checks and WebKit is about to add this check at https://bugs.webkit.org/attachment.cgi?id=455337&action=review. Would it make sense to update the spec accordingly?

marcoscaceres commented 2 years ago

Yeah, the spec predates the new gesture model. I'll update the spec.

marcoscaceres commented 2 years ago

@youennf, wrote:

Chrome and Firefox enforce these checks and WebKit is about to add this check at

I did some tests and checked the code of what Chrome and Firefox do... it seems that they don't consume the user activation.

Irrespective, I think it makes sense to consume the user active activation when calling subscribe().