eliott/ladybird
1
0
Fork 0
mirror of https://github.com/LadybirdBrowser/ladybird synced 2026-05-05 06:32:30 +02:00
Code Issues Packages Projects Releases Wiki Activity

NotificationServer: Close connection on notification close

Browse Source 
When the notification was closed, the connection was kept around. This
caused the core event loop to take up nearly all CPU, so instead of
checking the connection we clear it on close and add state variables
to check state.
This commit is contained in:
Nick Johnson
2021-03-13 15:51:33 -06:00
committed by Andreas Kling
parent 9d09594e44
commit ddcef0452a
Notes: sideshowbarker 2024-07-18 21:09:04 +09:00 
Author: https://github.com/Sylvyrfysh
Commit: https://github.com/SerenityOS/serenity/commit/ddcef0452aa
Pull-request: https://github.com/SerenityOS/serenity/pull/5732
Reviewed-by: https://github.com/BenWiederhake
2 changed files with 33 additions and 12 deletions
Download Patch File Download Diff File Expand all files Collapse all files

37
Userland/Libraries/LibGUI/Notification.cpp
View File

@@ -34,23 +34,28 @@ namespace GUI {
class NotificationServerConnection : public IPC::ServerConnection<NotificationClientEndpoint, NotificationServerEndpoint>
, public NotificationClientEndpoint {
C_OBJECT(NotificationServerConnection)
friend class Notification;
public:
virtual void handshake() override
{
send_sync<Messages::NotificationServer::Greet>();
}
virtual void die() override { m_connected = false; }
bool is_connected() const { return m_connected; }
virtual void die() override
{
m_notification->connection_closed();
}
private:
NotificationServerConnection()
explicit NotificationServerConnection(Notification* notification)
: IPC::ServerConnection<NotificationClientEndpoint, NotificationServerEndpoint>(*this, "/tmp/portal/notify")
, m_notification(notification)
{
}
virtual void handle(const Messages::NotificationClient::Dummy&) override { }
bool m_connected { true };
Notification* m_notification;
};
Notification::Notification()
@@ -63,25 +68,27 @@ Notification::~Notification()
void Notification::show()
{
VERIFY(!m_connection);
VERIFY(!m_shown && !m_destroyed);
auto icon = m_icon ? m_icon->to_shareable_bitmap() : Gfx::ShareableBitmap();
m_connection = NotificationServerConnection::construct();
m_connection = NotificationServerConnection::construct(this);
m_connection->send_sync<Messages::NotificationServer::ShowNotification>(m_text, m_title, icon);
m_shown = true;
}
void Notification::close()
{
VERIFY(m_connection);
if (!m_connection->is_connected()) {
VERIFY(m_shown);
if (!m_destroyed) {
m_connection->send_sync<Messages::NotificationServer::CloseNotification>();
connection_closed();
return;
}
m_connection->send_sync<Messages::NotificationServer::CloseNotification>();
}
bool Notification::update()
{
VERIFY(m_connection);
if (!m_connection->is_connected()) {
VERIFY(m_shown);
if (m_destroyed) {
return false;
}
@@ -99,4 +106,10 @@ bool Notification::update()
return true;
}
void Notification::connection_closed()
{
m_connection.clear();
m_destroyed = true;
}
}