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

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

Issue 2916873004: Quick fix for logging. (Closed)
Patch Set: updates 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.cc
diff --git a/components/translate/core/common/language_detection_logging_helper.cc b/components/translate/core/common/language_detection_logging_helper.cc
index e632bdfb84da281ae84b92a39e4d778177fb5305..b98ae8549c6769d7f21e73d65367ec1c4f5a74d5 100644
--- a/components/translate/core/common/language_detection_logging_helper.cc
+++ b/components/translate/core/common/language_detection_logging_helper.cc
@@ -14,13 +14,14 @@
namespace translate {
std::unique_ptr<sync_pb::UserEventSpecifics> ConstructLanguageDetectionEvent(
+ const int64_t navigation_id,
const LanguageDetectionDetails& details) {
auto specifics = base::MakeUnique<sync_pb::UserEventSpecifics>();
specifics->set_event_time_usec(base::Time::Now().ToInternalValue());
// TODO(renjieliu): Revisit this field when the best way to identify
// navigations is determined.
- specifics->set_navigation_id(base::Time::Now().ToInternalValue());
+ specifics->set_navigation_id(navigation_id);
sync_pb::LanguageDetection lang_detection;
auto* const lang = lang_detection.add_detected_languages();
@@ -33,4 +34,5 @@ std::unique_ptr<sync_pb::UserEventSpecifics> ConstructLanguageDetectionEvent(
*specifics->mutable_language_detection() = lang_detection;
return specifics;
}
+
} // namespace translate

Powered by Google App Engine
This is Rietveld 408576698