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

webrtc/logging/rtc_event_log/rtc_event_log_parser.cc

 /*
  *  Copyright (c) 2016 The WebRTC project authors. All Rights Reserved.
  *
  *  Use of this source code is governed by a BSD-style license
  *  that can be found in the LICENSE file in the root of the source
  *  tree. An additional intellectual property rights grant can be found
  *  in the file PATENTS.  All contributing project authors may
  *  be found in the AUTHORS file in the root of the source tree.
  */
 
@@ skipping 56 matching lines @@
     case rtclog::Event::AUDIO_RECEIVER_CONFIG_EVENT:
       return ParsedRtcEventLog::EventType::AUDIO_RECEIVER_CONFIG_EVENT;
     case rtclog::Event::AUDIO_SENDER_CONFIG_EVENT:
       return ParsedRtcEventLog::EventType::AUDIO_SENDER_CONFIG_EVENT;
     case rtclog::Event::AUDIO_NETWORK_ADAPTATION_EVENT:
       return ParsedRtcEventLog::EventType::AUDIO_NETWORK_ADAPTATION_EVENT;
     case rtclog::Event::BWE_PROBE_CLUSTER_CREATED_EVENT:
       return ParsedRtcEventLog::EventType::BWE_PROBE_CLUSTER_CREATED_EVENT;
     case rtclog::Event::BWE_PROBE_RESULT_EVENT:
       return ParsedRtcEventLog::EventType::BWE_PROBE_RESULT_EVENT;
+    case rtclog::Event::HOST_LOOKUP_EVENT:
+      return ParsedRtcEventLog::EventType::HOST_LOOKUP_EVENT;
   }
   RTC_NOTREACHED();
   return ParsedRtcEventLog::EventType::UNKNOWN_EVENT;
 }
 
 BandwidthUsage GetRuntimeDetectorState(
     rtclog::DelayBasedBweUpdate::DetectorState detector_state) {
   switch (detector_state) {
     case rtclog::DelayBasedBweUpdate::BWE_NORMAL:
       return BandwidthUsage::kBwNormal;
@@ skipping 560 matching lines @@
 // registered types first.
 ParsedRtcEventLog::MediaType ParsedRtcEventLog::GetMediaType(
     uint32_t ssrc,
     PacketDirection direction) const {
   for (auto rit = streams_.rbegin(); rit != streams_.rend(); ++rit) {
     if (rit->ssrc == ssrc && rit->direction == direction)
       return rit->media_type;
   }
   return MediaType::ANY;
 }
+
+void ParsedRtcEventLog::GetHostLookup(
+    size_t index,
+    int* error, int64_t* host_lookup_time_ms) const {
+  RTC_CHECK_LT(index, GetNumberOfEvents());
+  const rtclog::Event& event = events_[index];
+  RTC_CHECK(event.has_type());
+  RTC_CHECK_EQ(event.type(), rtclog::Event::HOST_LOOKUP_EVENT);
+  RTC_CHECK(event.has_host_lookup_result());
+  const rtclog::HostLookupResult& lookup_event = event.host_lookup_result();
+  RTC_CHECK(lookup_event.has_error());
+  *error = lookup_event.error();

[terelius, 2017/08/21 09:38:14]

We have previously allowed passing in nullptr if we're not interested in
particular field. I suggest changing to
if (error)
  *error = lookup_event.error();

+  RTC_CHECK(lookup_event.has_host_lookup_time_ms());
+  *host_lookup_time_ms = lookup_event.host_lookup_time_ms();

[terelius, 2017/08/21 09:38:14]

Same here.

+}
 }  // namespace webrtc