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

Unified Diff: chrome/renderer/translate/translate_helper_metrics.h

Issue 15728002: Translate: adopt html lang attribute if valid value is provided (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: fix WindowOpenClose crash 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/renderer/translate/translate_helper_metrics.h
diff --git a/chrome/renderer/translate/translate_helper_metrics.h b/chrome/renderer/translate/translate_helper_metrics.h
index a7fb55ebaedad690a584cae3efad9dee8c451799..74caf38c9acd94025bf676e7c5758102b22943b5 100644
--- a/chrome/renderer/translate/translate_helper_metrics.h
+++ b/chrome/renderer/translate/translate_helper_metrics.h
@@ -16,6 +16,7 @@ namespace TranslateHelperMetrics {
enum MetricsNameIndex {
UMA_LANGUAGE_DETECTION,
UMA_CONTENT_LANGUAGE,
+ UMA_HTML_LANG,
UMA_LANGUAGE_VERIFICATION,
UMA_TIME_TO_BE_READY,
UMA_TIME_TO_LOAD,
@@ -27,11 +28,11 @@ enum MetricsNameIndex {
// A page may provide a Content-Language HTTP header or a META tag.
// TranslateHelper checks if a server provides a valid Content-Language.
-enum ContentLanguageType {
- CONTENT_LANGUAGE_NOT_PROVIDED,
- CONTENT_LANGUAGE_VALID,
- CONTENT_LANGUAGE_INVALID,
- CONTENT_LANGUAGE_MAX,
+enum LanguageCheckType {
+ LANGUAGE_NOT_PROVIDED,
+ LANGUAGE_VALID,
+ LANGUAGE_INVALID,
+ LANGUAGE_MAX,
};
// When a valid Content-Language is provided, TranslateHelper checks if a
@@ -54,12 +55,19 @@ enum SchemeType {
};
// Called after TranslateHelper verifies a server providing Content-Language
-// header. |provided_code| contains a Content-Language header value which
+// header. |provided_code| contains a Content-Language header value which a
// server provides. It can be empty string when a server doesn't provide it.
// |revised_code| is a value modified by format error corrector.
void ReportContentLanguage(const std::string& provided_code,
const std::string& revised_code);
+// Called after TranslateHelper verifies a page providing html lang attribute.
+// |provided_code| contains a html lang attribute which a page provides. It can
+// be empty string when a page doesn't provide it. |revised_code| is a value
+// modified by format error corrector.
+void ReportHtmlLang(const std::string& provided_code,
+ const std::string& revised_code);
+
// Called when CLD verifies Content-Language header.
void ReportLanguageVerification(LanguageVerificationType type);
« no previous file with comments | « chrome/renderer/translate/translate_helper.cc ('k') | chrome/renderer/translate/translate_helper_metrics.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698