Simplify TranslateRanker API.

components/translate/core/browser/translate_ranker_impl.h

 // 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_CORE_BROWSER_TRANSLATE_RANKER_IMPL_H_
 #define COMPONENTS_TRANSLATE_CORE_BROWSER_TRANSLATE_RANKER_IMPL_H_
 
 #include <memory>
 #include <string>
 
@@ skipping 16 matching lines @@
 namespace ukm {
 class UkmRecorder;
 }  // namespace ukm
 
 namespace metrics {
 class TranslateEventProto;
 }  // namespace metrics
 
 namespace translate {
 
-class TranslatePrefs;
-
 // Features used to enable ranker query, enforcement and logging. Note that
 // enabling enforcement implies (forces) enabling queries.
 extern const base::Feature kTranslateRankerQuery;
 extern const base::Feature kTranslateRankerEnforcement;
 extern const base::Feature kTranslateRankerDecisionOverride;
 
 struct TranslateRankerFeatures {
   TranslateRankerFeatures();
 
   TranslateRankerFeatures(int accepted,
                           int denied,
                           int ignored,
                           const std::string& src,
                           const std::string& dst,
                           const std::string& cntry,
                           const std::string& locale);
 
-  TranslateRankerFeatures(const TranslatePrefs& prefs,
-                          const std::string& src,
-                          const std::string& dst,
-                          const std::string& locale);
+  TranslateRankerFeatures(const metrics::TranslateEventProto& tep);
 
   ~TranslateRankerFeatures();
 
   void WriteTo(std::ostream& stream) const;
 
   // Input value used to generate the features.
   int accepted_count;
   int denied_count;
   int ignored_count;
   int total_count;
@@ skipping 24 matching lines @@
   // Get the URL from which the download the translate ranker model, by default
   // from Finch.
   static GURL GetModelURL();
 
   // Override the default enabled/disabled state of translate event logging.
   void EnableLogging(bool value);
 
   // TranslateRanker...
   uint32_t GetModelVersion() const override;
   bool ShouldOfferTranslation(
-      const TranslatePrefs& translate_prefs,
-      const std::string& src_lang,
-      const std::string& dst_lang,
       metrics::TranslateEventProto* translate_event) override;
   void FlushTranslateEvents(
       std::vector<metrics::TranslateEventProto>* events) override;
   void RecordTranslateEvent(
       int event_type,
       const GURL& url,
       metrics::TranslateEventProto* translate_event) override;
   bool ShouldOverrideDecision(
       int event_type,
       const GURL& url,
       metrics::TranslateEventProto* translate_event) override;
 
   void OnModelAvailable(
       std::unique_ptr<chrome_intelligence::RankerModel> model);
 
   // Get the model decision on whether we should show the translate
-  // UI or not given |features|.
-  bool GetModelDecision(const TranslateRankerFeatures& features);
+  // UI or not given |translate_event|.
+  bool GetModelDecision(const metrics::TranslateEventProto& translate_event);
 
   // Check if the ModelLoader has been initialized. Used to test ModelLoader
   // logic.
   bool CheckModelLoaderForTesting();
 
  private:
   void SendEventToUKM(const metrics::TranslateEventProto& translate_event,
                       const GURL& url);
 
   // Caches the translate event.
@@ skipping 35 matching lines @@
 
   DISALLOW_COPY_AND_ASSIGN(TranslateRankerImpl);
 };
 
 }  // namespace translate
 
 std::ostream& operator<<(std::ostream& stream,
                          const translate::TranslateRankerFeatures& features);
 
 #endif  // COMPONENTS_TRANSLATE_CORE_BROWSER_TRANSLATE_RANKER_IMPL_H_