Measuring data use of different ModelTypes in Sync Service.

sync/util/data_type_histogram.cc

+// Copyright (c) 2015 The Chromium Authors. All rights reserved.

[Alexei Svitkine (slow), 2015/08/24 21:27:00]

Nit: No (c) for new comments.

[amohammadkhan, 2015/08/24 22:52:40]

Done.

+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+#include "sync/util/data_type_histogram.h"
+
+#include <string>

[Alexei Svitkine (slow), 2015/08/24 21:27:00]

This include should be in the header.

[amohammadkhan, 2015/08/24 22:52:40]

Done.

+
+#include "base/metrics/sparse_histogram.h"
+
+void SyncRecordDatatypeBin(std::string name, int sample, int value) {
+    base::HistogramBase* histogram = base::SparseHistogram::FactoryGet(

[Alexei Svitkine (slow), 2015/08/24 21:27:00]

This is indented too much.

[amohammadkhan, 2015/08/24 22:52:40]

Done.

+        name, base::HistogramBase::kUmaTargetedHistogramFlag);
+    histogram->AddCount(sample, value);
+}