Re: [w3c/clipboard-apis] Defer OS clipboard read from clipboard.read() to ClipboardItem.getType() (PR #248)

@dandclark commented on this pull request.



> @@ -918,6 +934,60 @@ url: https://storage.spec.whatwg.org/#obtain-a-storage-key-for-non-storage-purpo
 
     1. If |representation|'s [=representation/MIME type=] is |mimeType| and |representation|'s [=representation/isCustom=] is |isCustom|, then:
 
+     1. If [=this=]'s [=ClipboardItem/clipboard change count at read=] is not null, and the current [=clipboard change count=] is not equal to [=this=]'s [=ClipboardItem/clipboard change count at read=], then [=reject=] |p| with a {{"DataError"}} {{DOMException}} in |realm|, and return |p|.

I'm not totally sold on `DataError` being the right choice to reject. Does this come form WebKit's current behavior?
Worth considering some others that might be a better fit from https://webidl.spec.whatwg.org/#idl-DOMException-error-names:
- `InvalidStateError`
- `NotAllowedError`
- `SecurityError`

A comparison of the other things that can throw these errors and `DataError` might help grant some confidence of which might be the best fit.

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

Message ID: <w3c/clipboard-apis/pull/248/review/4423276147@github.com>