Send TranslateEventProtos to UMA.

chrome/browser/translate/chrome_translate_client.cc

 // 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.
 
 #include "chrome/browser/translate/chrome_translate_client.h"
 
 #include <vector>
 
 #include "base/logging.h"
 #include "base/path_service.h"
@@ skipping 17 matching lines @@
 #include "chrome/grit/theme_resources.h"
 #include "components/prefs/pref_service.h"
 #include "components/translate/core/browser/language_model.h"
 #include "components/translate/core/browser/language_state.h"
 #include "components/translate/core/browser/page_translated_details.h"
 #include "components/translate/core/browser/translate_accept_languages.h"
 #include "components/translate/core/browser/translate_download_manager.h"
 #include "components/translate/core/browser/translate_infobar_delegate.h"
 #include "components/translate/core/browser/translate_manager.h"
 #include "components/translate/core/browser/translate_prefs.h"
+#include "components/translate/core/browser/translate_ranker.h"
 #include "components/translate/core/common/language_detection_details.h"
 #include "components/variations/service/variations_service.h"
 #include "content/public/browser/notification_service.h"
 #include "content/public/browser/render_view_host.h"
 #include "content/public/browser/web_contents.h"
 #include "url/gurl.h"
 
 DEFINE_WEB_CONTENTS_USER_DATA_KEY(ChromeTranslateClient);
 
 ChromeTranslateClient::ChromeTranslateClient(content::WebContents* web_contents)
@@ skipping 112 matching lines @@
   if (!instance)
     return;
 
   instance->translate_driver().BindRequest(std::move(request));
 }
 
 translate::TranslateManager* ChromeTranslateClient::GetTranslateManager() {
   return translate_manager_.get();
 }
 
+// TODO(rogerm): This code can be moved into TranslateManager.

[Roger McFarlane (Chromium), 2016/10/12 18:18:28]

This can be removed. TranslateManager is is the intended client of this code,
but it can directly call
TranslterRanker::GetInstance()->RecordTranslateEvent(...), which already acts
based on IsLoggingEnabled().

[hamelphi, 2016/10/12 21:06:09]

Agreed. Removed.

+bool ChromeTranslateClient::RecordTranslateEvent(
+    const metrics::TranslateEventProto& translate_event) {
+  if (translate::TranslateRanker::IsLoggingEnabled()) {
+    translate::TranslateRanker* translate_ranker =
+        translate::TranslateRanker::GetInstance();
+    if (translate_ranker != nullptr) {
+      translate_ranker->RecordTranslateEvent(translate_event);
+      return true;
+    }
+  }
+  return false;
+}
+
+
 void ChromeTranslateClient::ShowTranslateUI(
     translate::TranslateStep step,
     const std::string& source_language,
     const std::string& target_language,
     translate::TranslateErrors::Type error_type,
     bool triggered_from_menu) {
   DCHECK(web_contents());
   if (error_type != translate::TranslateErrors::NONE)
     step = translate::TRANSLATE_STEP_TRANSLATE_ERROR;
 
@@ skipping 156 matching lines @@
     if (GetLanguageState().InTranslateNavigation())
       return;
   }
 
   TranslateBubbleFactory::Show(browser->window(), web_contents(), step,
                                error_type);
 #else
   NOTREACHED();
 #endif
 }