<html>
<head>
<base href="https://bugs.webkit.org/">
</head>
<body>
<p>
<div>
<b><a class="bz_bug_link
bz_status_NEW "
title="NEW - [WPE] PlatformDisplay::shutDownEglDisplays() is never called"
href="https://bugs.webkit.org/show_bug.cgi?id=217655#c14">Comment # 14</a>
on <a class="bz_bug_link
bz_status_NEW "
title="NEW - [WPE] PlatformDisplay::shutDownEglDisplays() is never called"
href="https://bugs.webkit.org/show_bug.cgi?id=217655">bug 217655</a>
from <span class="vcard"><a class="email" href="mailto:psaavedra@igalia.com" title="Pablo Saavedra <psaavedra@igalia.com>"> <span class="fn">Pablo Saavedra</span></a>
</span></b>
<pre>Also I've noticed that this commit (<a class="bz_bug_link
bz_status_RESOLVED bz_closed"
title="RESOLVED FIXED - [WK2] Make establishing a connection between the WebProcess and the Network process more robust"
href="show_bug.cgi?id=214307">https://bugs.webkit.org/show_bug.cgi?id=214307</a>) have removed some specific logic that affects to the WPE and GTK ports:
```
commit 813abc09e84c9a406836319bfa8f7fb83d378605
Author: <a href="mailto:cdumez@apple.com">cdumez@apple.com</a> <<a href="mailto:cdumez@apple.com">cdumez@apple.com</a>@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Date: Wed Jul 15 18:23:22 2020 +0000
[WK2] Make establishing a connection between the WebProcess and the Network process more robust
<a class="bz_bug_link
bz_status_RESOLVED bz_closed"
title="RESOLVED FIXED - [WK2] Make establishing a connection between the WebProcess and the Network process more robust"
href="show_bug.cgi?id=214307">https://bugs.webkit.org/show_bug.cgi?id=214307</a>
<<a href="rdar://problem/64592340">rdar://problem/64592340</a>>
Reviewed by Alex Christensen.
```
``` diff
diff --git a/Source/WebKit/WebProcess/WebProcess.cpp b/Source/WebKit/WebProcess/WebProcess.cpp
index 6f2d8fad165f..c73c43e9857c 100644
--- a/Source/WebKit/WebProcess/WebProcess.cpp
+++ b/Source/WebKit/WebProcess/WebProcess.cpp
@@ -1100,23 +1100,28 @@ void WebProcess::setInjectedBundleParameters(const IPC::DataReference& value)
static NetworkProcessConnectionInfo getNetworkProcessConnection(IPC::Connection& connection)
{
NetworkProcessConnectionInfo connectionInfo;
- if (!connection.sendSync(Messages::WebProcessProxy::GetNetworkProcessConnection(), Messages::WebProcessProxy::GetNetworkProcessConnection::Reply(connectionInfo), 0)) {
- // If we failed the first time, retry once. The attachment may have become invalid
- // before it was received by the web process if the network process crashed.
- if (!connection.sendSync(Messages::WebProcessProxy::GetNetworkProcessConnection(), Messages::WebProcessProxy::GetNetworkProcessConnection::Reply(connectionInfo), 0)) {
-#if PLATFORM(GTK) || PLATFORM(WPE)
- // GTK+ and WPE ports don't exit on send sync message failure.
- // In this particular case, the network process can be terminated by the UI process while the
- // Web process is still initializing, so we always want to exit instead of crashing. This can
- // happen when the WebView is created and then destroyed quickly.
- // See <a class="bz_bug_link
bz_status_RESOLVED bz_closed"
title="RESOLVED FIXED - REGRESSION(r222772): [GTK][WPE] WebProcess from WebKitGtk+ 2.19.9x SIGSEVs in WebKit::WebProcess::ensureNetworkProcessConnection() at Source/WebKit/WebProcess/WebProcess.cpp:1127"
href="show_bug.cgi?id=183348">https://bugs.webkit.org/show_bug.cgi?id=183348</a>.
+ auto requestConnection = [&] {
+ if (!connection.isValid()) {
+ // Connection to UIProcess has been severed, exit cleanly.
exit(0);
-#else
- CRASH();
-#endif
...
```
The change removes the specific logic for WPE and GTK added in:
```
commit f0627f5287186ec975c3439e9bcfe119493adc95
Author: <a href="mailto:carlosgc@webkit.org">carlosgc@webkit.org</a> <<a href="mailto:carlosgc@webkit.org">carlosgc@webkit.org</a>@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Date: Thu May 3 08:45:32 2018 +0000
REGRESSION(<a href="https://commits.webkit.org/r222772">r222772</a>): [GTK][WPE] WebProcess from WebKitGtk+ 2.19.9x SIGSEVs in WebKit::WebProcess::ensureNetworkProcessConnection() at Source/WebKit/WebProcess/WebProcess.cpp:1127
<a class="bz_bug_link
bz_status_RESOLVED bz_closed"
title="RESOLVED FIXED - REGRESSION(r222772): [GTK][WPE] WebProcess from WebKitGtk+ 2.19.9x SIGSEVs in WebKit::WebProcess::ensureNetworkProcessConnection() at Source/WebKit/WebProcess/WebProcess.cpp:1127"
href="show_bug.cgi?id=183348">https://bugs.webkit.org/show_bug.cgi?id=183348</a>
```
The comment in <a class="bz_bug_link
bz_status_RESOLVED bz_closed"
title="RESOLVED FIXED - REGRESSION(r222772): [GTK][WPE] WebProcess from WebKitGtk+ 2.19.9x SIGSEVs in WebKit::WebProcess::ensureNetworkProcessConnection() at Source/WebKit/WebProcess/WebProcess.cpp:1127"
href="show_bug.cgi?id=183348#c40">https://bugs.webkit.org/show_bug.cgi?id=183348#c40</a> offers a justification for that code.</pre>
</div>
</p>
<hr>
<span>You are receiving this mail because:</span>
<ul>
<li>You are the assignee for the bug.</li>
</ul>
</body>
</html>