-
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 profile stats to the CPU profiler #5340
Conversation
@@ -365,224 +367,62 @@ class _CpuProfilerState extends State<CpuProfiler> | |||
} | |||
} | |||
|
|||
class DisplayTreeGuidelinesToggle extends StatelessWidget { |
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.
all the deletes over here were moved to profiler_controls.dart with no functional changes. git diff picked this up weird unfortunately.
class DisplayTreeGuidelinesToggle extends StatelessWidget { | ||
const DisplayTreeGuidelinesToggle(); | ||
// TODO(kenz): one improvement we could make on this is to show the denominator | ||
// for filtered profiles (e.g. 'Sample count: 10/14), or to at least show the |
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.
review all the additions from here to the bottom of the file as new code - not a modification of code on the left.
@@ -0,0 +1,241 @@ | |||
// Copyright 2019 The Chromium Authors. All rights reserved. |
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.
no functional changes to this file. all moves from cpu_profiler.dart
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.
LGTM with a couple of comments
Work towards #3861
While I was here, I also created a new file
profiler_controls.dart
and pulled some controls from cpu_profiler.dart out into the new file. This was to make the cpu_profiler.dart file more manageable.