Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1437)

Unified Diff: chrome/browser/sync/glue/sync_backend_host_impl.cc

Issue 925673003: Add profiling for FinishConfigureDataTypesOnFrontendLoop (sync). (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Comments from vadimt. Created 5 years, 10 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/sync/glue/sync_backend_host_impl.cc
diff --git a/chrome/browser/sync/glue/sync_backend_host_impl.cc b/chrome/browser/sync/glue/sync_backend_host_impl.cc
index 4114a32fce8218f95f1fb38fbb1b3e1fe786b014..1455d172658c42aa86961de6b856bb87d571312f 100644
--- a/chrome/browser/sync/glue/sync_backend_host_impl.cc
+++ b/chrome/browser/sync/glue/sync_backend_host_impl.cc
@@ -609,6 +609,11 @@ void SyncBackendHostImpl::FinishConfigureDataTypesOnFrontendLoop(
const syncer::ModelTypeSet failed_configuration_types,
const base::Callback<void(syncer::ModelTypeSet,
syncer::ModelTypeSet)>& ready_task) {
+ // TODO(erikchen): Remove ScopedTracker below once http://crbug.com/458406 is
+ // fixed.
+ tracked_objects::ScopedTracker tracking_profile1(
+ FROM_HERE_WITH_EXPLICIT_FUNCTION(
+ "458406 SyncBackendHostImpl::FinishConfigureDataTOFL"));
if (!frontend_)
return;
@@ -618,6 +623,11 @@ void SyncBackendHostImpl::FinishConfigureDataTypesOnFrontendLoop(
ModelTypeSetToObjectIdSet(enabled_types));
}
+ // TODO(erikchen): Remove ScopedTracker below once http://crbug.com/458406 is
+ // fixed.
+ tracked_objects::ScopedTracker tracking_profile2(
+ FROM_HERE_WITH_EXPLICIT_FUNCTION(
+ "458406 SyncBackendHostImpl::FinishConfigureDataTOFL::ReadyTask"));
if (!ready_task.is_null())
ready_task.Run(succeeded_configuration_types, failed_configuration_types);
}
« no previous file with comments | « no previous file | components/sync_driver/data_type_manager_impl.cc » ('j') | components/sync_driver/data_type_manager_impl.cc » ('J')

Powered by Google App Engine
This is Rietveld 408576698