[SyncFS] Add a UMA to measure RTT in registration

chrome/browser/sync_file_system/drive_backend/sync_engine.cc

 // Copyright 2013 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.
 
 #include "chrome/browser/sync_file_system/drive_backend/sync_engine.h"
 
 #include <vector>
 
 #include "base/bind.h"
+#include "base/metrics/histogram.h"
 #include "base/threading/sequenced_worker_pool.h"
+#include "base/time/time.h"
 #include "base/values.h"
 #include "chrome/browser/drive/drive_api_service.h"
 #include "chrome/browser/drive/drive_notification_manager.h"
 #include "chrome/browser/drive/drive_notification_manager_factory.h"
 #include "chrome/browser/drive/drive_service_interface.h"
 #include "chrome/browser/drive/drive_uploader.h"
 #include "chrome/browser/extensions/extension_service.h"
 #include "chrome/browser/profiles/profile.h"
 #include "chrome/browser/signin/profile_oauth2_token_service_factory.h"
 #include "chrome/browser/signin/signin_manager_factory.h"
@@ skipping 199 matching lines @@
 }
 
 void SyncEngine::RegisterOrigin(
     const GURL& origin, const SyncStatusCallback& callback) {
   worker_task_runner_->PostTask(
       FROM_HERE,
       base::Bind(&SyncWorker::RegisterOrigin,
                  base::Unretained(sync_worker_.get()),
                  origin,
                  RelayCallbackToCurrentThread(
-                     FROM_HERE, callback)));
+                     FROM_HERE,
+                     base::Bind(&SyncEngine::DidRegisterOrigin,
+                                base::Time::Now(),
+                                callback))));
 }
 
 void SyncEngine::EnableOrigin(
     const GURL& origin, const SyncStatusCallback& callback) {
   worker_task_runner_->PostTask(
       FROM_HERE,
       base::Bind(&SyncWorker::EnableOrigin,
                  base::Unretained(sync_worker_.get()),
                  origin,
                  RelayCallbackToCurrentThread(
@@ skipping 279 matching lines @@
     bool is_app_enabled = extension_service_->IsExtensionEnabled(app_id);
     bool is_app_root_tracker_enabled =
         tracker.tracker_kind() == TRACKER_KIND_APP_ROOT;
     if (is_app_enabled && !is_app_root_tracker_enabled)
       EnableOrigin(origin, base::Bind(&EmptyStatusCallback));
     else if (!is_app_enabled && is_app_root_tracker_enabled)
       DisableOrigin(origin, base::Bind(&EmptyStatusCallback));
   }
 }
 
+void SyncEngine::DidRegisterOrigin(
+    base::Time start_time,

[tzik, 2014/05/08 06:39:10]

const ref?

[peria, 2014/05/08 06:51:52]

Done.

+    const SyncStatusCallback& callback,
+    SyncStatusCode status) {
+  base::TimeDelta delta(base::Time::Now() - start_time);

[tzik, 2014/05/08 06:39:10]

s/Time/TimeTicks/?

[peria, 2014/05/08 06:51:52]

Done.

+  HISTOGRAM_TIMES("SyncFileSystem.RegisterOriginTime", delta);
+  callback.Run(status);
+}
+
 }  // namespace drive_backend
 }  // namespace sync_file_system