UMA metrics - number of accounts per profile and account reconciler.

components/signin/core/browser/signin_metrics.h

Index: components/signin/core/browser/signin_metrics.h
diff --git a/components/signin/core/browser/signin_metrics.h b/components/signin/core/browser/signin_metrics.h
new file mode 100644
index 0000000000000000000000000000000000000000..fc6976266ed2ca658acc1a124591643c1bb89878
--- /dev/null
+++ b/components/signin/core/browser/signin_metrics.h
@@ -0,0 +1,25 @@
+// Copyright (c) 2014 The Chromium Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+#ifndef COMPONENTS_SIGNIN_CORE_BROWSER_SIGNIN_METRICS_H_
+#define COMPONENTS_SIGNIN_CORE_BROWSER_SIGNIN_METRICS_H_
+
+#include <stddef.h>
+#include <string>
+
+#include "base/basictypes.h"
+
+namespace base {
+class FilePath;
+}
+
+class SigninMetrics {
+ public:
+  static void LogSigninAccountReconciliation(int total_number_accounts,

[Roger Tawa OOO till Jul 10th, 2014/05/07 17:05:38]

Should have a comment describing what this function does and what the arguments
mean.  In header files its probably better to list each arg on its own line when
they don't fit the 80 char limit.

[Mike Lerman, 2014/05/07 18:49:37]

Done.

+        int count_added_to_cookie_jar, int count_added_to_token,
+        bool primary_accounts_same, bool is_first_reconcile);
+};

[Roger Tawa OOO till Jul 10th, 2014/05/07 17:05:38]

New code should not use classes that to define a scope for static functions. 
Better to wrap the function in a namespace like signin_metrics.

[Mike Lerman, 2014/05/07 18:49:37]

Done.

+
+
+#endif  // COMPONENTS_SIGNIN_CORE_BROWSER_SIGNIN_METRICS_H_