-
Notifications
You must be signed in to change notification settings - Fork 314
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add Method Table to CPU profiler #5366
Conversation
author Kenzie Schmoll <kenzieschmoll@google.com> 1678230010 -0800 committer Kenzie Schmoll <kenzieschmoll@google.com> 1678235124 -0800 parent ee98f44 author Kenzie Schmoll <kenzieschmoll@google.com> 1678230010 -0800 committer Kenzie Schmoll <kenzieschmoll@google.com> 1678235093 -0800 Make FlatTable scrollable when there is a single wide column revert test change fix up table formatting formatting size method table to fit Add Method Table to the CPU Profiler
packages/devtools_app/lib/src/screens/profiler/panes/method_table/method_table_model.dart
Outdated
Show resolved
Hide resolved
@@ -38,6 +39,8 @@ class CpuMethodTable extends StatelessWidget { | |||
} | |||
} | |||
|
|||
// TODO(kenz): ensure that this table automatically scrolls to the selected |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
When would this happen?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If you select a method from anywhere in the method graph (callees and callers), the graph reroots to that method, which means the method table should scroll to the new selection. We may eventually want to maintain some sort of history here for this reason.
.round(), | ||
); | ||
/// The number of cpu samples where this method is on top of the stack. | ||
int get selfCount => _selfCount; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can this just be final int selfCount
? Ditto for totalCount
.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
_selfCount and _totalCount are non-final because they are modified in the merge
method.
Fixes #841