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

@shwetabin commented on this pull request.



> +
+      1. If |isCustom| is true, then:
+
+       1. Let |webCustomFormatMap| be the [=os specific custom map name=].
+
+       1. Read |webCustomFormatMap| from the [=system clipboard=].
+
+       1. Let |webCustomFormatMapString| be the JSON string deserialized from |webCustomFormatMap|.
+
+       1. Let |osFormatName| be the value in |webCustomFormatMapString| whose key matches |mimeType| [=serialize a MIME type|serialized=].
+
+       1. If |osFormatName| is not found, then [=queue a global task=] on the [=clipboard task source=], given |realm|'s [=realm/global object=], to [=reject=] |p| with a {{"NotFoundError"}} {{DOMException}} in |realm|, then abort these steps.
+
+      1. Else, let |osFormatName| be the result of running [=os specific well-known format=] given |mimeType|.
+
+      1. Let |rawData| be the result of reading bytes for |osFormatName| from the [=system clipboard=].

Thanks for the feedback. I simplified this to avoid introducing a separate helper algorithm.
I removed the standalone “read bytes” definition and inlined the system clipboard data fetch directly in getType, so the spec now does the lookup at the call site (get current system clipboard item, find representation by osFormatName, then use its data).

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

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