From be32b7d2d9966863c2157b558114dd8ac7951d2a Mon Sep 17 00:00:00 2001 From: hai007 Date: Thu, 9 Feb 2023 11:18:08 -0800 Subject: [PATCH] Indicate "Their supported medium" is not real if the other device is host PiperOrigin-RevId: 508426593 --- .../implementation/base_pcp_handler.cc | 22 ++++++++++++------- 1 file changed, 14 insertions(+), 8 deletions(-) diff --git a/connections/implementation/base_pcp_handler.cc b/connections/implementation/base_pcp_handler.cc index 70d168e6..2c2ebff3 100644 --- a/connections/implementation/base_pcp_handler.cc +++ b/connections/implementation/base_pcp_handler.cc @@ -214,17 +214,23 @@ BooleanMediumSelector BasePcpHandler::ComputeIntersectionOfSupportedMediums( if (their_mediums.empty()) { their_mediums.push_back(GetDefaultUpgradeMedium()); } - - for (auto medium : their_mediums) { - NEARBY_LOGS(VERBOSE) << "Their supported medium name: " - << location::nearby::proto::connections::Medium_Name( - medium); + // TODO(b/268243340): Add Supported Medium field to ConnectionResponseFrame + if (connection_info.is_incoming) { + for (auto medium : their_mediums) { + NEARBY_LOGS(INFO) << "Their supported medium name: " + << location::nearby::proto::connections::Medium_Name( + medium); + } + } else { + NEARBY_LOGS(INFO) + << "Current ConnectionResponseFrame from host has no Supported Mediums " + "field, so use calculated default medium instead."; } for (Medium my_medium : GetConnectionMediumsByPriority()) { - NEARBY_LOGS(VERBOSE) << "Our supported medium name: " - << location::nearby::proto::connections::Medium_Name( - my_medium); + NEARBY_LOGS(INFO) << "Our supported medium name: " + << location::nearby::proto::connections::Medium_Name( + my_medium); if (std::find(their_mediums.begin(), their_mediums.end(), my_medium) != their_mediums.end()) { // We use advertising options as a proxy to whether or not the local