[SyncFS] Add UMAs to track syncfs usage

[SyncFS] Add UMAs to track syncfs usage

BUG=347424
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=255678

[tzik, 2014-03-06 07:29:02]

PTL

[kinuko, 2014-03-06 12:14:54]

lgtm

[tzik, 2014-03-06 14:44:55]

The CQ bit was checked by tzik@chromium.org

[tzik, 2014-03-06 14:45:11]

The CQ bit was unchecked by tzik@chromium.org

[commit-bot: I haz the power, 2014-03-06 14:45:16]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/tzik@chromium.org/188293005/20001

[commit-bot: I haz the power, 2014-03-06 14:45:21]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2014-03-06 14:45:22]

Failed to apply patch for
chrome/browser/sync_file_system/sync_file_system_service.cc:
While running patch -p1 --forward --force --no-backup-if-mismatch;
  patching file chrome/browser/sync_file_system/sync_file_system_service.cc
  Hunk #2 FAILED at 389.
  Hunk #3 succeeded at 476 (offset 1 line).
  1 out of 3 hunks FAILED -- saving rejects to file
chrome/browser/sync_file_system/sync_file_system_service.cc.rej

Patch:       chrome/browser/sync_file_system/sync_file_system_service.cc
Index: chrome/browser/sync_file_system/sync_file_system_service.cc
diff --git a/chrome/browser/sync_file_system/sync_file_system_service.cc
b/chrome/browser/sync_file_system/sync_file_system_service.cc
index
a3ef87d84f91232aeb7fbcc276e8df1a8626b2a1..bc1b1adcff5aceb2491f70b88e0ef71e65d7048a
100644
--- a/chrome/browser/sync_file_system/sync_file_system_service.cc
+++ b/chrome/browser/sync_file_system/sync_file_system_service.cc
@@ -10,6 +10,7 @@
 #include "base/format_macros.h"
 #include "base/logging.h"
 #include "base/memory/ref_counted.h"
+#include "base/metrics/histogram.h"
 #include "base/stl_util.h"
 #include "chrome/browser/chrome_notification_types.h"
 #include
"chrome/browser/extensions/api/sync_file_system/extension_sync_event_observer.h"
@@ -388,6 +389,8 @@ SyncFileSystemService::GetConflictResolutionPolicy() const {
 
 SyncStatusCode SyncFileSystemService::SetConflictResolutionPolicy(
     ConflictResolutionPolicy policy) {
+  UMA_HISTOGRAM_ENUMERATION("SyncFileSystem.ConflictResolutionPolicy",
+                            policy, CONFLICT_RESOLUTION_POLICY_MAX);
   return remote_service_->SetConflictResolutionPolicy(policy);
 }
 
@@ -474,6 +477,10 @@ void SyncFileSystemService::DidRegisterOrigin(
             app_origin.spec().c_str(),
             SyncStatusCodeToString(status));
 
+  UMA_HISTOGRAM_ENUMERATION("SyncFileSystem.RegisterOriginResult",
+                            GetRemoteService(app_origin)->GetCurrentState(),
+                            REMOTE_SERVICE_STATE_MAX);
+
   if (status == SYNC_STATUS_FAILED) {
     // If we got generic error return the service status information.
     switch (GetRemoteService(app_origin)->GetCurrentState()) {

[tzik, 2014-03-07 09:34:45]

The CQ bit was checked by tzik@chromium.org

[commit-bot: I haz the power, 2014-03-07 09:37:03]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/tzik@chromium.org/188293005/80001

[commit-bot: I haz the power, 2014-03-07 09:51:23]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2014-03-07 09:51:23]

Try jobs failed on following builders: linux_chromium_clang_dbg

[tzik, 2014-03-07 10:46:32]

The CQ bit was checked by tzik@chromium.org

[commit-bot: I haz the power, 2014-03-07 10:48:30]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/tzik@chromium.org/188293005/100001

[commit-bot: I haz the power, 2014-03-07 10:52:07]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2014-03-07 10:52:08]

Try jobs failed on following builders: mac_chromium_compile_dbg

[tzik, 2014-03-07 13:13:39]

The CQ bit was checked by tzik@chromium.org

[commit-bot: I haz the power, 2014-03-07 13:15:22]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/tzik@chromium.org/188293005/100001

[commit-bot: I haz the power, 2014-03-07 19:18:38]

Change committed as 255678