Allow TranslateRanker to override decisions taken by heuristics.

components/translate/core/browser/translate_manager.h

 // Copyright 2014 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_MANAGER_H_
 #define COMPONENTS_TRANSLATE_CORE_BROWSER_TRANSLATE_MANAGER_H_
 
 #include <map>
 #include <memory>
 #include <string>
@@ skipping 106 matching lines @@
 
   // Record an event of the given |event_type| using the currently saved
   // |translate_event_| as context. |event_type| must be one of the values
   // defined by metrics::TranslateEventProto::EventType.
   void RecordTranslateEvent(int event_type);
 
   // By default, don't offer to translate in builds lacking an API key. For
   // testing, set to true to offer anyway.
   static void SetIgnoreMissingKeyForTesting(bool ignore);
 
+  // Returns true if the decision should be overridden and logs the event
+  // appropriately. |event_type| must be one of the
+  // values defined by metrics::TranslateEventProto::EventType.
+  bool ShouldOverrideDecision(int event_type);
+
+  // Returns true if the BubbleUI should be suppressed.
+  bool ShouldSuppressBubbleUI(bool triggered_from_menu,
+                              const std::string& source_language);

[Roger McFarlane (Chromium), 2017/04/19 16:16:16]

why is this bubble ui specific?

[hamelphi, 2017/04/19 18:19:48]

I don't know the reasons why this logic is only applied to BubbleUI. Groby@
might have more insights on this?

+
  private:
   friend class translate::testing::TranslateManagerTest;
 
   // Sends a translation request to the TranslateDriver.
   void DoTranslatePage(const std::string& translate_script,
                        const std::string& source_lang,
                        const std::string& target_lang);
 
   // Returns the language to translate to by looking at ULP. Return empty string
   // If it cannot conclude from ULP.
@@ skipping 37 matching lines @@
   // By default, don't offer to translate in builds lacking an API key. For
   // testing, set to true to offer anyway.
   static bool ignore_missing_key_for_testing_;
 
   DISALLOW_COPY_AND_ASSIGN(TranslateManager);
 };
 
 }  // namespace translate
 
 #endif  // COMPONENTS_TRANSLATE_CORE_BROWSER_TRANSLATE_MANAGER_H_