<html>
<head>
<base href="https://bugs.webkit.org/" />
</head>
<body>
<p>
<div>
<b><a class="bz_bug_link
bz_status_NEW "
title="NEW - [GTK] Crash of WebProcess on the last WebView disconnect"
href="https://bugs.webkit.org/show_bug.cgi?id=161605#c13">Comment # 13</a>
on <a class="bz_bug_link
bz_status_NEW "
title="NEW - [GTK] Crash of WebProcess on the last WebView disconnect"
href="https://bugs.webkit.org/show_bug.cgi?id=161605">bug 161605</a>
from <span class="vcard"><a class="email" href="mailto:mcatanzaro@igalia.com" title="Michael Catanzaro <mcatanzaro@igalia.com>"> <span class="fn">Michael Catanzaro</span></a>
</span></b>
<pre>(In reply to <a href="show_bug.cgi?id=161605#c6">comment #6</a>)
<span class="quote">> This is not accurate. PlatformDisplay is not deleted in an exit handler,
> it's a static std::unique_ptr that is deleted after all exit handlers.</span >
Where do you think static objects get deleted. :)
<span class="quote">> Never liked that, TBH, I was tempted to remove it when reworked the
> GLContext. It's probably a workaround for a bug in nvidia drivers that might
> have already been fixed. In any case, all contexts created now except the
> sharing context that is owned by the PlatformDisplay should be freed already
> on exit.</span >
Agreed.
<span class="quote">> This is not possible because the GLContext exit handler is a workaround used
> on for X11 contexts, and other classes shouldn't rely on that.</span >
Good point.
<span class="quote">> > create a throwaway GLContext the
> > first time PlatformDisplay::initializeEGLDisplay is called to ensure the
> > GLContext exit handler gets registered first (and therefore executed last at
> > exit time)... let's see if Carlos Garcia has an opinion on what's best.
>
> This has nothing to do with EGLDisplay, the crash is happening on
> ~PlatformDisplay and the exit handler of the EGLDisplay doesn't delete the
> PlatformDisplay.</span >
Yes, I was completely wrong; for some reason I thought we were deleting the PlatformDisplay in the exit handler registered in that function, but clearly that's not the case.</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>