Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[GTK][WPE] PlatformDisplay::terminateEglDisplays() is never called #1271

Merged
merged 1 commit into from
Jan 9, 2024

Conversation

pgorszkowski-igalia
Copy link

https://bugs.webkit.org/show_bug.cgi?id=217655

Stop the run loop for GTK and WPE to ensure a normal exit, since we need atexit handlers to be called to cleanup resources like EGL displays.

Reviewed by Carlos Garcia Campos.

  • Shared/AuxiliaryProcess.cpp: (WebKit::AuxiliaryProcess::didClose):
  • WebProcess/WebProcess.cpp: (WebKit::WebProcess::initializeConnection):
  • WebProcess/WebProcess.h:
  • WebProcess/glib/WebProcessGLib.cpp: (WebKit::WebProcess::stopRunLoop):

Canonical link: https://commits.webkit.org/247678@main
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@290360 268f45cc-cd09-0410-ab3c-d52691b4dbfc

https://bugs.webkit.org/show_bug.cgi?id=217655

Stop the run loop for GTK and WPE to ensure a normal exit, since we need
atexit handlers to be called to cleanup resources like EGL displays.

Reviewed by Carlos Garcia Campos.

* Shared/AuxiliaryProcess.cpp:
(WebKit::AuxiliaryProcess::didClose):
* WebProcess/WebProcess.cpp:
(WebKit::WebProcess::initializeConnection):
* WebProcess/WebProcess.h:
* WebProcess/glib/WebProcessGLib.cpp:
(WebKit::WebProcess::stopRunLoop):

Canonical link: https://commits.webkit.org/247678@main
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@290360 268f45cc-cd09-0410-ab3c-d52691b4dbfc
@pgorszkowski-igalia
Copy link
Author

This change introduced a regression in upstream (https://bugs.webkit.org/show_bug.cgi?id=238494) but the regression was related to GTK and seems to be safe for WPE. I discussed it with cgarcia.

@magomez magomez merged commit c1a8d2c into wpe-2.28 Jan 9, 2024
@magomez magomez deleted the pgorszkowski/2.28/close_pages_on_webprocess_exit branch January 9, 2024 16:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
wpe-2.28 Only for PR affecting 2.28
Development

Successfully merging this pull request may close these issues.

3 participants