[translate] Fix enforcement vs logging for TranslateRanker.

components/translate/core/browser/translate_ranker.cc

Index: components/translate/core/browser/translate_ranker.cc
diff --git a/components/translate/core/browser/translate_ranker.cc b/components/translate/core/browser/translate_ranker.cc
index 0ebe722719ffbd4761a602406deb847308d69ebf..fe2a214ee397936d1210379f595bfeac9c99b616 100644
--- a/components/translate/core/browser/translate_ranker.cc
+++ b/components/translate/core/browser/translate_ranker.cc
@@ -180,9 +180,7 @@ bool TranslateRanker::ShouldOfferTranslation(
 
   UMA_HISTOGRAM_BOOLEAN("Translate.Ranker.QueryResult", result);
 
-  // If enforcement is enabled, return the real result; otherwise, return the
-  // default.
-  return IsEnforcementEnabled() ? result : kDefaultResponse;

[Roger McFarlane (Chromium), 2016/11/29 18:43:30]

The TranslateManager code where this is enforced is at
https://cs.chromium.org/chromium/src/components/translate/core/browser/transl...

[groby1, 2016/11/29 19:03:43]

The behavior in translate manager is subtly different, in that the ranker
response is always logged as the actual result, even when enforcement is
disabled. Intentionally, or bug?

[hamelphi, 2016/11/29 19:37:06]

It is intentional. We want to log the response the ranker would have given even
if we don't enforce it. See b/33176772.

[Roger McFarlane (Chromium), 2016/11/29 20:01:00]

Precisely.  :)

+  return result;
 }
 
 TranslateRanker::TranslateRanker() {}