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

Unified Diff: components/translate/content/browser/language_model_observer.cc

Issue 2396783002: Add LanguageModel, a keyed service that collects language info from CLD. (Closed)
Patch Set: 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/content/browser/language_model_observer.cc
diff --git a/components/translate/content/browser/language_model_observer.cc b/components/translate/content/browser/language_model_observer.cc
new file mode 100644
index 0000000000000000000000000000000000000000..1c6e9f66b65b6ba81f50c68b782561e74b301cac
--- /dev/null
+++ b/components/translate/content/browser/language_model_observer.cc
@@ -0,0 +1,25 @@
+// Copyright 2016 The Chromium Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+#include "components/translate/content/browser/language_model_observer.h"
+
+#include "components/translate/core/browser/language_model.h"
+#include "components/translate/core/common/language_detection_details.h"
+
+namespace translate {
+
+LanguageModelObserver::LanguageModelObserver(
+ translate::LanguageModel* language_model)
+ : language_model_(language_model) {}
+
+LanguageModelObserver::~LanguageModelObserver() {}
+
+void LanguageModelObserver::OnLanguageDetermined(
+ const translate::LanguageDetectionDetails& details) {
+ // Inform the language model about the language detected by CLD.
+ if (details.is_cld_reliable)
+ language_model_->OnPageVisited(details.cld_language);
droger 2016/10/05 13:37:26 Actually you can probably simplify here: get rid o
jkrcal 2016/10/05 15:14:46 Done.
+}
+
+} // namespace translate

Powered by Google App Engine
This is Rietveld 408576698