From 3ed5a73edea567002e9cd38dd475e376357204d2 Mon Sep 17 00:00:00 2001 From: Andreas Kling Date: Fri, 14 May 2021 17:04:31 +0200 Subject: [PATCH] Profiler: Mark model columns as non-sortable This removes the ability to click on the column headers to resort. Resorting didn't do anything anyway. --- Userland/DevTools/Profiler/DisassemblyModel.h | 1 + Userland/DevTools/Profiler/ProfileModel.h | 1 + Userland/DevTools/Profiler/SamplesModel.h | 1 + 3 files changed, 3 insertions(+) diff --git a/Userland/DevTools/Profiler/DisassemblyModel.h b/Userland/DevTools/Profiler/DisassemblyModel.h index 6982d03d9f..dd882c35e8 100644 --- a/Userland/DevTools/Profiler/DisassemblyModel.h +++ b/Userland/DevTools/Profiler/DisassemblyModel.h @@ -45,6 +45,7 @@ public: virtual String column_name(int) const override; virtual GUI::Variant data(const GUI::ModelIndex&, GUI::ModelRole) const override; virtual void update() override; + virtual bool is_column_sortable(int) const override { return false; } private: DisassemblyModel(Profile&, ProfileNode&); diff --git a/Userland/DevTools/Profiler/ProfileModel.h b/Userland/DevTools/Profiler/ProfileModel.h index 5678a077ca..84c685940e 100644 --- a/Userland/DevTools/Profiler/ProfileModel.h +++ b/Userland/DevTools/Profiler/ProfileModel.h @@ -37,6 +37,7 @@ public: virtual GUI::ModelIndex parent_index(const GUI::ModelIndex&) const override; virtual void update() override; virtual int tree_column() const override { return Column::StackFrame; } + virtual bool is_column_sortable(int) const override { return false; } private: explicit ProfileModel(Profile&); diff --git a/Userland/DevTools/Profiler/SamplesModel.h b/Userland/DevTools/Profiler/SamplesModel.h index 30c594fcab..8e89d5fc82 100644 --- a/Userland/DevTools/Profiler/SamplesModel.h +++ b/Userland/DevTools/Profiler/SamplesModel.h @@ -37,6 +37,7 @@ public: virtual String column_name(int) const override; virtual GUI::Variant data(const GUI::ModelIndex&, GUI::ModelRole) const override; virtual void update() override; + virtual bool is_column_sortable(int) const override { return false; } private: explicit SamplesModel(Profile&);