diff --git a/DevTools/Profiler/ProcessChooser.cpp b/DevTools/Profiler/ProcessChooser.cpp index b464ef0535..3689176ad6 100644 --- a/DevTools/Profiler/ProcessChooser.cpp +++ b/DevTools/Profiler/ProcessChooser.cpp @@ -52,8 +52,10 @@ void ProcessChooser::build() widget.set_fill_with_background_color(true); widget.set_layout(); auto& table_view = widget.add(); - table_view.set_model(GUI::SortingProxyModel::create(Profiler::RunningProcessesModel::create())); - table_view.model()->set_key_column_and_sort_order(Profiler::RunningProcessesModel::Column::PID, GUI::SortOrder::Descending); + auto sorting_model = GUI::SortingProxyModel::create(Profiler::RunningProcessesModel::create()); + sorting_model->set_sort_role(GUI::Model::Role::Display); + sorting_model->set_key_column_and_sort_order(Profiler::RunningProcessesModel::Column::PID, GUI::SortOrder::Descending); + table_view.set_model(sorting_model); auto& button_container = widget.add(); button_container.set_preferred_size(0, 30); button_container.set_size_policy(GUI::SizePolicy::Fill, GUI::SizePolicy::Fixed); diff --git a/DevTools/Profiler/RunningProcessesModel.cpp b/DevTools/Profiler/RunningProcessesModel.cpp index 2870c9965d..45d08ea850 100644 --- a/DevTools/Profiler/RunningProcessesModel.cpp +++ b/DevTools/Profiler/RunningProcessesModel.cpp @@ -115,21 +115,6 @@ GUI::Variant RunningProcessesModel::data(const GUI::ModelIndex& index, Role role } ASSERT_NOT_REACHED(); } - - if (role == Role::Sort) { - switch (index.column()) { - case Column::Icon: - return {}; - case Column::PID: - return process.pid; - case Column::UID: - return process.uid; - case Column::Name: - return process.name; - } - ASSERT_NOT_REACHED(); - } - return {}; }