Send UMA recording state to TranslateRanker.

chrome/browser/translate/translate_ranker_metrics_provider.cc

 // Copyright 2017 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/translate_ranker_metrics_provider.h"
 
 #include "chrome/browser/browser_process.h"
 #include "chrome/browser/profiles/profile_manager.h"
 #include "chrome/browser/translate/translate_ranker_factory.h"
 #include "components/metrics/proto/chrome_user_metrics_extension.pb.h"
 #include "components/metrics/proto/translate_event.pb.h"
 #include "components/translate/core/browser/translate_ranker.h"
 
 namespace translate {
 
 void TranslateRankerMetricsProvider::ProvideGeneralMetrics(
     metrics::ChromeUserMetricsExtension* uma_proto) {
   std::vector<Profile*> loaded_profiles =
       g_browser_process->profile_manager()->GetLoadedProfiles();
   for (Profile* profile : loaded_profiles) {
     TranslateRanker* ranker =
         TranslateRankerFactory::GetForBrowserContext(profile);
     if (!ranker)
-      return;
+      continue;

[Roger McFarlane (Chromium), 2017/06/06 19:28:27]

doh! good catch!

[hamelphi, 2017/06/06 20:28:05]

Acknowledged.

 
     std::vector<metrics::TranslateEventProto> translate_events;
     ranker->FlushTranslateEvents(&translate_events);
 
     for (auto& event : translate_events) {
       uma_proto->add_translate_event()->Swap(&event);
     }
   }
 }
 
+void TranslateRankerMetricsProvider::SetLoggingState(bool logging_enabled) {
+  std::vector<Profile*> loaded_profiles =
+      g_browser_process->profile_manager()->GetLoadedProfiles();
+  for (Profile* profile : loaded_profiles) {
+    TranslateRanker* ranker =
+        TranslateRankerFactory::GetForBrowserContext(profile);
+    if (!ranker)
+      continue;
+    ranker->EnableLogging(logging_enabled);

[Roger McFarlane (Chromium), 2017/06/06 19:28:27]

optional:

if (ranker)
  ranker->EnableLogging(...);

[hamelphi, 2017/06/06 20:28:05]

Done.

+  }
+}
+
+void TranslateRankerMetricsProvider::OnRecordingEnabled() {
+  SetLoggingState(true);
+}
+
+void TranslateRankerMetricsProvider::OnRecordingDisabled() {
+  SetLoggingState(false);
+}
+
 }  // namespace translate