261720 2023-09-18 21:10:10 -0700 Give ability for IPC calls to work with NativePromise 2023-09-21 00:53:29 -0700 1 1 1 Unclassified WebKit WebKit Process Model WebKit Nightly Build Unspecified Unspecified RESOLVED FIXED https://bugs.webkit.org/show_bug.cgi?id=257934 https://bugs.webkit.org/show_bug.cgi?id=261864 InRadar P2 Normal --- 1 jean-yves.avenard jean-yves.avenard webkit-bug-importer oldest_to_newest 1978198 0 jean-yves.avenard 2023-09-18 21:10:10 -0700 Would give the ability to instead of doing: ``` m_connectionToWebProcess->connection().sendWithAsyncReply(Messages::MediaSourcePrivateRemote::WaitForTarget(target), WTFMove(completionHandler), m_identifier); ``` you could instead do: ``` m_connectionToWebProcess->connection().sendWithPromisedReply(Messages::MediaSourcePrivateRemote::WaitForTarget(target), m_identifier) ->whenSettled(runLoop, __func__, [] (MediaSourcePrivateRemote::WaitForTarget::Promise::Result&& result) { if (result) // do something with result.value() else // do something with result.error() }); ``` this would also gives the ability to propagate the IPC error, see bug 257934 It also allows to guarantee on which thread the completionHandler will run once the IPC operation has returned its value. 1978199 1 webkit-bug-importer 2023-09-18 21:12:48 -0700 <rdar://problem/115704438> 1978584 2 jean-yves.avenard 2023-09-20 00:45:36 -0700 Pull request: https://github.com/WebKit/WebKit/pull/17965 1978920 3 ews-feeder 2023-09-21 00:53:26 -0700 Committed 268240@main (f997a9edb80c): <https://commits.webkit.org/268240@main> Reviewed commits have been landed. Closing PR #17965 and removing active labels.