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

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

Issue 12820010: sync: Define histogram ints for model types (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Remove comment Created 7 years, 9 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/model_association_manager.cc
diff --git a/chrome/browser/sync/glue/model_association_manager.cc b/chrome/browser/sync/glue/model_association_manager.cc
index 31900fc716558371c7b05e054a6f9a094bd488e7..d5f1f7938a79f7d50fc42cc4fdd7f7810192b227 100644
--- a/chrome/browser/sync/glue/model_association_manager.cc
+++ b/chrome/browser/sync/glue/model_association_manager.cc
@@ -11,7 +11,7 @@
#include "base/logging.h"
#include "base/message_loop.h"
#include "base/metrics/histogram.h"
-
+#include "sync/internal_api/public/base/model_type.h"
using content::BrowserThread;
using syncer::ModelTypeSet;
@@ -300,7 +300,7 @@ void ModelAssociationManager::AppendToFailedDatatypesAndLogError(
LOG(ERROR) << "Failed to associate models for "
<< syncer::ModelTypeToString(error.type());
UMA_HISTOGRAM_ENUMERATION("Sync.ConfigureFailed",
- error.type(),
+ ModelTypeToHistogramInt(error.type()),
syncer::MODEL_TYPE_COUNT);
}

Powered by Google App Engine
This is Rietveld 408576698