Browse Source

Profiler: Extract the bar label String into a private method

Rok Povsic 3 năm trước cách đây
mục cha
commit
d3a80b1a6e

+ 11 - 5
Userland/DevTools/Profiler/FlameGraphView.cpp

@@ -116,11 +116,7 @@ void FlameGraphView::paint_event(GUI::PaintEvent& event)
     painter.add_clip_rect(event.rect());
     painter.add_clip_rect(event.rect());
 
 
     for (const auto& bar : m_bars) {
     for (const auto& bar : m_bars) {
-        auto label_index = bar.index.sibling_at_column(m_text_column);
-        String label = "All";
-        if (label_index.is_valid()) {
-            label = m_model.data(label_index).to_string();
-        }
+        auto label = bar_label(bar);
 
 
         auto color = m_colors[label.hash() % m_colors.size()];
         auto color = m_colors[label.hash() % m_colors.size()];
 
 
@@ -148,6 +144,16 @@ void FlameGraphView::paint_event(GUI::PaintEvent& event)
     }
     }
 }
 }
 
 
+String FlameGraphView::bar_label(StackBar const& bar) const
+{
+    auto label_index = bar.index.sibling_at_column(m_text_column);
+    String label = "All";
+    if (label_index.is_valid()) {
+        label = m_model.data(label_index).to_string();
+    }
+    return label;
+}
+
 void FlameGraphView::layout_bars()
 void FlameGraphView::layout_bars()
 {
 {
     m_bars.clear();
     m_bars.clear();

+ 1 - 0
Userland/DevTools/Profiler/FlameGraphView.h

@@ -45,6 +45,7 @@ private:
         bool selected;
         bool selected;
     };
     };
 
 
+    String bar_label(StackBar const&) const;
     void layout_bars();
     void layout_bars();
     void layout_children(GUI::ModelIndex& parent, int depth, int left, int right, Vector<GUI::ModelIndex>& selected);
     void layout_children(GUI::ModelIndex& parent, int depth, int left, int right, Vector<GUI::ModelIndex>& selected);