[w3c/manifest] Scope should remove the query and fragment at parse time (#793)

Since our [within scope](https://www.w3.org/TR/appmanifest/#dfn-within-scope) algorithm ignores the query and fragment part of the scope, I feel like we should strip those out in the [steps for processing the scope](https://www.w3.org/TR/appmanifest/#dfn-processing-the-scope-member).

Thus if your `scope` is `"/foo/?bar"`, we would canonically say that the scope is `"/foo/"`, reflecting the fact that the query string is irrelevant.

This change would not actually have any direct (specified) user-visible changes, since the only place in the manifest spec that scope is used is the "within scope" algorithm (which ignores query and fragment). However, it might be exposed to the user, for example, through developer tools that displays the canonical parse of the manifest, in which case it would be best to strip the query and fragment for clarity.

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/w3c/manifest/issues/793