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

Unified Diff: chrome/browser/translate/translate_browser_metrics.cc

Issue 15987004: Translate: Filter and record languages whose names Chrome doesn't show (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Made Translate.ServerReportedUnsupportedLanguage deprecated Created 7 years, 7 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/translate/translate_browser_metrics.cc
diff --git a/chrome/browser/translate/translate_manager_metrics.cc b/chrome/browser/translate/translate_browser_metrics.cc
similarity index 70%
rename from chrome/browser/translate/translate_manager_metrics.cc
rename to chrome/browser/translate/translate_browser_metrics.cc
index 0098c74a4f0a140cdced48d33318e0dab20b5ab1..e913df9a35d3daa90f925fc72de35f9586aaa930 100644
--- a/chrome/browser/translate/translate_manager_metrics.cc
+++ b/chrome/browser/translate/translate_browser_metrics.cc
@@ -2,7 +2,7 @@
// Use of this source code is governed by a BSD-style license that can be
// found in the LICENSE file.
-#include "chrome/browser/translate/translate_manager_metrics.h"
+#include "chrome/browser/translate/translate_browser_metrics.h"
#include <string>
@@ -19,34 +19,34 @@ const char kTranslateInitiationStatus[] =
"Translate.InitiationStatus";
const char kTranslateReportLanguageDetectionError[] =
"Translate.ReportLanguageDetectionError";
-const char kTranslateServerReportedUnsupportedLanguage[] =
- "Translate.ServerReportedUnsupportedLanguage";
const char kTranslateUnsupportedLanguageAtInitiation[] =
"Translate.UnsupportedLanguageAtInitiation";
+const char kTranslateUndisplayableLanguage[] =
+ "Translate.UndisplayableLanguage";
Ilya Sherman 2013/06/01 00:59:47 nit: Alphabetize?
hajimehoshi 2013/06/05 04:44:05 Done.
struct MetricsEntry {
- TranslateManagerMetrics::MetricsNameIndex index;
+ TranslateBrowserMetrics::MetricsNameIndex index;
const char* const name;
};
// This entry table should be updated when new UMA items are added.
const MetricsEntry kMetricsEntries[] = {
- { TranslateManagerMetrics::UMA_INITIATION_STATUS,
+ { TranslateBrowserMetrics::UMA_INITIATION_STATUS,
kTranslateInitiationStatus },
- { TranslateManagerMetrics::UMA_LANGUAGE_DETECTION_ERROR,
+ { TranslateBrowserMetrics::UMA_LANGUAGE_DETECTION_ERROR,
kTranslateReportLanguageDetectionError },
- { TranslateManagerMetrics::UMA_SERVER_REPORTED_UNSUPPORTED_LANGUAGE,
- kTranslateServerReportedUnsupportedLanguage },
- { TranslateManagerMetrics::UMA_UNSUPPORTED_LANGUAGE_AT_INITIATION,
+ { TranslateBrowserMetrics::UMA_UNSUPPORTED_LANGUAGE_AT_INITIATION,
kTranslateUnsupportedLanguageAtInitiation },
+ { TranslateBrowserMetrics::UMA_UNDISPLAYABLE_LANGUAGE,
+ kTranslateUndisplayableLanguage },
};
-COMPILE_ASSERT(arraysize(kMetricsEntries) == TranslateManagerMetrics::UMA_MAX,
+COMPILE_ASSERT(arraysize(kMetricsEntries) == TranslateBrowserMetrics::UMA_MAX,
arraysize_of_kMetricsEntries_should_be_UMA_MAX);
} // namespace
-namespace TranslateManagerMetrics {
+namespace TranslateBrowserMetrics {
void ReportInitiationStatus(InitiationStatusType type) {
UMA_HISTOGRAM_ENUMERATION(kTranslateInitiationStatus,
@@ -58,16 +58,18 @@ void ReportLanguageDetectionError() {
UMA_HISTOGRAM_COUNTS(kTranslateReportLanguageDetectionError, 1);
}
-void ReportUnsupportedLanguage() {
- UMA_HISTOGRAM_COUNTS(kTranslateServerReportedUnsupportedLanguage, 1);
-}
-
void ReportUnsupportedLanguageAtInitiation(const std::string& language) {
int language_code = LanguageUsageMetrics::ToLanguageCode(language);
UMA_HISTOGRAM_SPARSE_SLOWLY(kTranslateUnsupportedLanguageAtInitiation,
language_code);
}
+void ReportUndisplayableLanguage(const std::string& language) {
+ int language_code = LanguageUsageMetrics::ToLanguageCode(language);
+ UMA_HISTOGRAM_SPARSE_SLOWLY(kTranslateUndisplayableLanguage,
+ language_code);
+}
+
const char* GetMetricsName(MetricsNameIndex index) {
for (size_t i = 0; i < arraysize(kMetricsEntries); ++i) {
if (kMetricsEntries[i].index == index)
@@ -77,4 +79,4 @@ const char* GetMetricsName(MetricsNameIndex index) {
return NULL;
}
-} // namespace TranslateManagerMetrics
+} // namespace TranslateBrowserMetrics

Powered by Google App Engine
This is Rietveld 408576698