Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1832)

Unified Diff: device/bluetooth/bluetooth_adapter_chromeos.cc

Issue 1124883004: Submission for C++ Readability (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 5 years, 7 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: device/bluetooth/bluetooth_adapter_chromeos.cc
diff --git a/device/bluetooth/bluetooth_adapter_chromeos.cc b/device/bluetooth/bluetooth_adapter_chromeos.cc
index 5af69c84613975d64aa0653f2182e90bdb503183..ff77038d2163424372e0f0ea7c55b2f6d82c21a0 100644
--- a/device/bluetooth/bluetooth_adapter_chromeos.cc
+++ b/device/bluetooth/bluetooth_adapter_chromeos.cc
@@ -87,7 +87,7 @@ void BluetoothAdapterChromeOS::Shutdown() {
if (IsPresent())
RemoveAdapter(); // Also deletes devices_.
DCHECK(devices_.empty());
- // profiles_ should be empty because all BluetoothSockets have been signaled
+ // profiles_ is empty because all BluetoothSockets have been notified
// that this adapter is disappearing.
DCHECK(profiles_.empty());
@@ -324,9 +324,8 @@ void BluetoothAdapterChromeOS::RegisterAdvertisement(
void BluetoothAdapterChromeOS::RemovePairingDelegateInternal(
BluetoothDevice::PairingDelegate* pairing_delegate) {
- // Before removing a pairing delegate make sure that there aren't any devices
- // currently using it; if there are, clear the pairing context which will
- // make any responses no-ops.
+ // Check if any device is using the pairing delegate.
+ // If so, clear the pairing context which will make any responses no-ops.
for (DevicesMap::iterator iter = devices_.begin();
iter != devices_.end(); ++iter) {
BluetoothDeviceChromeOS* device_chromeos =

Powered by Google App Engine
This is Rietveld 408576698