Commit 1b93fd12 authored by Ryan Hansberry's avatar Ryan Hansberry Committed by Commit Bot

[Multidevice] Clarify SecureChannel connection process logs.

These logs were previously incorrect making it slightly
confusing to read debug logs after the fact. This CL
makes it clear when advertisements are received and
a connection is beginning to be created.

Change-Id: I93ff8b8b415878c499844ae8b3b99d99f9f2a481
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2134770
Commit-Queue: Ryan Hansberry <hansberry@chromium.org>
Auto-Submit: Ryan Hansberry <hansberry@chromium.org>
Reviewed-by: default avatarKyle Horimoto <khorimoto@chromium.org>
Cr-Commit-Position: refs/heads/master@{#756381}
parent 6361e1e8
......@@ -389,7 +389,7 @@ void BleConnectionManagerImpl::SetAuthenticatingChannel(
PauseConnectionAttemptsToDevice(remote_device_id);
if (DoesAuthenticatingChannelExist(remote_device_id)) {
PA_LOG(ERROR) << "BleConnectionManager::OnReceivedAdvertisement(): A new "
PA_LOG(ERROR) << "BleConnectionManager::SetAuthenticatingChannel(): A new "
<< "channel was created, one already exists for the same "
<< "remote device ID. ID: "
<< multidevice::RemoteDeviceRef::TruncateDeviceIdForLogs(
......@@ -399,9 +399,9 @@ void BleConnectionManagerImpl::SetAuthenticatingChannel(
SecureChannel* secure_channel_raw = secure_channel.get();
PA_LOG(INFO) << "BleConnectionManager::OnReceivedAdvertisement(): Connection "
<< "established; starting authentication process. Remote device "
<< "ID: "
PA_LOG(INFO) << "BleConnectionManager::SetAuthenticatingChannel(): "
<< "Advertisement received; establishing connection. "
<< "Remote device ID: "
<< multidevice::RemoteDeviceRef::TruncateDeviceIdForLogs(
remote_device_id)
<< ", Connection role: " << connection_role;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment