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

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

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.h
diff --git a/components/translate/content/browser/language_model_observer.h b/components/translate/content/browser/language_model_observer.h
new file mode 100644
index 0000000000000000000000000000000000000000..6e3da9023687eac3f5bedce0f12a3fe8aa72a424
--- /dev/null
+++ b/components/translate/content/browser/language_model_observer.h
@@ -0,0 +1,36 @@
+// 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.
+
+#ifndef COMPONENTS_TRANSLATE_CONTENT_BROWSER_LANGUAGE_MODEL_OBSERVER_H_
+#define COMPONENTS_TRANSLATE_CONTENT_BROWSER_LANGUAGE_MODEL_OBSERVER_H_
+
+#include <map>
+
+#include "base/macros.h"
+#include "components/translate/content/browser/content_translate_driver.h"
+
+namespace translate {
+
+struct LanguageDetectionDetails;
+class LanguageModel;
+
+// Routes the language info from ContentTranslateDriver to LanguageModel.
+class LanguageModelObserver : public ContentTranslateDriver::Observer {
+ public:
+ explicit LanguageModelObserver(translate::LanguageModel* language_model);
Bernhard Bauer 2016/10/05 14:48:16 Please explain the lifetime of |language_model|
jkrcal 2016/10/05 15:14:46 This class got removed.
+ ~LanguageModelObserver() override;
+
+ // Called when the page language has been determined.
+ void OnLanguageDetermined(
+ const translate::LanguageDetectionDetails& details) override;
+
+ private:
+ translate::LanguageModel* language_model_;
+
+ DISALLOW_COPY_AND_ASSIGN(LanguageModelObserver);
+};
+
+} // namespace translate
+
+#endif // COMPONENTS_TRANSLATE_CONTENT_BROWSER_LANGUAGE_MODEL_OBSERVER_H_

Powered by Google App Engine
This is Rietveld 408576698