Initial proto for language detection.

components/sync/protocol/user_event_specifics.proto

Index: components/sync/protocol/user_event_specifics.proto
diff --git a/components/sync/protocol/user_event_specifics.proto b/components/sync/protocol/user_event_specifics.proto
index fa74e61191c68b9baa92b54ece3a38c978069253..4cb45a2502cdb69ca14a2606253856646ce1f62c 100644
--- a/components/sync/protocol/user_event_specifics.proto
+++ b/components/sync/protocol/user_event_specifics.proto
@@ -13,6 +13,18 @@ option optimize_for = LITE_RUNTIME;
 
 package sync_pb;
 
+// Language detection output

[napper, 2017/05/18 04:55:06]

Add . to comment

[renjieliu1, 2017/05/18 05:48:21]

Done.

+message LanguageDetection {
+  message Language {
+    optional string language_code = 1;

[napper, 2017/05/18 04:55:06]

Can you add a comment as to what language code format is used?

[renjieliu1, 2017/05/18 05:48:21]

Done.

+    optional int32 percent = 2;

[napper, 2017/05/18 04:55:06]

int8?

[renjieliu1, 2017/05/18 05:48:21]

int32 is the minimum we can get for proto buffer :(

+  }

[napper, 2017/05/18 04:55:06]

Add a comment describing what percent is. Also percent doesnt seem like a very
clear name.

[renjieliu1, 2017/05/18 05:48:21]

how about proportion?

+  // Top n languages. Typically we just log the top one language, but for

[napper, 2017/05/18 04:55:07]

Change to "Typically we just log the top language"

[renjieliu1, 2017/05/18 05:48:21]

Done.

+  // page that we're not confident, we may log up to 3 top languages.

[napper, 2017/05/18 04:55:06]

"a page we are not confidence about,"

[renjieliu1, 2017/05/18 05:48:21]

Done.

+  repeated Language languages = 1;

[napper, 2017/05/18 04:55:06]

Would detected_languages be a better name here?

[renjieliu1, 2017/05/18 05:48:21]

sounds good :)

+  optional string adopted_language = 2;
+}
+
 message FieldTrialEvent {
   message FieldTrial {
     optional fixed32 name_id = 1;
@@ -32,5 +44,8 @@ message UserEventSpecifics {
   // other events belonged to.
   optional fixed64 session_id = 3;
 
-  oneof event { FieldTrialEvent field_trial_event = 4; }
+  oneof event {
+    FieldTrialEvent field_trial_event = 4;
+    LanguageDetection translate_language_detection = 5;
+  }
 }