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

Unified Diff: components/translate/core/browser/translate_ranker.cc

Issue 2400503002: [Translate] Integrate TranslateEventProto UMA logging into TranslateManager. (Closed)
Patch Set: review candidate Created 4 years, 2 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: components/translate/core/browser/translate_ranker.cc
diff --git a/components/translate/core/browser/translate_ranker.cc b/components/translate/core/browser/translate_ranker.cc
index 358675025ec78fbedfcdddd5dd152d3bb0367fef..3aad345cc44ece208c88c904ace6d11028cfb116 100644
--- a/components/translate/core/browser/translate_ranker.cc
+++ b/components/translate/core/browser/translate_ranker.cc
@@ -65,14 +65,6 @@ GURL GetTranslateRankerURL() {
: kTranslateRankerModelURL);
}
-bool IsQueryEnabled() {
- return base::FeatureList::IsEnabled(kTranslateRankerQuery);
-}
-
-bool IsEnforcementEnabled() {
- return base::FeatureList::IsEnabled(kTranslateRankerEnforcement);
-}
-
void ReportModelStatus(ModelStatus model_status) {
UMA_HISTOGRAM_ENUMERATION("Translate.Ranker.Model.Status", model_status,
MODEL_STATUS_MAX);
@@ -94,6 +86,16 @@ bool TranslateRanker::IsEnabled() {
}
// static
+bool TranslateRanker::IsQueryEnabled() {
+ return base::FeatureList::IsEnabled(kTranslateRankerQuery);
+}
+
+// static
+bool TranslateRanker::IsEnforcementEnabled() {
+ return base::FeatureList::IsEnabled(kTranslateRankerEnforcement);
+}
+
+// static
TranslateRanker* TranslateRanker::GetInstance() {
return base::Singleton<TranslateRanker>::get();
}

Powered by Google App Engine
This is Rietveld 408576698