| Index: components/proximity_auth/bluetooth_connection.cc
|
| diff --git a/components/proximity_auth/bluetooth_connection.cc b/components/proximity_auth/bluetooth_connection.cc
|
| index f038564a1d67aacad52d6786f9775351c1aec32c..f06d52f6d260d05e435a8ebc3a091ecfe8425271 100644
|
| --- a/components/proximity_auth/bluetooth_connection.cc
|
| +++ b/components/proximity_auth/bluetooth_connection.cc
|
| @@ -9,6 +9,7 @@
|
| #include "base/numerics/safe_conversions.h"
|
| #include "base/single_thread_task_runner.h"
|
| #include "base/thread_task_runner_handle.h"
|
| +#include "components/proximity_auth/logging/logging.h"
|
| #include "components/proximity_auth/remote_device.h"
|
| #include "components/proximity_auth/wire_message.h"
|
| #include "device/bluetooth/bluetooth_adapter_factory.h"
|
| @@ -26,19 +27,20 @@ BluetoothConnection::BluetoothConnection(const RemoteDevice& remote_device,
|
| }
|
|
|
| BluetoothConnection::~BluetoothConnection() {
|
| - Disconnect();
|
| + if (status() != DISCONNECTED)
|
| + Disconnect();
|
| }
|
|
|
| void BluetoothConnection::Connect() {
|
| if (status() != DISCONNECTED) {
|
| - VLOG(1)
|
| - << "[BC] Ignoring attempt to connect a non-disconnected connection.";
|
| + PA_LOG(WARNING)
|
| + << "Ignoring attempt to connect a non-disconnected connection.";
|
| return;
|
| }
|
|
|
| if (!device::BluetoothAdapterFactory::IsBluetoothAdapterAvailable()) {
|
| - VLOG(1)
|
| - << "[BC] Connection failed: Bluetooth is unsupported on this platform.";
|
| + PA_LOG(WARNING)
|
| + << "Connection failed: Bluetooth is unsupported on this platform.";
|
| return;
|
| }
|
|
|
| @@ -50,8 +52,8 @@ void BluetoothConnection::Connect() {
|
|
|
| void BluetoothConnection::Disconnect() {
|
| if (status() == DISCONNECTED) {
|
| - VLOG(1)
|
| - << "[BC] Ignoring attempt to disconnect a non-connected connection.";
|
| + PA_LOG(WARNING)
|
| + << "Ignoring attempt to disconnect a non-connected connection.";
|
| return;
|
| }
|
|
|
| @@ -93,7 +95,7 @@ void BluetoothConnection::DeviceRemoved(device::BluetoothAdapter* adapter,
|
| return;
|
|
|
| DCHECK_NE(status(), DISCONNECTED);
|
| - VLOG(1) << "[BC] Device disconnected...";
|
| + PA_LOG(INFO) << "Device disconnected...";
|
| Disconnect();
|
| }
|
|
|
| @@ -109,8 +111,8 @@ void BluetoothConnection::OnAdapterInitialized(
|
| const std::string address = remote_device().bluetooth_address;
|
| device::BluetoothDevice* bluetooth_device = adapter->GetDevice(address);
|
| if (!bluetooth_device) {
|
| - VLOG(1) << "[BC] Device with address " << address
|
| - << " is not known to the system Bluetooth daemon.";
|
| + PA_LOG(WARNING) << "Device with address " << address
|
| + << " is not known to the system Bluetooth daemon.";
|
| // TOOD(isherman): Optimistically attempt to seek the device and connect
|
| // anyway, as was previously implemented in BluetoothConnectionFinder.
|
| Disconnect();
|
| @@ -133,31 +135,31 @@ void BluetoothConnection::OnConnected(
|
| // This case is reachable if the client of |this| connection called
|
| // |Disconnect()| while the backing Bluetooth connection was pending.
|
| DCHECK_EQ(status(), DISCONNECTED);
|
| - VLOG(1) << "[BC] Ignoring successful backend Bluetooth connection to an "
|
| - << "already disconnected logical connection.";
|
| + PA_LOG(WARNING) << "Ignoring successful backend Bluetooth connection to an "
|
| + << "already disconnected logical connection.";
|
| return;
|
| }
|
|
|
| - VLOG(1) << "[BC] Connection established with "
|
| - << remote_device().bluetooth_address;
|
| + PA_LOG(INFO) << "Connection established with "
|
| + << remote_device().bluetooth_address;
|
| socket_ = socket;
|
| SetStatus(CONNECTED);
|
| StartReceive();
|
| }
|
|
|
| void BluetoothConnection::OnConnectionError(const std::string& error_message) {
|
| - VLOG(1) << "[BC] Connection failed: " << error_message;
|
| + PA_LOG(WARNING) << "Connection failed: " << error_message;
|
| Disconnect();
|
| }
|
|
|
| void BluetoothConnection::OnSend(int bytes_sent) {
|
| - VLOG(1) << "[BC] Successfully sent " << bytes_sent << " bytes.";
|
| + PA_LOG(INFO) << "Successfully sent " << bytes_sent << " bytes.";
|
| OnDidSendMessage(*pending_message_, true);
|
| pending_message_.reset();
|
| }
|
|
|
| void BluetoothConnection::OnSendError(const std::string& error_message) {
|
| - VLOG(1) << "[BC] Error when sending bytes: " << error_message;
|
| + PA_LOG(WARNING) << "Error when sending bytes: " << error_message;
|
| OnDidSendMessage(*pending_message_, false);
|
| pending_message_.reset();
|
|
|
| @@ -166,7 +168,7 @@ void BluetoothConnection::OnSendError(const std::string& error_message) {
|
|
|
| void BluetoothConnection::OnReceive(int bytes_received,
|
| scoped_refptr<net::IOBuffer> buffer) {
|
| - VLOG(1) << "[BC] Received " << bytes_received << " bytes.";
|
| + PA_LOG(INFO) << "Received " << bytes_received << " bytes.";
|
| OnBytesReceived(std::string(buffer->data(), bytes_received));
|
|
|
| // Post a task to delay the read until the socket is available, as
|
| @@ -179,7 +181,7 @@ void BluetoothConnection::OnReceive(int bytes_received,
|
| void BluetoothConnection::OnReceiveError(
|
| device::BluetoothSocket::ErrorReason error_reason,
|
| const std::string& error_message) {
|
| - VLOG(1) << "[BC] Error receiving bytes: " << error_message;
|
| + PA_LOG(WARNING) << "Error receiving bytes: " << error_message;
|
|
|
| // Post a task to delay the read until the socket is available, as
|
| // calling StartReceive at this point would error with ERR_IO_PENDING.
|
|
|