Measuring data use of different ModelTypes in Sync Service.

sync/util/data_type_histogram.h

Index: sync/util/data_type_histogram.h
diff --git a/sync/util/data_type_histogram.h b/sync/util/data_type_histogram.h
index 0f7e6a99e594fb5d678ae8a4d4a3a875a3dc3584..bd8c202303a0ed1472fcdd58b91b1d644d64e3a3 100644
--- a/sync/util/data_type_histogram.h
+++ b/sync/util/data_type_histogram.h
@@ -6,9 +6,19 @@
 #define SYNC_UTIL_DATA_TYPE_HISTOGRAM_H_
 
 #include "base/metrics/histogram.h"
+#include "base/metrics/sparse_histogram.h"
 #include "base/time/time.h"
 #include "sync/internal_api/public/base/model_type.h"
 
+// This macro adds |value| to |sample| bucket of histogram |name|. |value|
+// should be greater or equal to 1 and |name| can be variable.
+#define UMA_HISTOGRAM_SPARSE_SLOWLY_WITH_VALUE(name, sample, value)     \

[Nicolas Zea, 2015/08/18 21:55:18]

I think it would be better to prefix this with SYNC rather than UMA, since it's
not something defined within the normal UMA histograms. Maybe
SYNC_RECORD_DATATYPE_BIN or something like that. Also add a comment describing
how it's used for datatypes.

[amohammadkhan, 2015/08/19 17:33:36]

Done.

+  do {                                                                  \
+    base::HistogramBase* histogram = base::SparseHistogram::FactoryGet( \
+        name, base::HistogramBase::kUmaTargetedHistogramFlag);          \
+    histogram->AddCount(sample, value);                                 \
+  } while (0)
+
 // For now, this just implements UMA_HISTOGRAM_LONG_TIMES. This can be adjusted
 // if we feel the min, max, or bucket count amount are not appropriate.
 #define SYNC_FREQ_HISTOGRAM(name, time) UMA_HISTOGRAM_CUSTOM_TIMES( \