Implementation of translation event logging.

components/translate/core/common/translation_logging_helper.cc

Index: components/translate/core/common/translation_logging_helper.cc
diff --git a/components/translate/core/common/translation_logging_helper.cc b/components/translate/core/common/translation_logging_helper.cc
new file mode 100644
index 0000000000000000000000000000000000000000..950e155ca68020006031b2563112f2fef5c1a81e
--- /dev/null
+++ b/components/translate/core/common/translation_logging_helper.cc
@@ -0,0 +1,70 @@
+// 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 "components/translate/core/common/translation_logging_helper.h"
+
+#include "base/logging.h"
+#include "base/time/time.h"
+#include "components/metrics/proto/translate_event.pb.h"
+#include "components/sync/protocol/user_event_specifics.pb.h"
+
+namespace translate {
+namespace {
+using metrics::TranslateEventProto;
+}  // namespace
+
+bool ConstructTranslateEvent(const TranslateEventProto& translate_event,
+                             sync_pb::UserEventSpecifics* const specifics) {
+  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());
+  auto* const translation = specifics->mutable_translation();
+  translation->set_from_language_code(translate_event.source_language());
+  translation->set_to_language_code(translate_event.target_language());
+  switch (translate_event.event_type()) {
+    case TranslateEventProto::UNKNOWN:
+      translation->set_interaction(sync_pb::Translation::UNKNOWN);
+      break;
+    case TranslateEventProto::USER_ACCEPT:
+      if (translate_event.has_modified_source_language() ||
+          translate_event.has_modified_target_language()) {
+        // Special case, since we don't have event enum telling us it's actually
+        // modified by user, we do this by check whether this event has modified
+        // source or target language.
+        if (translate_event.has_modified_source_language()) {
+          translation->set_from_language_code(
+              translate_event.modified_source_language());
+        }
+        if (translate_event.has_modified_target_language()) {
+          translation->set_to_language_code(
+              translate_event.modified_target_language());
+        }
+        translation->set_interaction(sync_pb::Translation::MANUAL);

[napper, 2017/05/31 04:49:56]

Why do we need MANUAL? Don't you know it is manual if
!modified_source_language().empty() or !modified_target_language().empty()

[renjieliu1, 2017/05/31 05:22:32]

currently in our proto
(https://codereview.chromium.org/2911603002/patch/60001/70004) we don't have
modified_source_language and modified_target_language fields, do we need to
record that as well?

+      } else {
+        translation->set_interaction(sync_pb::Translation::ACCEPT);
+      }
+      break;
+    case TranslateEventProto::USER_DECLINE:
+      translation->set_interaction(sync_pb::Translation::DECLINE);
+      break;
+    case TranslateEventProto::USER_IGNORE:
+      translation->set_interaction(sync_pb::Translation::IGNORED);
+      break;
+    case TranslateEventProto::USER_DISMISS:
+      translation->set_interaction(sync_pb::Translation::DISMISSED);
+      break;
+    case TranslateEventProto::USER_REVERT:
+      translation->set_interaction(sync_pb::Translation::TRANSLATION_REVERTED);
+      break;
+    case TranslateEventProto::AUTOMATICALLY_TRANSLATED:
+      translation->set_interaction(sync_pb::Translation::AUTOMATIC_TRANSLATION);
+      break;
+    default:  // We don't care about other events.
+      return false;
+  }
+  return true;
+}
+}  // namespace translate