Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(748)

Unified Diff: components/translate/core/common/language_detection_logging_helper_unittest.cc

Issue 2942703002: [Sync] Make children messages of UserEventSpecifics' consistent use event suffix. (Closed)
Patch Set: Rebase Created 3 years, 6 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: components/translate/core/common/language_detection_logging_helper_unittest.cc
diff --git a/components/translate/core/common/language_detection_logging_helper_unittest.cc b/components/translate/core/common/language_detection_logging_helper_unittest.cc
index c22a0ee0eabdd042f32940336442bdbc3d0b1346..fc9c7ebae0a333a7c9f1db77241ad47e3c228d27 100644
--- a/components/translate/core/common/language_detection_logging_helper_unittest.cc
+++ b/components/translate/core/common/language_detection_logging_helper_unittest.cc
@@ -29,7 +29,7 @@ TEST(LanguageDetectionLoggingHelperTest, ConstructUserEventSpecifics) {
ConstructLanguageDetectionEvent(navigation_id, details);
// Expect the navigation id is correctly set.
EXPECT_EQ(user_event->navigation_id(), navigation_id);
- EXPECT_EQ(user_event->language_detection().SerializeAsString(),
+ EXPECT_EQ(user_event->language_detection_event().SerializeAsString(),
lang_detection.SerializeAsString());
}
@@ -49,8 +49,8 @@ TEST(LanguageDetectionLoggingHelperTest, DontSetAdoptedLanguage) {
ConstructLanguageDetectionEvent(100, details);
// Expect the navigation id is correctly set.
EXPECT_EQ(user_event->navigation_id(), 100);
- EXPECT_EQ(user_event->language_detection().SerializeAsString(),
+ EXPECT_EQ(user_event->language_detection_event().SerializeAsString(),
lang_detection.SerializeAsString());
}
-} // namespace translate
+} // namespace translate

Powered by Google App Engine
This is Rietveld 408576698