Simplify TranslateRanker API.

components/translate/core/browser/translate_ranker.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_H_
 #define COMPONENTS_TRANSLATE_CORE_BROWSER_TRANSLATE_RANKER_H_
 
 #include <memory>
 #include <string>
 #include <vector>
 
 #include "base/macros.h"
 #include "components/keyed_service/core/keyed_service.h"
 
 class GURL;
 
 namespace metrics {
 class TranslateEventProto;
 }  // namespace metrics
 
 namespace translate {
 
-class TranslatePrefs;
-
 // If enabled, downloads a translate ranker model and uses it to determine
 // whether the user should be given a translation prompt or not.
 class TranslateRanker : public KeyedService {
  public:
   TranslateRanker() = default;
 
   // Returns the version id for the ranker model.
   virtual uint32_t GetModelVersion() const = 0;
 
   // Returns true if executing the ranker model in the translation prompt
-  // context described by |translate_prefs|, |src_lang|, |dst_lang| and possibly
+  // context described by |translate_event| and possibly
   // other global browser context attributes suggests that the user should be
   // prompted as to whether translation should be performed.
-  // TODO(hamelphi): Take only the proto as input and extract features from it.
   virtual bool ShouldOfferTranslation(
-      const TranslatePrefs& translate_prefs,
-      const std::string& src_lang,
-      const std::string& dst_lang,
       metrics::TranslateEventProto* translate_event) = 0;
 
   // Transfers cached translate events to the given vector pointer and clears
   // the cache.
   virtual void FlushTranslateEvents(
       std::vector<metrics::TranslateEventProto>* events) = 0;
 
   // Record |translate_event| with the given |event_type| and |url|.
   // |event_type| must be one of the values defined by
   // metrics::TranslateEventProto::EventType.
@@ skipping 13 matching lines @@
       const GURL& url,
       metrics::TranslateEventProto* translate_event) = 0;
 
  private:
   DISALLOW_COPY_AND_ASSIGN(TranslateRanker);
 };
 
 }  // namespace translate
 
 #endif  // COMPONENTS_TRANSLATE_CORE_BROWSER_TRANSLATE_RANKER_H_