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

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

Issue 2400503002: [Translate] Integrate TranslateEventProto UMA logging into TranslateManager. (Closed)
Patch Set: hamelphi@'s comments 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 5d6b6b03992ade9e5a7e30e01d5a9dce6226b48b..b055721eb325db1e82d15aa549066796af7892df 100644
--- a/components/translate/core/browser/translate_ranker.cc
+++ b/components/translate/core/browser/translate_ranker.cc
@@ -66,14 +66,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);
@@ -103,6 +95,16 @@ bool TranslateRanker::IsLoggingEnabled() {
}
// 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