WICG / manifest-incubations

Before install prompt API for installing web applications
https://wicg.github.io/manifest-incubations/
Other
99 stars 29 forks source link

Protocol handlers should not normatively handle the protocol launch (HTML already does this) #64

Open mgiuca opened 1 year ago

mgiuca commented 1 year ago

There's a section here "Handling a protocol launch" which says:

When a protocol handler description _protocolhandler having manifest manifest is invoked, it goes through the same steps used to invoke a protocol handler defined in HTML, where the user agent SHOULD navigate to url and the appropriate browsing context is set to a new top level browsing context.

However, if I'm right, I believe this is redundant and doesn't need normative text at all.

In "Processing the protocol_handlers member", there is a step:

For each processedProtocolHandlers, the user agent SHOULD register a protocol handler.

This means that we are explicitly invoking the HTML registration system, which would set up the scheme and URL found in the manifest within the user agent handler database. Therefore, HTML will automatically invoke the protocol handler when a navigation is detected to that protocol. There is no need to have any normative text under "Handling a protocol launch".

That section could be replaced with a non-normative note, explaining that this is so.