Rename TraceCategory struct to match style guide

This patch renames TraceCategory enum values to match the style guide
and resolve the in-code TODO: comment left by rwkeane. This is the first
change in a series of trace logging related changes.

https: //google.github.io/styleguide/cppguide.html#Enumerator_Names
Change-Id: Ia3ebdafe6bce2f8432a34ec77872a002665e0743
Reviewed-on: https://chromium-review.googlesource.com/c/openscreen/+/2036491
Reviewed-by: Ryan Keane <rwkeane@google.com>
Commit-Queue: Jordan Bayles <jophba@chromium.org>

5 files changed, 46 insertions, 44 deletions

diff --git a/osp/impl/discovery/mdns/mdns_responder_adapter_impl.cc b/osp/impl/discovery/mdns/mdns_responder_adapter_impl.cc
index 57fd0b2d..b6b86c86 100644
--- a/osp/impl/discovery/mdns/mdns_responder_adapter_impl.cc
+++ b/osp/impl/discovery/mdns/mdns_responder_adapter_impl.cc
@@ -204,7 +204,7 @@ Error MdnsResponderAdapterImpl::Init() {
 }
 
 void MdnsResponderAdapterImpl::Close() {
-  TRACE_SCOPED(TraceCategory::mDNS, "MdnsResponderAdapterImpl::Close");
+  TRACE_SCOPED(TraceCategory::kMdns, "MdnsResponderAdapterImpl::Close");
   mDNS_StartExit(&mdns_);
   // Let all services send goodbyes.
   while (!service_records_.empty()) {
@@ -226,7 +226,7 @@ void MdnsResponderAdapterImpl::Close() {
 }
 
 Error MdnsResponderAdapterImpl::SetHostLabel(const std::string& host_label) {
-  TRACE_SCOPED(TraceCategory::mDNS, "MdnsResponderAdapterImpl::SetHostLabel");
+  TRACE_SCOPED(TraceCategory::kMdns, "MdnsResponderAdapterImpl::SetHostLabel");
   if (host_label.size() > DomainName::kDomainNameMaxLabelLength)
     return Error::Code::kDomainNameTooLong;
 
@@ -243,7 +243,7 @@ Error MdnsResponderAdapterImpl::RegisterInterface(
     const InterfaceInfo& interface_info,
     const IPSubnet& interface_address,
     UdpSocket* socket) {
-  TRACE_SCOPED(TraceCategory::mDNS,
+  TRACE_SCOPED(TraceCategory::kMdns,
                "MdnsResponderAdapterImpl::RegisterInterface");
   OSP_DCHECK(socket);
 
@@ -283,7 +283,7 @@ Error MdnsResponderAdapterImpl::RegisterInterface(
 }
 
 Error MdnsResponderAdapterImpl::DeregisterInterface(UdpSocket* socket) {
-  TRACE_SCOPED(TraceCategory::mDNS,
+  TRACE_SCOPED(TraceCategory::kMdns,
                "MdnsResponderAdapterImpl::DeregisterInterface");
   const auto info_it = responder_interface_info_.find(socket);
   if (info_it == responder_interface_info_.end())
@@ -308,7 +308,7 @@ void MdnsResponderAdapterImpl::OnRead(UdpSocket* socket,
   }
 
   UdpPacket packet = std::move(packet_or_error.value());
-  TRACE_SCOPED(TraceCategory::mDNS, "MdnsResponderAdapterImpl::OnRead");
+  TRACE_SCOPED(TraceCategory::kMdns, "MdnsResponderAdapterImpl::OnRead");
   mDNSAddr src;
   if (packet.source().address.IsV4()) {
     src.type = mDNSAddrType_IPv4;
@@ -348,7 +348,7 @@ void MdnsResponderAdapterImpl::OnError(UdpSocket* socket, Error error) {
 }
 
 Clock::duration MdnsResponderAdapterImpl::RunTasks() {
-  TRACE_SCOPED(TraceCategory::mDNS, "MdnsResponderAdapterImpl::RunTasks");
+  TRACE_SCOPED(TraceCategory::kMdns, "MdnsResponderAdapterImpl::RunTasks");
 
   mDNS_Execute(&mdns_);
 
@@ -405,7 +405,7 @@ std::vector<AaaaEvent> MdnsResponderAdapterImpl::TakeAaaaResponses() {
 MdnsResponderErrorCode MdnsResponderAdapterImpl::StartPtrQuery(
     UdpSocket* socket,
     const DomainName& service_type) {
-  TRACE_SCOPED(TraceCategory::mDNS, "MdnsResponderAdapterImpl::StartPtrQuery");
+  TRACE_SCOPED(TraceCategory::kMdns, "MdnsResponderAdapterImpl::StartPtrQuery");
   auto& ptr_questions = socket_to_questions_[socket].ptr;
   if (ptr_questions.find(service_type) != ptr_questions.end())
     return MdnsResponderErrorCode::kNoError;
@@ -452,7 +452,7 @@ MdnsResponderErrorCode MdnsResponderAdapterImpl::StartPtrQuery(
 MdnsResponderErrorCode MdnsResponderAdapterImpl::StartSrvQuery(
     UdpSocket* socket,
     const DomainName& service_instance) {
-  TRACE_SCOPED(TraceCategory::mDNS, "MdnsResponderAdapterImpl::StartSrvQuery");
+  TRACE_SCOPED(TraceCategory::kMdns, "MdnsResponderAdapterImpl::StartSrvQuery");
   if (!service_instance.EndsWithLocalDomain())
     return MdnsResponderErrorCode::kInvalidParameters;
 
@@ -490,7 +490,7 @@ MdnsResponderErrorCode MdnsResponderAdapterImpl::StartSrvQuery(
 MdnsResponderErrorCode MdnsResponderAdapterImpl::StartTxtQuery(
     UdpSocket* socket,
     const DomainName& service_instance) {
-  TRACE_SCOPED(TraceCategory::mDNS, "MdnsResponderAdapterImpl::StartTxtQuery");
+  TRACE_SCOPED(TraceCategory::kMdns, "MdnsResponderAdapterImpl::StartTxtQuery");
   if (!service_instance.EndsWithLocalDomain())
     return MdnsResponderErrorCode::kInvalidParameters;
 
@@ -528,7 +528,7 @@ MdnsResponderErrorCode MdnsResponderAdapterImpl::StartTxtQuery(
 MdnsResponderErrorCode MdnsResponderAdapterImpl::StartAQuery(
     UdpSocket* socket,
     const DomainName& domain_name) {
-  TRACE_SCOPED(TraceCategory::mDNS, "MdnsResponderAdapterImpl::StartAQuery");
+  TRACE_SCOPED(TraceCategory::kMdns, "MdnsResponderAdapterImpl::StartAQuery");
   if (!domain_name.EndsWithLocalDomain())
     return MdnsResponderErrorCode::kInvalidParameters;
 
@@ -566,7 +566,8 @@ MdnsResponderErrorCode MdnsResponderAdapterImpl::StartAQuery(
 MdnsResponderErrorCode MdnsResponderAdapterImpl::StartAaaaQuery(
     UdpSocket* socket,
     const DomainName& domain_name) {
-  TRACE_SCOPED(TraceCategory::mDNS, "MdnsResponderAdapterImpl::StartAaaaQuery");
+  TRACE_SCOPED(TraceCategory::kMdns,
+               "MdnsResponderAdapterImpl::StartAaaaQuery");
   if (!domain_name.EndsWithLocalDomain())
     return MdnsResponderErrorCode::kInvalidParameters;
 
@@ -604,7 +605,7 @@ MdnsResponderErrorCode MdnsResponderAdapterImpl::StartAaaaQuery(
 MdnsResponderErrorCode MdnsResponderAdapterImpl::StopPtrQuery(
     UdpSocket* socket,
     const DomainName& service_type) {
-  TRACE_SCOPED(TraceCategory::mDNS, "MdnsResponderAdapterImpl::StopPtrQuery");
+  TRACE_SCOPED(TraceCategory::kMdns, "MdnsResponderAdapterImpl::StopPtrQuery");
   auto interface_entry = socket_to_questions_.find(socket);
   if (interface_entry == socket_to_questions_.end())
     return MdnsResponderErrorCode::kNoError;
@@ -622,7 +623,7 @@ MdnsResponderErrorCode MdnsResponderAdapterImpl::StopPtrQuery(
 MdnsResponderErrorCode MdnsResponderAdapterImpl::StopSrvQuery(
     UdpSocket* socket,
     const DomainName& service_instance) {
-  TRACE_SCOPED(TraceCategory::mDNS, "MdnsResponderAdapterImpl::StopSrvQuery");
+  TRACE_SCOPED(TraceCategory::kMdns, "MdnsResponderAdapterImpl::StopSrvQuery");
   auto interface_entry = socket_to_questions_.find(socket);
   if (interface_entry == socket_to_questions_.end())
     return MdnsResponderErrorCode::kNoError;
@@ -640,7 +641,7 @@ MdnsResponderErrorCode MdnsResponderAdapterImpl::StopSrvQuery(
 MdnsResponderErrorCode MdnsResponderAdapterImpl::StopTxtQuery(
     UdpSocket* socket,
     const DomainName& service_instance) {
-  TRACE_SCOPED(TraceCategory::mDNS, "MdnsResponderAdapterImpl::StopTxtQuery");
+  TRACE_SCOPED(TraceCategory::kMdns, "MdnsResponderAdapterImpl::StopTxtQuery");
   auto interface_entry = socket_to_questions_.find(socket);
   if (interface_entry == socket_to_questions_.end())
     return MdnsResponderErrorCode::kNoError;
@@ -658,7 +659,7 @@ MdnsResponderErrorCode MdnsResponderAdapterImpl::StopTxtQuery(
 MdnsResponderErrorCode MdnsResponderAdapterImpl::StopAQuery(
     UdpSocket* socket,
     const DomainName& domain_name) {
-  TRACE_SCOPED(TraceCategory::mDNS, "MdnsResponderAdapterImpl::StopAQuery");
+  TRACE_SCOPED(TraceCategory::kMdns, "MdnsResponderAdapterImpl::StopAQuery");
   auto interface_entry = socket_to_questions_.find(socket);
   if (interface_entry == socket_to_questions_.end())
     return MdnsResponderErrorCode::kNoError;
@@ -676,7 +677,7 @@ MdnsResponderErrorCode MdnsResponderAdapterImpl::StopAQuery(
 MdnsResponderErrorCode MdnsResponderAdapterImpl::StopAaaaQuery(
     UdpSocket* socket,
     const DomainName& domain_name) {
-  TRACE_SCOPED(TraceCategory::mDNS, "MdnsResponderAdapterImpl::StopAaaaQuery");
+  TRACE_SCOPED(TraceCategory::kMdns, "MdnsResponderAdapterImpl::StopAaaaQuery");
   auto interface_entry = socket_to_questions_.find(socket);
   if (interface_entry == socket_to_questions_.end())
     return MdnsResponderErrorCode::kNoError;
@@ -698,7 +699,7 @@ MdnsResponderErrorCode MdnsResponderAdapterImpl::RegisterService(
     const DomainName& target_host,
     uint16_t target_port,
     const std::map<std::string, std::string>& txt_data) {
-  TRACE_SCOPED(TraceCategory::mDNS,
+  TRACE_SCOPED(TraceCategory::kMdns,
                "MdnsResponderAdapterImpl::RegisterService");
   OSP_DCHECK(IsValidServiceName(service_name));
   OSP_DCHECK(IsValidServiceProtocol(service_protocol));
@@ -743,7 +744,7 @@ MdnsResponderErrorCode MdnsResponderAdapterImpl::DeregisterService(
     const std::string& service_instance,
     const std::string& service_name,
     const std::string& service_protocol) {
-  TRACE_SCOPED(TraceCategory::mDNS,
+  TRACE_SCOPED(TraceCategory::kMdns,
                "MdnsResponderAdapterImpl::DeregisterService");
   domainlabel instance;
   domainlabel name;
@@ -773,7 +774,7 @@ MdnsResponderErrorCode MdnsResponderAdapterImpl::UpdateTxtData(
     const std::string& service_name,
     const std::string& service_protocol,
     const std::map<std::string, std::string>& txt_data) {
-  TRACE_SCOPED(TraceCategory::mDNS, "MdnsResponderAdapterImpl::UpdateTxtData");
+  TRACE_SCOPED(TraceCategory::kMdns, "MdnsResponderAdapterImpl::UpdateTxtData");
   domainlabel instance;
   domainlabel name;
   domainlabel protocol;
@@ -807,7 +808,8 @@ void MdnsResponderAdapterImpl::AQueryCallback(mDNS* m,
                                               DNSQuestion* question,
                                               const ResourceRecord* answer,
                                               QC_result added) {
-  TRACE_SCOPED(TraceCategory::mDNS, "MdnsResponderAdapterImpl::AQueryCallback");
+  TRACE_SCOPED(TraceCategory::kMdns,
+               "MdnsResponderAdapterImpl::AQueryCallback");
   OSP_DCHECK(question);
   OSP_DCHECK(answer);
   OSP_DCHECK_EQ(answer->rrtype, kDNSType_A);
@@ -838,7 +840,7 @@ void MdnsResponderAdapterImpl::AaaaQueryCallback(mDNS* m,
                                                  DNSQuestion* question,
                                                  const ResourceRecord* answer,
                                                  QC_result added) {
-  TRACE_SCOPED(TraceCategory::mDNS,
+  TRACE_SCOPED(TraceCategory::kMdns,
                "MdnsResponderAdapterImpl::AaaaQueryCallback");
   OSP_DCHECK(question);
   OSP_DCHECK(answer);
@@ -870,7 +872,7 @@ void MdnsResponderAdapterImpl::PtrQueryCallback(mDNS* m,
                                                 DNSQuestion* question,
                                                 const ResourceRecord* answer,
                                                 QC_result added) {
-  TRACE_SCOPED(TraceCategory::mDNS,
+  TRACE_SCOPED(TraceCategory::kMdns,
                "MdnsResponderAdapterImpl::PtrQueryCallback");
   OSP_DCHECK(question);
   OSP_DCHECK(answer);
@@ -901,7 +903,7 @@ void MdnsResponderAdapterImpl::SrvQueryCallback(mDNS* m,
                                                 DNSQuestion* question,
                                                 const ResourceRecord* answer,
                                                 QC_result added) {
-  TRACE_SCOPED(TraceCategory::mDNS,
+  TRACE_SCOPED(TraceCategory::kMdns,
                "MdnsResponderAdapterImpl::SrvQueryCallback");
   OSP_DCHECK(question);
   OSP_DCHECK(answer);
@@ -986,7 +988,7 @@ void MdnsResponderAdapterImpl::ServiceCallback(mDNS* m,
 }
 
 void MdnsResponderAdapterImpl::AdvertiseInterfaces() {
-  TRACE_SCOPED(TraceCategory::mDNS,
+  TRACE_SCOPED(TraceCategory::kMdns,
                "MdnsResponderAdapterImpl::AdvertiseInterfaces");
   for (auto& info : responder_interface_info_) {
     UdpSocket* socket = info.first;
diff --git a/osp/impl/mdns_responder_service.cc b/osp/impl/mdns_responder_service.cc
index aca15e18..f1aedbdc 100644
--- a/osp/impl/mdns_responder_service.cc
+++ b/osp/impl/mdns_responder_service.cc
@@ -63,7 +63,7 @@ void MdnsResponderService::SetServiceConfig(
 
 void MdnsResponderService::OnRead(UdpSocket* socket,
                                   ErrorOr<UdpPacket> packet) {
-  TRACE_SCOPED(TraceCategory::mDNS, "MdnsResponderService::OnRead");
+  TRACE_SCOPED(TraceCategory::kMdns, "MdnsResponderService::OnRead");
   if (!mdns_responder_) {
     return;
   }
@@ -211,7 +211,7 @@ bool MdnsResponderService::NetworkScopedDomainNameComparator::operator()(
 }
 
 void MdnsResponderService::HandleMdnsEvents() {
-  TRACE_SCOPED(TraceCategory::mDNS, "MdnsResponderService::HandleMdnsEvents");
+  TRACE_SCOPED(TraceCategory::kMdns, "MdnsResponderService::HandleMdnsEvents");
   // NOTE: In the common case, we will get a single combined packet for
   // PTR/SRV/TXT/A and then no other packets.  If we don't loop here, we would
   // start SRV/TXT queries based on the PTR response, but never check for events
diff --git a/osp/impl/presentation/presentation_receiver.cc b/osp/impl/presentation/presentation_receiver.cc
index 162642c0..eb60e0bb 100644
--- a/osp/impl/presentation/presentation_receiver.cc
+++ b/osp/impl/presentation/presentation_receiver.cc
@@ -106,10 +106,10 @@ ErrorOr<size_t> Receiver::OnStreamMessage(uint64_t endpoint_id,
                                           const uint8_t* buffer,
                                           size_t buffer_size,
                                           Clock::time_point now) {
-  TRACE_SCOPED(TraceCategory::Presentation, "Receiver::OnStreamMessage");
+  TRACE_SCOPED(TraceCategory::kPresentation, "Receiver::OnStreamMessage");
   switch (message_type) {
     case msgs::Type::kPresentationUrlAvailabilityRequest: {
-      TRACE_SCOPED(TraceCategory::Presentation,
+      TRACE_SCOPED(TraceCategory::kPresentation,
                    "kPresentationUrlAvailabilityRequest");
       OSP_VLOG << "got presentation-url-availability-request";
       msgs::PresentationUrlAvailabilityRequest request;
@@ -135,7 +135,7 @@ ErrorOr<size_t> Receiver::OnStreamMessage(uint64_t endpoint_id,
     }
 
     case msgs::Type::kPresentationStartRequest: {
-      TRACE_SCOPED(TraceCategory::Presentation, "kPresentationStartRequest");
+      TRACE_SCOPED(TraceCategory::kPresentation, "kPresentationStartRequest");
       OSP_VLOG << "got presentation-start-request";
       msgs::PresentationStartRequest request;
       const ssize_t result =
@@ -196,7 +196,7 @@ ErrorOr<size_t> Receiver::OnStreamMessage(uint64_t endpoint_id,
     }
 
     case msgs::Type::kPresentationConnectionOpenRequest: {
-      TRACE_SCOPED(TraceCategory::Presentation,
+      TRACE_SCOPED(TraceCategory::kPresentation,
                    "kPresentationConnectionOpenRequest");
       OSP_VLOG << "Got a presentation-connection-open-request";
       msgs::PresentationConnectionOpenRequest request;
@@ -264,7 +264,7 @@ ErrorOr<size_t> Receiver::OnStreamMessage(uint64_t endpoint_id,
     }
 
     case msgs::Type::kPresentationTerminationRequest: {
-      TRACE_SCOPED(TraceCategory::Presentation,
+      TRACE_SCOPED(TraceCategory::kPresentation,
                    "kPresentationTerminationRequest");
       OSP_VLOG << "got presentation-termination-request";
       msgs::PresentationTerminationRequest request;
diff --git a/osp/impl/quic/quic_connection_factory_impl.cc b/osp/impl/quic/quic_connection_factory_impl.cc
index 264ebd85..6514b791 100644
--- a/osp/impl/quic/quic_connection_factory_impl.cc
+++ b/osp/impl/quic/quic_connection_factory_impl.cc
@@ -102,7 +102,7 @@ void QuicConnectionFactoryImpl::SetServerDelegate(
 
 void QuicConnectionFactoryImpl::OnRead(UdpSocket* socket,
                                        ErrorOr<UdpPacket> packet_or_error) {
-  TRACE_SCOPED(TraceCategory::Quic, "QuicConnectionFactoryImpl::OnRead");
+  TRACE_SCOPED(TraceCategory::kQuic, "QuicConnectionFactoryImpl::OnRead");
   if (packet_or_error.is_error()) {
     return;
   }
diff --git a/osp/impl/quic/quic_connection_impl.cc b/osp/impl/quic/quic_connection_impl.cc
index e3b36fcf..778d6b4d 100644
--- a/osp/impl/quic/quic_connection_impl.cc
+++ b/osp/impl/quic/quic_connection_impl.cc
@@ -30,7 +30,7 @@ UdpTransport& UdpTransport::operator=(UdpTransport&&) noexcept = default;
 int UdpTransport::Write(const char* buffer,
                         size_t buffer_length,
                         const PacketInfo& info) {
-  TRACE_SCOPED(TraceCategory::Quic, "UdpTransport::Write");
+  TRACE_SCOPED(TraceCategory::kQuic, "UdpTransport::Write");
   socket_->SendMessage(buffer, buffer_length, destination_);
   OSP_DCHECK_LE(buffer_length,
                 static_cast<size_t>(std::numeric_limits<int>::max()));
@@ -46,7 +46,7 @@ QuicStreamImpl::QuicStreamImpl(QuicStream::Delegate* delegate,
 QuicStreamImpl::~QuicStreamImpl() = default;
 
 void QuicStreamImpl::Write(const uint8_t* data, size_t data_size) {
-  TRACE_SCOPED(TraceCategory::Quic, "QuicStreamImpl::Write");
+  TRACE_SCOPED(TraceCategory::kQuic, "QuicStreamImpl::Write");
   OSP_DCHECK(!stream_->write_side_closed());
   stream_->WriteOrBufferData(
       ::quic::QuicStringPiece(reinterpret_cast<const char*>(data), data_size),
@@ -54,7 +54,7 @@ void QuicStreamImpl::Write(const uint8_t* data, size_t data_size) {
 }
 
 void QuicStreamImpl::CloseWriteEnd() {
-  TRACE_SCOPED(TraceCategory::Quic, "QuicStreamImpl::CloseWriteEnd");
+  TRACE_SCOPED(TraceCategory::kQuic, "QuicStreamImpl::CloseWriteEnd");
   if (!stream_->write_side_closed())
     stream_->FinishWriting();
 }
@@ -62,12 +62,12 @@ void QuicStreamImpl::CloseWriteEnd() {
 void QuicStreamImpl::OnReceived(::quic::QuartcStream* stream,
                                 const char* data,
                                 size_t data_size) {
-  TRACE_SCOPED(TraceCategory::Quic, "QuicStreamImpl::OnReceived");
+  TRACE_SCOPED(TraceCategory::kQuic, "QuicStreamImpl::OnReceived");
   delegate_->OnReceived(this, data, data_size);
 }
 
 void QuicStreamImpl::OnClose(::quic::QuartcStream* stream) {
-  TRACE_SCOPED(TraceCategory::Quic, "QuicStreamImpl::OnClose");
+  TRACE_SCOPED(TraceCategory::kQuic, "QuicStreamImpl::OnClose");
   delegate_->OnClose(stream->id());
 }
 
@@ -77,7 +77,7 @@ void QuicStreamImpl::OnBufferChanged(::quic::QuartcStream* stream) {}
 // any stream data, it will be passed automatically to the relevant
 // QuicStream::Delegate objects.
 void QuicConnectionImpl::OnRead(UdpSocket* socket, ErrorOr<UdpPacket> data) {
-  TRACE_SCOPED(TraceCategory::Quic, "QuicConnectionImpl::OnRead");
+  TRACE_SCOPED(TraceCategory::kQuic, "QuicConnectionImpl::OnRead");
   if (data.is_error()) {
     TRACE_SET_RESULT(data.error());
     return;
@@ -106,7 +106,7 @@ QuicConnectionImpl::QuicConnectionImpl(
       parent_factory_(parent_factory),
       session_(std::move(session)),
       udp_transport_(std::move(udp_transport)) {
-  TRACE_SCOPED(TraceCategory::Quic, "QuicConnectionImpl::QuicConnectionImpl");
+  TRACE_SCOPED(TraceCategory::kQuic, "QuicConnectionImpl::QuicConnectionImpl");
   session_->SetDelegate(this);
   session_->OnTransportCanWrite();
   session_->StartCryptoHandshake();
@@ -116,24 +116,24 @@ QuicConnectionImpl::~QuicConnectionImpl() = default;
 
 std::unique_ptr<QuicStream> QuicConnectionImpl::MakeOutgoingStream(
     QuicStream::Delegate* delegate) {
-  TRACE_SCOPED(TraceCategory::Quic, "QuicConnectionImpl::MakeOutgoingStream");
+  TRACE_SCOPED(TraceCategory::kQuic, "QuicConnectionImpl::MakeOutgoingStream");
   ::quic::QuartcStream* stream = session_->CreateOutgoingDynamicStream();
   return std::make_unique<QuicStreamImpl>(delegate, stream);
 }
 
 void QuicConnectionImpl::Close() {
-  TRACE_SCOPED(TraceCategory::Quic, "QuicConnectionImpl::Close");
+  TRACE_SCOPED(TraceCategory::kQuic, "QuicConnectionImpl::Close");
   session_->CloseConnection("closed");
 }
 
 void QuicConnectionImpl::OnCryptoHandshakeComplete() {
-  TRACE_SCOPED(TraceCategory::Quic,
+  TRACE_SCOPED(TraceCategory::kQuic,
                "QuicConnectionImpl::OnCryptoHandshakeComplete");
   delegate_->OnCryptoHandshakeComplete(session_->connection_id());
 }
 
 void QuicConnectionImpl::OnIncomingStream(::quic::QuartcStream* stream) {
-  TRACE_SCOPED(TraceCategory::Quic, "QuicConnectionImpl::OnIncomingStream");
+  TRACE_SCOPED(TraceCategory::kQuic, "QuicConnectionImpl::OnIncomingStream");
   auto public_stream = std::make_unique<QuicStreamImpl>(
       delegate_->NextStreamDelegate(session_->connection_id(), stream->id()),
       stream);
@@ -146,7 +146,7 @@ void QuicConnectionImpl::OnConnectionClosed(
     ::quic::QuicErrorCode error_code,
     const ::quic::QuicString& error_details,
     ::quic::ConnectionCloseSource source) {
-  TRACE_SCOPED(TraceCategory::Quic, "QuicConnectionImpl::OnConnectionClosed");
+  TRACE_SCOPED(TraceCategory::kQuic, "QuicConnectionImpl::OnConnectionClosed");
   parent_factory_->OnConnectionClosed(this);
   delegate_->OnConnectionClosed(session_->connection_id());
 }