Title: [240317] trunk/Source/WebKit
Revision
240317
Author
[email protected]
Date
2019-01-22 20:06:56 -0800 (Tue, 22 Jan 2019)

Log Message

Unreviewed attempt to fix GTK/WPE bots
https://bugs.webkit.org/show_bug.cgi?id=193580
<rdar://problem/47457742>

Doesn't make sense to clear the storage session right after creating it. This should fix
network process crash on startup under G_DEBUG=fatal-criticals due to the storage session
not having a cookie jar.

* NetworkProcess/NetworkProcess.cpp:
(WebKit::NetworkProcess::NetworkProcess):

Modified Paths

Diff

Modified: trunk/Source/WebKit/ChangeLog (240316 => 240317)


--- trunk/Source/WebKit/ChangeLog	2019-01-23 02:12:08 UTC (rev 240316)
+++ trunk/Source/WebKit/ChangeLog	2019-01-23 04:06:56 UTC (rev 240317)
@@ -1,3 +1,16 @@
+2019-01-22  Michael Catanzaro  <[email protected]>
+
+        Unreviewed attempt to fix GTK/WPE bots
+        https://bugs.webkit.org/show_bug.cgi?id=193580
+        <rdar://problem/47457742>
+
+        Doesn't make sense to clear the storage session right after creating it. This should fix
+        network process crash on startup under G_DEBUG=fatal-criticals due to the storage session
+        not having a cookie jar.
+
+        * NetworkProcess/NetworkProcess.cpp:
+        (WebKit::NetworkProcess::NetworkProcess):
+
 2019-01-22  Megan Gardner  <[email protected]>
 
         Cancel Web Touches Properly so that long presses on YouTube links do not incorrectly trigger a load

Modified: trunk/Source/WebKit/NetworkProcess/NetworkProcess.cpp (240316 => 240317)


--- trunk/Source/WebKit/NetworkProcess/NetworkProcess.cpp	2019-01-23 02:12:08 UTC (rev 240316)
+++ trunk/Source/WebKit/NetworkProcess/NetworkProcess.cpp	2019-01-23 04:06:56 UTC (rev 240317)
@@ -163,7 +163,6 @@
     DNSResolveQueueSoup::setGlobalDefaultNetworkStorageSessionAccessor([this]() -> NetworkStorageSession& {
         return defaultStorageSession();
     });
-    defaultStorageSession().clearSoupNetworkSessionAndCookieStorage();
 #endif
 
     NetworkStateNotifier::singleton().addListener([this](bool isOnLine) {
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to