[NTP Snippets] Another round of clang-tidy cleanups

components/translate/core/browser/language_model_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/language_model.h"
 
 #include "components/prefs/testing_pref_service.h"
 #include "testing/gmock/include/gmock/gmock.h"
 #include "testing/gtest/include/gtest/gtest.h"
 
 using testing::ElementsAre;
 using testing::FloatEq;
 using testing::Gt;
 
 namespace {
 
 const char kLang1[] = "en";
 const char kLang2[] = "de";
 const char kLang3[] = "es";
 
 }  // namespace
 
 namespace translate {
 
+bool operator==(const LanguageModel::LanguageInfo& lhs,
+                const LanguageModel::LanguageInfo& rhs) {
+  return lhs.language_code == rhs.language_code;
+}
+
 TEST(LanguageModelTest, ListSorted) {
   TestingPrefServiceSimple prefs;
   LanguageModel::RegisterProfilePrefs(prefs.registry());
   LanguageModel model(&prefs);
 
   for (int i = 0; i < 50; i++) {
     model.OnPageVisited(kLang1);
     model.OnPageVisited(kLang1);
     model.OnPageVisited(kLang1);
     model.OnPageVisited(kLang2);
   }
 
+  // Note: LanguageInfo's operator== only checks the language code, not the
+  // frequency.
   EXPECT_THAT(model.GetTopLanguages(),
-              ElementsAre(LanguageModel::LanguageInfo{kLang1},
-                          LanguageModel::LanguageInfo{kLang2}));
+              ElementsAre(LanguageModel::LanguageInfo(kLang1, 0.0f),
+                          LanguageModel::LanguageInfo(kLang2, 0.0f)));
 }
 
 TEST(LanguageModelTest, ListSortedReversed) {
   TestingPrefServiceSimple prefs;
   LanguageModel::RegisterProfilePrefs(prefs.registry());
   LanguageModel model(&prefs);
 
   for (int i = 0; i < 50; i++) {
     model.OnPageVisited(kLang2);
     model.OnPageVisited(kLang1);
     model.OnPageVisited(kLang1);
     model.OnPageVisited(kLang1);
   }
 
+  // Note: LanguageInfo's operator== only checks the language code, not the
+  // frequency.
   EXPECT_THAT(model.GetTopLanguages(),
-              ElementsAre(LanguageModel::LanguageInfo{kLang1},
-                          LanguageModel::LanguageInfo{kLang2}));
+              ElementsAre(LanguageModel::LanguageInfo(kLang1, 0.0f),
+                          LanguageModel::LanguageInfo(kLang2, 0.0f)));
 }
 
 TEST(LanguageModelTest, RightFrequencies) {
   TestingPrefServiceSimple prefs;
   LanguageModel::RegisterProfilePrefs(prefs.registry());
   LanguageModel model(&prefs);
 
   for (int i = 0; i < 50; i++) {
     model.OnPageVisited(kLang1);
     model.OnPageVisited(kLang1);
@@ skipping 23 matching lines @@
 
   // Another 100 visits cause the cleanup (total > 1000).
   for (int i = 0; i < 100; i++)
     model.OnPageVisited(kLang1);
   // Lang 2 is removed from the model.
   EXPECT_THAT(model.GetTopLanguages(),
               ElementsAre(LanguageModel::LanguageInfo{kLang1, 1}));
 }
 
 }  // namespace translate