Re: [w3c/manifest] Add `navigator.install()` (PR #1175)

@marcoscaceres commented on this pull request.



> +                  <li>[=Reject=] [=this=].{{Navigator/[[installPromise]]}} with
+                  an {{"DataError"}} {{DOMException}}.
+                  </li>
+                  <li>Set [=this=].{{Navigator/[[installPromise]]}} to `null`.
+                  </li>
+                  <li>Terminate this algorithm.
+                  </li>
+                </ol>
+              </li>
+              <li>Present an installation UI for the |document|'s
+              [=Document/processed manifest=].
+                <aside class="note">
+                  If a user agent only allows one instance of the application
+                  to be [=installed=], it may show a launch UI instead. In this
+                  case, the user agent should return the result of the launch
+                  prompt. For privacy reasons, this API should not reveal

It can't do this (reveal anything) anyway, so the should not doesn't make sense here. 

-- 
Reply to this email directly or view it on GitHub:
https://github.com/w3c/manifest/pull/1175#pullrequestreview-2948868807
You are receiving this because you are subscribed to this thread.

Message ID: <w3c/manifest/pull/1175/review/2948868807@github.com>