Title: [256719] trunk/Source/WebKit
- Revision
- 256719
- Author
- [email protected]
- Date
- 2020-02-16 18:11:42 -0800 (Sun, 16 Feb 2020)
Log Message
Log WebSocket failure in case of network crash
https://bugs.webkit.org/show_bug.cgi?id=207801
Reviewed by Darin Adler.
Covered by http/tests/websocket/tests/hybi/network-process-crash-error.html.
* WebProcess/Network/WebSocketChannel.cpp:
(WebKit::WebSocketChannel::didReceiveMessageError):
(WebKit::WebSocketChannel::networkProcessCrashed):
Add a console log message in case of error coming from network process.
Modified Paths
Diff
Modified: trunk/Source/WebKit/ChangeLog (256718 => 256719)
--- trunk/Source/WebKit/ChangeLog 2020-02-17 02:06:19 UTC (rev 256718)
+++ trunk/Source/WebKit/ChangeLog 2020-02-17 02:11:42 UTC (rev 256719)
@@ -1,3 +1,17 @@
+2020-02-16 Youenn Fablet <[email protected]>
+
+ Log WebSocket failure in case of network crash
+ https://bugs.webkit.org/show_bug.cgi?id=207801
+
+ Reviewed by Darin Adler.
+
+ Covered by http/tests/websocket/tests/hybi/network-process-crash-error.html.
+
+ * WebProcess/Network/WebSocketChannel.cpp:
+ (WebKit::WebSocketChannel::didReceiveMessageError):
+ (WebKit::WebSocketChannel::networkProcessCrashed):
+ Add a console log message in case of error coming from network process.
+
2020-02-16 Brady Eidson <[email protected]>
Expose "allowsContentJavaScript" on WKWebpagePreferences
Modified: trunk/Source/WebKit/WebProcess/Network/WebSocketChannel.cpp (256718 => 256719)
--- trunk/Source/WebKit/WebProcess/Network/WebSocketChannel.cpp 2020-02-17 02:06:19 UTC (rev 256718)
+++ trunk/Source/WebKit/WebProcess/Network/WebSocketChannel.cpp 2020-02-17 02:11:42 UTC (rev 256719)
@@ -298,13 +298,15 @@
return;
}
- // FIXME: do something with errorMessage.
+ if (m_document)
+ m_document->addConsoleMessage(MessageSource::Network, MessageLevel::Error, errorMessage);
+
m_client->didReceiveMessageError();
}
void WebSocketChannel::networkProcessCrashed()
{
- didReceiveMessageError({ });
+ didReceiveMessageError("WebSocket network error: Network process crashed."_s);
}
void WebSocketChannel::suspend()
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes
