From 5bb79ea0a79322d944368825ec617ccfb8912b81 Mon Sep 17 00:00:00 2001
From: Gunnar Beutner <gbeutner@serenityos.org>
Date: Mon, 3 May 2021 13:33:59 +0200
Subject: Userland: Update IPC calls to use proxies

This updates all existing code to use the auto-generated client
methods instead of post_message/send_sync.
---
 Userland/Libraries/LibProtocol/RequestClient.cpp   | 17 +++++++++--------
 Userland/Libraries/LibProtocol/WebSocketClient.cpp | 14 +++++++-------
 2 files changed, 16 insertions(+), 15 deletions(-)

(limited to 'Userland/Libraries/LibProtocol')

diff --git a/Userland/Libraries/LibProtocol/RequestClient.cpp b/Userland/Libraries/LibProtocol/RequestClient.cpp
index 0fb195c243..8c25c1ea27 100644
--- a/Userland/Libraries/LibProtocol/RequestClient.cpp
+++ b/Userland/Libraries/LibProtocol/RequestClient.cpp
@@ -18,12 +18,12 @@ RequestClient::RequestClient()
 
 void RequestClient::handshake()
 {
-    send_sync<Messages::RequestServer::Greet>();
+    greet();
 }
 
 bool RequestClient::is_supported_protocol(const String& protocol)
 {
-    return send_sync<Messages::RequestServer::IsSupportedProtocol>(protocol)->supported();
+    return IPCProxy::is_supported_protocol(protocol).supported();
 }
 
 template<typename RequestHashMapTraits>
@@ -33,29 +33,30 @@ RefPtr<Request> RequestClient::start_request(const String& method, const String&
     for (auto& it : request_headers)
         header_dictionary.add(it.key, it.value);
 
-    auto response = send_sync<Messages::RequestServer::StartRequest>(method, url, header_dictionary, ByteBuffer::copy(request_body));
-    auto request_id = response->request_id();
-    if (request_id < 0 || !response->response_fd().has_value())
+    auto response = IPCProxy::start_request(method, url, header_dictionary, ByteBuffer::copy(request_body));
+    auto request_id = response.request_id();
+    if (request_id < 0 || !response.response_fd().has_value())
         return nullptr;
-    auto response_fd = response->response_fd().value().take_fd();
+    auto response_fd = response.response_fd().value().take_fd();
     auto request = Request::create_from_id({}, *this, request_id);
     request->set_request_fd({}, response_fd);
     m_requests.set(request_id, request);
     return request;
+    return nullptr;
 }
 
 bool RequestClient::stop_request(Badge<Request>, Request& request)
 {
     if (!m_requests.contains(request.id()))
         return false;
-    return send_sync<Messages::RequestServer::StopRequest>(request.id())->success();
+    return IPCProxy::stop_request(request.id()).success();
 }
 
 bool RequestClient::set_certificate(Badge<Request>, Request& request, String certificate, String key)
 {
     if (!m_requests.contains(request.id()))
         return false;
-    return send_sync<Messages::RequestServer::SetCertificate>(request.id(), move(certificate), move(key))->success();
+    return IPCProxy::set_certificate(request.id(), move(certificate), move(key)).success();
 }
 
 void RequestClient::request_finished(i32 request_id, bool success, u32 total_size)
diff --git a/Userland/Libraries/LibProtocol/WebSocketClient.cpp b/Userland/Libraries/LibProtocol/WebSocketClient.cpp
index d322828d7f..e8607d7e2b 100644
--- a/Userland/Libraries/LibProtocol/WebSocketClient.cpp
+++ b/Userland/Libraries/LibProtocol/WebSocketClient.cpp
@@ -17,7 +17,7 @@ WebSocketClient::WebSocketClient()
 
 void WebSocketClient::handshake()
 {
-    send_sync<Messages::WebSocketServer::Greet>();
+    greet();
 }
 
 RefPtr<WebSocket> WebSocketClient::connect(const URL& url, const String& origin, const Vector<String>& protocols, const Vector<String>& extensions, const HashMap<String, String>& request_headers)
@@ -25,8 +25,8 @@ RefPtr<WebSocket> WebSocketClient::connect(const URL& url, const String& origin,
     IPC::Dictionary header_dictionary;
     for (auto& it : request_headers)
         header_dictionary.add(it.key, it.value);
-    auto response = send_sync<Messages::WebSocketServer::Connect>(url, origin, protocols, extensions, header_dictionary);
-    auto connection_id = response->connection_id();
+    auto response = IPCProxy::connect(url, origin, protocols, extensions, header_dictionary);
+    auto connection_id = response.connection_id();
     if (connection_id < 0)
         return nullptr;
     auto connection = WebSocket::create_from_id({}, *this, connection_id);
@@ -38,28 +38,28 @@ u32 WebSocketClient::ready_state(Badge<WebSocket>, WebSocket& connection)
 {
     if (!m_connections.contains(connection.id()))
         return (u32)WebSocket::ReadyState::Closed;
-    return send_sync<Messages::WebSocketServer::ReadyState>(connection.id())->ready_state();
+    return IPCProxy::ready_state(connection.id()).ready_state();
 }
 
 void WebSocketClient::send(Badge<WebSocket>, WebSocket& connection, ByteBuffer data, bool is_text)
 {
     if (!m_connections.contains(connection.id()))
         return;
-    post_message(Messages::WebSocketServer::Send(connection.id(), is_text, move(data)));
+    async_send(connection.id(), is_text, move(data));
 }
 
 void WebSocketClient::close(Badge<WebSocket>, WebSocket& connection, u16 code, String message)
 {
     if (!m_connections.contains(connection.id()))
         return;
-    post_message(Messages::WebSocketServer::Close(connection.id(), code, move(message)));
+    async_close(connection.id(), code, move(message));
 }
 
 bool WebSocketClient::set_certificate(Badge<WebSocket>, WebSocket& connection, String certificate, String key)
 {
     if (!m_connections.contains(connection.id()))
         return false;
-    return send_sync<Messages::WebSocketServer::SetCertificate>(connection.id(), move(certificate), move(key))->success();
+    return IPCProxy::set_certificate(connection.id(), move(certificate), move(key)).success();
 }
 
 void WebSocketClient::connected(i32 connection_id)
-- 
cgit v1.2.3