Title: [203762] trunk/Source/WebCore
- Revision
- 203762
- Author
- [email protected]
- Date
- 2016-07-26 22:20:04 -0700 (Tue, 26 Jul 2016)
Log Message
[GTK] ASSERTION FAILED: !m_adoptionIsRequired when Inspector Server is connected
https://bugs.webkit.org/show_bug.cgi?id=160229
Patch by Fujii Hironori <[email protected]> on 2016-07-26
Reviewed by Carlos Garcia Campos.
An assertion fails because refcount of SocketStreamHandle is
incremented before adoptRef, in the constructor of
SocketStreamHandle. The constructor of SocketStreamHandle needs
to increment recount because it passes this pointer to libsoup.
* platform/network/soup/SocketStreamHandleSoup.cpp:
(WebCore::SocketStreamHandle::SocketStreamHandle): Do
relaxAdoptionRequirement() as well as the another constructor.
Modified Paths
Diff
Modified: trunk/Source/WebCore/ChangeLog (203761 => 203762)
--- trunk/Source/WebCore/ChangeLog 2016-07-27 04:27:12 UTC (rev 203761)
+++ trunk/Source/WebCore/ChangeLog 2016-07-27 05:20:04 UTC (rev 203762)
@@ -1,3 +1,19 @@
+2016-07-26 Fujii Hironori <[email protected]>
+
+ [GTK] ASSERTION FAILED: !m_adoptionIsRequired when Inspector Server is connected
+ https://bugs.webkit.org/show_bug.cgi?id=160229
+
+ Reviewed by Carlos Garcia Campos.
+
+ An assertion fails because refcount of SocketStreamHandle is
+ incremented before adoptRef, in the constructor of
+ SocketStreamHandle. The constructor of SocketStreamHandle needs
+ to increment recount because it passes this pointer to libsoup.
+
+ * platform/network/soup/SocketStreamHandleSoup.cpp:
+ (WebCore::SocketStreamHandle::SocketStreamHandle): Do
+ relaxAdoptionRequirement() as well as the another constructor.
+
2016-07-26 Chris Dumez <[email protected]>
Move 'dir' attribute from HTMLDocument to Document
Modified: trunk/Source/WebCore/platform/network/soup/SocketStreamHandleSoup.cpp (203761 => 203762)
--- trunk/Source/WebCore/platform/network/soup/SocketStreamHandleSoup.cpp 2016-07-27 04:27:12 UTC (rev 203761)
+++ trunk/Source/WebCore/platform/network/soup/SocketStreamHandleSoup.cpp 2016-07-27 05:20:04 UTC (rev 203762)
@@ -70,6 +70,7 @@
{
LOG(Network, "SocketStreamHandle %p new client %p", this, &m_client);
GRefPtr<GSocketConnection> connection = socketConnection;
+ relaxAdoptionRequirement();
connected(WTFMove(connection));
}
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes
