Send TranslateEventProtos to UMA.

components/translate/core/browser/translate_ranker_unittest.cc

 // 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.
 
 #include "components/translate/core/browser/translate_ranker.h"
 
 #include <initializer_list>
 #include <memory>
 
 #include "base/feature_list.h"
 #include "base/strings/stringprintf.h"
 #include "base/test/scoped_feature_list.h"
+#include "components/metrics/proto/translate_event.pb.h"
 #include "components/pref_registry/testing_pref_service_syncable.h"
 #include "components/prefs/scoped_user_pref_update.h"
 #include "components/translate/core/browser/proto/translate_ranker_model.pb.h"
 #include "components/translate/core/browser/translate_download_manager.h"
 #include "components/translate/core/browser/translate_prefs.h"
 #include "testing/gtest/include/gtest/gtest.h"
 
 namespace translate {
 
 class TranslateRankerTest : public ::testing::Test {
@@ skipping 50 matching lines @@
     auto& locale_weight = *details->mutable_locale_weight();
     locale_weight["en-us"] = 0.10f;
     locale_weight["en-ca"] = 0.11f;
     locale_weight["zh-cn"] = 0.12f;  // Normalized to lowercase.
 
     return TranslateRanker::CreateForTesting(model.SerializeAsString());
   }
 
   static double Sigmoid(double x) { return 1.0 / (1.0 + exp(-x)); }
 
+  static metrics::TranslateEventProto CreateTranslateEvent(
+      const std::string& src_lang,
+      const std::string& dst_lang,
+      int accept_count,
+      int decline_count,
+      int ignore_count) {
+    metrics::TranslateEventProto translate_event;
+    translate_event.set_source_language(src_lang);
+    translate_event.set_target_language(dst_lang);
+    translate_event.set_accept_count(accept_count);
+    translate_event.set_decline_count(decline_count);
+    translate_event.set_ignore_count(ignore_count);
+    return translate_event;
+  }
+
   static const char* const kPreferredLanguagePrefs;
 
   std::unique_ptr<user_prefs::TestingPrefServiceSyncable> prefs_;
   std::unique_ptr<translate::TranslatePrefs> translate_prefs_;
 
  private:
   // Manages the enabling/disabling of features within the scope of a test.
   base::test::ScopedFeatureList scoped_feature_list_;
 
   // Cache and reset the application locale for each test.
@@ skipping 56 matching lines @@
 TEST_F(TranslateRankerTest, ShouldOfferTranslation) {
   InitFeatures({kTranslateRankerQuery, kTranslateRankerEnforcement}, {});
   // With a bias of -0.5 en->fr is not over the threshold.
   EXPECT_FALSE(GetRankerForTest(-0.5f)->ShouldOfferTranslation(
       *translate_prefs_, "en", "fr"));
   // With a bias of 0.25 en-fr is over the threshold.
   EXPECT_TRUE(GetRankerForTest(0.25f)->ShouldOfferTranslation(*translate_prefs_,
                                                               "en", "fr"));
 }
 
+TEST_F(TranslateRankerTest, RecordAndFlushEvents) {
+  InitFeatures({kTranslateRankerQuery, kTranslateRankerEnforcement}, {});
+  std::unique_ptr<translate::TranslateRanker> ranker = GetRankerForTest(0.0f);
+  std::vector<metrics::TranslateEventProto> flushed_events;
+
+  // Check that flushing an empty cache will return an empty vector.
+  ranker->FlushTranslateEvents(&flushed_events);
+  EXPECT_EQ(0U, flushed_events.size());
+
+  auto event_1 = CreateTranslateEvent("fr", "en", 1, 0, 3);
+  auto event_2 = CreateTranslateEvent("jp", "en", 2, 0, 3);
+  auto event_3 = CreateTranslateEvent("es", "de", 4, 5, 6);
+  ranker->RecordTranslateEvent(event_1);
+  ranker->RecordTranslateEvent(event_2);
+  ranker->RecordTranslateEvent(event_3);
+
+  // Capture the data and verify that it is as expected.
+  ranker->FlushTranslateEvents(&flushed_events);
+  EXPECT_EQ(3U, flushed_events.size());
+  ASSERT_EQ("fr", flushed_events[0].source_language());
+  ASSERT_EQ("jp", flushed_events[1].source_language());
+  ASSERT_EQ("es", flushed_events[2].source_language());
+
+  // Check that the cache has been cleared.
+  ranker->FlushTranslateEvents(&flushed_events);
+  EXPECT_EQ(0U, flushed_events.size());
+}
+

[Alexei Svitkine (slow), 2016/10/12 15:21:58]

Nit: Remove extra empty line.

[hamelphi, 2016/10/12 16:17:21]

Done.

+
 }  // namespace translate