Libraries: Fix visibility of Object-derivative constructors

Derivatives of Core::Object should be constructed through
ClassName::construct(), to avoid handling ref-counted objects with
refcount zero. Fixing the visibility means that misuses like this are
more difficult.

2 files changed, 4 insertions, 2 deletions

diff --git a/Userland/Libraries/LibWebSocket/Impl/TCPWebSocketConnectionImpl.h b/Userland/Libraries/LibWebSocket/Impl/TCPWebSocketConnectionImpl.h
index 3c686e194c..1ece3f8bef 100644
--- a/Userland/Libraries/LibWebSocket/Impl/TCPWebSocketConnectionImpl.h
+++ b/Userland/Libraries/LibWebSocket/Impl/TCPWebSocketConnectionImpl.h
@@ -22,7 +22,6 @@ class TCPWebSocketConnectionImpl final : public AbstractWebSocketImpl {
 
 public:
     virtual ~TCPWebSocketConnectionImpl() override;
-    explicit TCPWebSocketConnectionImpl(Core::Object* parent = nullptr);
 
     virtual void connect(ConnectionInfo const& connection) override;
 
@@ -39,6 +38,8 @@ public:
     virtual void discard_connection() override;
 
 private:
+    explicit TCPWebSocketConnectionImpl(Core::Object* parent = nullptr);
+
     RefPtr<Core::Notifier> m_notifier;
     RefPtr<Core::TCPSocket> m_socket;
 };
diff --git a/Userland/Libraries/LibWebSocket/Impl/TLSv12WebSocketConnectionImpl.h b/Userland/Libraries/LibWebSocket/Impl/TLSv12WebSocketConnectionImpl.h
index fecdd4d38f..6cfe8b1592 100644
--- a/Userland/Libraries/LibWebSocket/Impl/TLSv12WebSocketConnectionImpl.h
+++ b/Userland/Libraries/LibWebSocket/Impl/TLSv12WebSocketConnectionImpl.h
@@ -21,7 +21,6 @@ class TLSv12WebSocketConnectionImpl final : public AbstractWebSocketImpl {
 
 public:
     virtual ~TLSv12WebSocketConnectionImpl() override;
-    explicit TLSv12WebSocketConnectionImpl(Core::Object* parent = nullptr);
 
     void connect(ConnectionInfo const& connection) override;
 
@@ -38,6 +37,8 @@ public:
     virtual void discard_connection() override;
 
 private:
+    explicit TLSv12WebSocketConnectionImpl(Core::Object* parent = nullptr);
+
     RefPtr<TLS::TLSv12> m_socket;
 };