Add logging of host lookups made by TurnPort to the RtcEventLog.

webrtc/logging/rtc_event_log/rtc_event_log.proto

 syntax = "proto2";
 option optimize_for = LITE_RUNTIME;
 package webrtc.rtclog;
 
 enum MediaType {
   ANY = 0;
   AUDIO = 1;
   VIDEO = 2;
   DATA = 3;
 }
@@ skipping 22 matching lines @@
     AUDIO_PLAYOUT_EVENT = 5;
     LOSS_BASED_BWE_UPDATE = 6;
     DELAY_BASED_BWE_UPDATE = 7;
     VIDEO_RECEIVER_CONFIG_EVENT = 8;
     VIDEO_SENDER_CONFIG_EVENT = 9;
     AUDIO_RECEIVER_CONFIG_EVENT = 10;
     AUDIO_SENDER_CONFIG_EVENT = 11;
     AUDIO_NETWORK_ADAPTATION_EVENT = 16;
     BWE_PROBE_CLUSTER_CREATED_EVENT = 17;
     BWE_PROBE_RESULT_EVENT = 18;
+    HOST_LOOKUP_EVENT = 19;
   }
 
   // required - Indicates the type of this event
   optional EventType type = 2;
 
   oneof subtype {
     // required if type == RTP_EVENT
     RtpPacket rtp_packet = 3;
 
     // required if type == RTCP_EVENT
@@ skipping 21 matching lines @@
     AudioSendConfig audio_sender_config = 11;
 
     // required if type == AUDIO_NETWORK_ADAPTATION_EVENT
     AudioNetworkAdaptation audio_network_adaptation = 16;
 
     // required if type == BWE_PROBE_CLUSTER_CREATED_EVENT
     BweProbeCluster probe_cluster = 17;
 
     // required if type == BWE_PROBE_RESULT_EVENT
     BweProbeResult probe_result = 18;
+
+    // required if type == HOST_LOOKUP_EVENT
+    HostLookupResult host_lookup_result = 19;
   }
 }
 
 message RtpPacket {
   // required - True if the packet is incoming w.r.t. the user logging the data
   optional bool incoming = 1;
 
   optional MediaType type = 2 [deprecated = true];
 
   // required - The size of the packet including both payload and header.
@@ skipping 213 matching lines @@
     INVALID_SEND_RECEIVE_RATIO = 2;
     TIMEOUT = 3;
   }
 
   // required - The result of this probing attempt.
   optional ResultType result = 2;
 
   // optional - but required if result == SUCCESS. The resulting bitrate in bps.
   optional uint64 bitrate_bps = 3;
 }
+
+message HostLookupResult {
+  // optional - hostname.
+  optional string hostname = 1;

[terelius, 2017/08/16 14:19:06]

Generally speaking, we don't want to have strings in the proto. Is this really
necessary?

+
+  // required - error code.
+  optional int32 error = 2;
+
+  // optional - resulting IP address (anonymized).
+  optional string lookup_result = 3;

[terelius, 2017/08/16 14:19:07]

Same here. How do we guarantee anonymization?

+
+  // required - Time elapsed for lookup.
+  optional int64 elapsed_time_in_microseconds = 4;

[terelius, 2017/08/16 14:19:06]

Is microsecond accuracy needed here?

+}
+