Separate ARC++ Provisioning Histogram into managed and unmanaged

chrome/browser/chromeos/arc/arc_optin_uma.cc

Index: chrome/browser/chromeos/arc/arc_optin_uma.cc
diff --git a/chrome/browser/chromeos/arc/arc_optin_uma.cc b/chrome/browser/chromeos/arc/arc_optin_uma.cc
index dd92ecbfc4dd6d69d6a553e23a7c6127142104f3..92ad2c1cddd5901045bfb951c604a912cd5443cc 100644
--- a/chrome/browser/chromeos/arc/arc_optin_uma.cc
+++ b/chrome/browser/chromeos/arc/arc_optin_uma.cc
@@ -22,9 +22,16 @@ void UpdateEnabledStateUMA(bool enabled) {
   UMA_HISTOGRAM_BOOLEAN("Arc.State", enabled);
 }
 
-void UpdateProvisioningResultUMA(ProvisioningResult result) {
-  UMA_HISTOGRAM_ENUMERATION("Arc.Provisioning.Result", static_cast<int>(result),
-                            static_cast<int>(ProvisioningResult::SIZE));
+void UpdateProvisioningResultUMA(ProvisioningResult result, bool managed) {
+  if (managed) {
+    UMA_HISTOGRAM_ENUMERATION("Arc.Provisioning.Result.Managed",

[Luis Héctor Chávez, 2016/09/15 16:00:32]

Can you follow the convention in L40 and build the histogram name?

[phweiss, 2016/09/15 16:55:51]

Yes, but then we cannot use the macro anymore. I'm not sure which version I like
more, since the macro is easier to read, but it doubles the code. In L40, the
non-macro code replaces 4 different histograms, so the decision was more
clear-cut.

[Luis Héctor Chávez, 2016/09/15 17:22:03]

oh that expansion is also gnarlier than the one in UpdateProvisioningTiming :/
I'm not sure now either, so if you feel more comfortable with the previous one,
you can go back.

+                              static_cast<int>(result),
+                              static_cast<int>(ProvisioningResult::SIZE));
+  } else {
+    UMA_HISTOGRAM_ENUMERATION("Arc.Provisioning.Result.Unmanaged",
+                              static_cast<int>(result),
+                              static_cast<int>(ProvisioningResult::SIZE));
+  }
 }
 
 void UpdateProvisioningTiming(const base::TimeDelta& elapsed_time,