Changeset 252853 in webkit

Timestamp:

Nov 25, 2019 3:39:48 AM (4 years ago)

Author:

Carlos Garcia Campos

Message:

REGRESSION(r252770): [GTK][WPE] Remove inspector broken in debug builds after r252770
​https://bugs.webkit.org/show_bug.cgi?id=204569

Reviewed by Žan Doberšek.

We need to call relaxAdoptionRequirement() in SocketConnection constructor because we are taking a reference for
the read monitor lambda.

• wtf/glib/SocketConnection.cpp:

(WTF::SocketConnection::SocketConnection):

Location:

trunk/Source/WTF

Files:

• ChangeLog (modified) (1 diff)
• wtf/glib/SocketConnection.cpp (modified) (1 diff)

trunk/Source/WTF/ChangeLog

@@ +1 @@
+2019-11-25  Carlos Garcia Campos  <cgarcia@igalia.com>
+
+        REGRESSION(r252770): [GTK][WPE] Remove inspector broken in debug builds after r252770
+        https://bugs.webkit.org/show_bug.cgi?id=204569
+
+        Reviewed by Žan Doberšek.
+
+        We need to call relaxAdoptionRequirement() in SocketConnection constructor because we are taking a reference for
+        the read monitor lambda.
+
+        * wtf/glib/SocketConnection.cpp:
+        (WTF::SocketConnection::SocketConnection):
+
 2019-11-25  Fujii Hironori  <Hironori.Fujii@sony.com>
 

trunk/Source/WTF/wtf/glib/SocketConnection.cpp

@@ -35 +35 @@
     , m_userData(userData)
 {
+    relaxAdoptionRequirement();
+
     m_readBuffer.reserveInitialCapacity(defaultBufferSize);
     m_writeBuffer.reserveInitialCapacity(defaultBufferSize);