Re: [w3c/manifest] Adding file_handlers and launch consumer (#1005)

@dmurph commented on this pull request.



> +    </section>
+    <section data-cite="file-system-access">
+      <h2>
+        <dfn>Execute a file handler launch</dfn>
+      </h2>
+      <p>
+        A {{Window/window}} has an associated {{LaunchConsumer}} <dfn>assigned
+        launch consumer</dfn>.
+      </p>
+      <p>
+        A {{Window/window}} has an associated [=list=] of {{LaunchParams}}
+        <dfn>unconsumed launch params</dfn>.
+      </p>
+      <p>
+        When a [=file type=] is registered with a web app and one or more of
+        these registered files are launched on the platform, run the following

I believe it should just be for redirects and not navigations? But I'm not sure. @evanstade probably has more info here.

> +            <li>|item|["name"] [=map/exists=] and is not a [=string=].
+            </li>
+            <li>|item|["accept"] doesn't [=map/exist=].
+            </li>
+            <li>|item|["accept"] is not a [=dictionary=].
+            </li>
+            <li>|item|["accept"] is [=list/empty].
+            </li>
+          </ul>
+        </li>
+        <li>Let |url:URL| be the result of [=URL Parser|parsing=]
+        |item|["action"] with |start_url URL| as the base URL.
+        </li>
+        <li>If |url| is failure, return failure.
+        </li>
+        <li>If |url| is not [=manifest/within scope=] of |scope|, return

It looks like it takes a URL, right? https://w3c.github.io/manifest/#dfn-within-scope


> +        <li>If [=assigned launch consumer=] is set, call [=assigned launch
+        consumer=] with |params| and return
+        </li>
+        <li>[=list/Append=] |params| to [=unconsumed launch params=].

I noted that they were on the {{Window/window}}, but if we decide to change where they live, I can update that again.

> +    <section data-cite="file-system-access">
+      <h2>
+        <dfn>Execute a file handler launch</dfn>
+      </h2>
+      <p>
+        A {{Window/window}} has an associated {{LaunchConsumer}} <dfn>assigned
+        launch consumer</dfn>.
+      </p>
+      <p>
+        A {{Window/window}} has an associated [=list=] of {{LaunchParams}}
+        <dfn>unconsumed launch params</dfn>.
+      </p>
+      <p>
+        When a [=file type=] is registered with a web app and one or more of
+        these registered files are launched on the platform, run the following
+        steps, given the launched {{FrozenArray}} of {{FileSystemHandle}}s

This would be from... the real of the document? I don't know what a JS realm is 😬 

> +        A {{Window/window}} has an associated [=list=] of {{LaunchParams}}
+        <dfn>unconsumed launch params</dfn>.

Good question. I think this is forward-looking for when these APIs can choose to not to create a new context per launch. I can change to a single one.

-- 
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/pull/1005#discussion_r723706544