mirrors/ladybird
0
0
Fork 0
mirror of https://github.com/LadybirdBrowser/ladybird.git synced 2025-08-03 14:50:18 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions 9

SystemMonitor: Register ProcessStateModel with ProcessModel on construct

Browse source 
Since this was missing, ProcessStateModel never received
model_did_update, and it was possible for the ProcessModel index to go
stale and show another process.

Fixes #9129.
This commit is contained in:
sin-ack 2021-07-31 16:22:40 +00:00 • committed by Ali Mohammad Pur
commit eb6cf00c0e
Notes: sideshowbarker 2024-07-18 07:38:38 +09:00
1 changed files with 6 additions and 1 deletions
Download patch file Download diff file Expand all files Collapse all files

7
Userland/Applications/SystemMonitor/ProcessStateWidget.cpp
View file

@ -23,9 +23,14 @@ public:
: m_target(target)
, m_pid(pid)
{
m_target.register_client(*this);
refresh();
}
virtual ~ProcessStateModel() override { }
virtual ~ProcessStateModel() override
{
m_target.unregister_client(*this);
}
virtual int row_count(const GUI::ModelIndex& = GUI::ModelIndex()) const override { return m_target.column_count({}); }
virtual int column_count(const GUI::ModelIndex& = GUI::ModelIndex()) const override { return 2; }