Let MetricsService know about some Android Activities

chrome/common/metrics/metrics_service_base.cc

Index: chrome/common/metrics/metrics_service_base.cc
diff --git a/chrome/common/metrics/metrics_service_base.cc b/chrome/common/metrics/metrics_service_base.cc
index 5f4edc25e44f8f82901d7ea88ee33de5104ab026..05bb0e066f136e75fe138f6d0473a9ef1f191d78 100644
--- a/chrome/common/metrics/metrics_service_base.cc
+++ b/chrome/common/metrics/metrics_service_base.cc
@@ -26,7 +26,17 @@ const char MetricsServiceBase::kMimeType[] = "application/vnd.chrome.uma";
 
 void MetricsServiceBase::RecordCurrentHistograms() {
   DCHECK(log_manager_.current_log());
-  histogram_snapshot_manager_.PrepareDeltas(base::Histogram::kNoFlags, true);
+  histogram_snapshot_manager_.PrepareDeltas(
+      base::Histogram::kNoFlags, base::Histogram::kUmaTargetedHistogramFlag);
+}
+
+void MetricsServiceBase::RecordCurrentStabilityHistograms() {
+  DCHECK(log_manager_.current_log());
+  histogram_snapshot_manager_.PrepareDeltas(
+      base::Histogram::kNoFlags,
+      static_cast<base::Histogram::Flags>(

[Alexei Svitkine (slow), 2014/01/27 19:12:54]

This cast is kind of ugly, but I understand why it's there.

I know I'm changing my mind here, but now I think it would be better to have the
stability histogram mean both UMA and Stability, so that this isn't needed.

However I think it can be defined thus:

  kUmaStabilityHistogramFlag = kUmaTargetedHistogramFlag | 0x2,

That way, code checking for just kUmaTargetedHistogramFlag will still include
these (without needing special handling), but this code can just pass in
kUmaTargetedHistogramFlag without ugly casts.

[Kibeom Kim (inactive), 2014/01/29 01:52:32]

Done. (nice suggestion!)

+          base::Histogram::kUmaTargetedHistogramFlag |
+          base::Histogram::kStabilityHistogramFlag));
 }
 
 void MetricsServiceBase::RecordDelta(