sync: Introduce classes for per-type counters

sync/internal_api/public/sessions/update_counters.h

Index: sync/internal_api/public/sessions/update_counters.h
diff --git a/sync/internal_api/public/sessions/update_counters.h b/sync/internal_api/public/sessions/update_counters.h
new file mode 100644
index 0000000000000000000000000000000000000000..22dbf55e02f07d037162cdf404dc8ce725a203d2
--- /dev/null
+++ b/sync/internal_api/public/sessions/update_counters.h
@@ -0,0 +1,38 @@
+// Copyright 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 SYNC_INTERNAL_API_PUBLIC_SESSIONS_UPDATE_COUNTERS_H_
+#define SYNC_INTERNAL_API_PUBLIC_SESSIONS_UPDATE_COUNTERS_H_
+
+#include "base/memory/scoped_ptr.h"
+#include "base/values.h"
+#include "sync/base/sync_export.h"
+
+namespace syncer {
+
+// A class to maintain counts related to the update requests and responses for
+// a particular sync type.
+class SYNC_EXPORT_PRIVATE UpdateCounters {
+ public:
+  UpdateCounters();
+  ~UpdateCounters();
+
+  scoped_ptr<base::DictionaryValue> ToValue() const;
+  std::string ToString() const;
+
+  int num_updates_received;

[tim (not reviewing), 2014/04/28 21:26:21]

Make this a struct if you want public data members.

[rlarocque, 2014/04/28 21:59:59]

Done.

+  int num_reflected_updates_received;
+  int num_tombstone_updates_received;
+
+  int num_updates_applied;
+  int num_hierarchy_conflict_application_failures;
+  int num_encryption_conflict_application_failures;
+
+  int num_server_overwrites;
+  int num_local_overwrites;
+};
+
+}  // namespace syncer
+
+#endif  // SYNC_INTERNAL_API_PUBLIC_SESSIONS_UPDATE_COUNTERS_H_