Let MetricsService know about some Android Activities

chrome/common/pref_names.cc

Index: chrome/common/pref_names.cc
diff --git a/chrome/common/pref_names.cc b/chrome/common/pref_names.cc
index eece5eb62a8a99984546ae59c251e176d5383896..17dab90678683f685312dd2621420ad8782f4520 100644
--- a/chrome/common/pref_names.cc
+++ b/chrome/common/pref_names.cc
@@ -1493,6 +1493,33 @@ const char kStabilityKernelCrashCount[] =
 const char kStabilitySystemUncleanShutdownCount[] =
     "user_experience_metrics.stability.system_unclean_shutdowns";
 
+#if defined(OS_ANDROID)
+// ID of the Activity that is currently in the foreground.

[Alexei Svitkine (slow), 2013/12/23 16:28:01]

What kind of ID and how is it stored (string / int)? Document this.

[gone, 2013/12/28 01:33:14]

Done.

+const char kStabilityForegroundActivityType[] =
+    "user_experience_metrics.stability.foreground_activity";
+
+// ORed set of flags indicating which Activities were launched during the

[Alexei Svitkine (slow), 2013/12/23 16:28:01]

Which flags? Can you point to the corresponding constants/enums in the code in
this comment?

[gone, 2013/12/28 01:33:14]

Done.

+// last session.
+const char kStabilityLaunchedActivityFlags[] =
+    "user_experience_metrics.stability.launched_activity_flags";
+
+// Counts how many times the main browser Activity was launched.
+const char kStabilityLaunchCountMainActivity[] =
+    "user_experience_metrics.stability.launch_count_main_activity";

[Alexei Svitkine (slow), 2013/12/23 16:28:01]

Instead of special casing main activity, full screen activity, etc, would it
make more sense to use a list item here, where each entry has activity_type,
launch_count, crash_count, is_foreground? Seems like that would be cleaner and
more extensible in the future. (See how we store plugins for UMA stability for
an example of this).

[gone, 2013/12/28 01:33:14]

Which activity is in a singular value; only one can be in the foreground at any
given time.  It gets translated to a crash state later.  I've switched to lists
otherwise; thanks for the heads up.

+
+// Counts how many times a FullScreenActivity was launched.
+const char kStabilityLaunchCountFullScreenActivity[] =
+    "user_experience_metrics.stability.launch_count_full_screen_activity";
+
+// Counts how many times the main browser Activity crashed.
+const char kStabilityCrashCountMainActivity[] =
+    "user_experience_metrics.stability.crash_count_main_activity";
+
+// Counts how many times a FullScreenActivity crashed.
+const char kStabilityCrashCountFullScreenActivity[] =
+    "user_experience_metrics.stability.crash_count_full_screen_activity";
+#endif
+
 // Number of times the browser has been able to register crash reporting.
 const char kStabilityBreakpadRegistrationSuccess[] =
     "user_experience_metrics.stability.breakpad_registration_ok";