Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(242)

Unified Diff: components/metrics/call_stack_profile_metrics_provider.h

Issue 2444143002: Add process lifetime annotations to stack samples. (Closed)
Patch Set: fixed non-windows constant Created 4 years, 1 month ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: components/metrics/call_stack_profile_metrics_provider.h
diff --git a/components/metrics/call_stack_profile_metrics_provider.h b/components/metrics/call_stack_profile_metrics_provider.h
index bd0b721c8093dc4ad9764d64c74f951932e2b9cb..8a2ea737decf0cf0db6c3a29b1117281343bf90a 100644
--- a/components/metrics/call_stack_profile_metrics_provider.h
+++ b/components/metrics/call_stack_profile_metrics_provider.h
@@ -19,6 +19,31 @@ class ChromeUserMetricsExtension;
// Performs metrics logging for the stack sampling profiler.
class CallStackProfileMetricsProvider : public MetricsProvider {
public:
+ // These phases of a process lifetime can be passed as process "phases" to
+ // StackSmaplingProfile::SetProcessPhase(). Be sure to update the translation
+ // constants at the top of the .cc file when this is changed.
+ enum Phases : int {
+ MAIN_LOOP_START,
+ MAIN_NAVIGATION_START,
+ FIRST_NONEMPTY_PAINT,
+ FIRST_CONTENTFUL_PAINT,
+
+ SHUTDOWN_START,
+
+ PHASES_MAX_VALUE
+ };
+
+ // These common activities that occur within the browser lifetime can be
+ // passed to StackSmaplingProfile::ProcessActivityBegin/End(). Be sure to
+ // update the translation constants at the top of the .cc file when this is
+ // changed.
+ enum Activities : int {
+ COLLECTING_METRICS,
+ WRITING_PROFILES_STATE,
+
+ ACTIVITIES_MAX_VALUE
+ };
+
CallStackProfileMetricsProvider();
~CallStackProfileMetricsProvider() override;

Powered by Google App Engine
This is Rietveld 408576698