LibWeb: Revert proxy_data ipc parameters

Author: kunlinglio

Libraries/LibIPC/Decoder.cpp

@@ -13,6 +13,7 @@
 #include <LibCore/AnonymousBuffer.h>
 #include <LibIPC/Decoder.h>
 #include <LibIPC/File.h>
+#include <LibProxy/Proxy.h>
 #include <LibURL/Parser.h>
 #include <LibURL/URL.h>
 
@@ -174,6 +175,16 @@ ErrorOr<Core::AnonymousBuffer> decode(Decoder& decoder)
     return Core::AnonymousBuffer::create_from_anon_fd(anon_file.take_fd(), size);
 }
 
+template<>
+ErrorOr<Proxy::ProxyData> decode(Decoder& decoder)
+{
+    auto type = TRY(decoder.decode<Proxy::ProxyData::Type>());
+    auto host = TRY(decoder.decode<String>());
+    auto port = TRY(decoder.decode<u16>());
+
+    return Proxy::ProxyData { type, host, port };
+}
+
 template<>
 ErrorOr<URL::BlobURLEntry::Blob> decode<URL::BlobURLEntry::Blob>(Decoder& decoder)
 {

Libraries/LibIPC/Decoder.h

@@ -23,6 +23,7 @@
 #include <LibIPC/Concepts.h>
 #include <LibIPC/File.h>
 #include <LibIPC/Forward.h>
+#include <LibProxy/Proxy.h>
 #include <LibURL/Origin.h>
 #include <LibURL/URL.h>
 
@@ -139,7 +140,7 @@ template<>
 ErrorOr<Core::AnonymousBuffer> decode(Decoder&);
 
 template<>
-ErrorOr<Core::ProxyData> decode(Decoder&);
+ErrorOr<Proxy::ProxyData> decode(Decoder&);
 
 template<>
 ErrorOr<URL::BlobURLEntry::Blob> decode(Decoder&);

Libraries/LibIPC/Encoder.cpp

@@ -20,6 +20,7 @@
 #include <LibIPC/Attachment.h>
 #include <LibIPC/Encoder.h>
 #include <LibIPC/File.h>
+#include <LibProxy/Proxy.h>
 #include <LibURL/Origin.h>
 #include <LibURL/URL.h>
 
@@ -196,6 +197,15 @@ ErrorOr<void> encode(Encoder& encoder, Core::AnonymousBuffer const& buffer)
     return {};
 }
 
+template<>
+ErrorOr<void> encode(Encoder& encoder, Proxy::ProxyData const& proxy)
+{
+    TRY(encoder.encode(proxy.type));
+    TRY(encoder.encode(proxy.host));
+    TRY(encoder.encode(proxy.port));
+    return {};
+}
+
 template<>
 ErrorOr<void> encode(Encoder& encoder, URL::BlobURLEntry::Blob const& blob)
 {

Libraries/LibIPC/Encoder.h

@@ -18,6 +18,7 @@
 #include <LibIPC/File.h>
 #include <LibIPC/Forward.h>
 #include <LibIPC/Message.h>
+#include <LibProxy/Proxy.h>
 #include <LibURL/URL.h>
 
 namespace IPC {
@@ -143,7 +144,7 @@ template<>
 ErrorOr<void> encode(Encoder&, Core::AnonymousBuffer const&);
 
 template<>
-ErrorOr<void> encode(Encoder&, Core::ProxyData const&);
+ErrorOr<void> encode(Encoder&, Proxy::ProxyData const&);
 
 template<>
 ErrorOr<void> encode(Encoder&, URL::BlobURLEntry::Blob const&);

Libraries/LibProxy/Platform/ProxyEnv.cpp

@@ -7,10 +7,8 @@
 #include "../Proxy.h"
 #include "ProxyPlatform.h"
 #include <AK/ByteString.h>
-#include <CoreFoundation/CoreFoundation.h>
 #include <LibCore/Environment.h>
 #include <LibURL/Parser.h>
-#include <SystemConfiguration/SystemConfiguration.h>
 
 namespace Proxy {
 

Libraries/LibProxy/Platform/ProxyPlatform.cpp

@@ -25,6 +25,9 @@ void watch_system_proxy_changes(Function<void()> callback)
 {
 #if defined(AK_OS_MACOS)
     watch_system_proxy_changes_mac(move(callback));
+#else
+    // Avoid "unused parameter" error.
+    (void)callback;
 #endif
 }
 

Libraries/LibRequests/RequestClient.cpp

@@ -76,12 +76,12 @@ void RequestClient::die()
     }
 }
 
-RefPtr<Request> RequestClient::start_request(ByteString const& method, URL::URL const& url, Optional<HTTP::HeaderList const&> request_headers, ReadonlyBytes request_body, HTTP::CacheMode cache_mode, HTTP::Cookie::IncludeCredentials include_credentials)
+RefPtr<Request> RequestClient::start_request(ByteString const& method, URL::URL const& url, Optional<HTTP::HeaderList const&> request_headers, ReadonlyBytes request_body, HTTP::CacheMode cache_mode, HTTP::Cookie::IncludeCredentials include_credentials, ::Proxy::ProxyData const& proxy_data)
 {
     auto request_id = m_next_request_id++;
     auto headers = request_headers.map([](auto const& headers) { return headers.headers().span(); }).value_or({});
 
-    IPCProxy::async_start_request(request_id, method, url, headers, request_body, cache_mode, include_credentials);
+    IPCProxy::async_start_request(request_id, method, url, headers, request_body, cache_mode, include_credentials, proxy_data);
     auto request = Request::create_from_id({}, *this, request_id);
     m_requests.set(request_id, request);
     return request;

Libraries/LibRequests/RequestClient.h

@@ -13,6 +13,7 @@
 #include <LibHTTP/Cookie/IncludeCredentials.h>
 #include <LibHTTP/HeaderList.h>
 #include <LibIPC/ConnectionToServer.h>
+#include <LibProxy/Proxy.h>
 #include <LibRequests/CacheSizes.h>
 #include <LibRequests/RequestTimingInfo.h>
 #include <LibRequests/WebSocket.h>
@@ -35,7 +36,7 @@ class RequestClient final
     explicit RequestClient(NonnullOwnPtr<IPC::Transport>);
     virtual ~RequestClient() override;
 
-    RefPtr<Request> start_request(ByteString const& method, URL::URL const&, Optional<HTTP::HeaderList const&> request_headers = {}, ReadonlyBytes request_body = {}, HTTP::CacheMode = HTTP::CacheMode::Default, HTTP::Cookie::IncludeCredentials = HTTP::Cookie::IncludeCredentials::Yes);
+    RefPtr<Request> start_request(ByteString const& method, URL::URL const&, Optional<HTTP::HeaderList const&> request_headers = {}, ReadonlyBytes request_body = {}, HTTP::CacheMode = HTTP::CacheMode::Default, HTTP::Cookie::IncludeCredentials = HTTP::Cookie::IncludeCredentials::Yes, ::Proxy::ProxyData const& proxy_data = {});
     bool stop_request(Badge<Request>, Request&);
     void ensure_connection(URL::URL const&, RequestServer::CacheLevel);