Bluetooth low energy connection.

components/proximity_auth/ble/proximity_auth_ble_system.cc

 // Copyright 2015 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "components/proximity_auth/ble/proximity_auth_ble_system.h"
 
 #include "base/bind.h"
 #include "base/logging.h"
+#include "components/proximity_auth/ble/bluetooth_low_energy_connection.h"
 #include "components/proximity_auth/ble/bluetooth_low_energy_connection_finder.h"
 #include "components/proximity_auth/connection.h"
+#include "components/proximity_auth/remote_device.h"
 #include "device/bluetooth/bluetooth_device.h"
 #include "device/bluetooth/bluetooth_gatt_connection.h"
 
 namespace proximity_auth {
 
 namespace {
 
 // The UUID of the Bluetooth Low Energy service.
 const char kSmartLockServiceUUID[] = "b3b7e28e-a000-3e17-bd86-6e97b9e28c11";
 
@@ skipping 34 matching lines @@
     case ScreenlockBridge::LockHandler::OTHER_SCREEN:
       connection_finder_.reset();
       break;
   }
 };
 
 void ProximityAuthBleSystem::OnScreenDidUnlock(
     ScreenlockBridge::LockHandler::ScreenType screen_type) {
   VLOG(1) << "OnScreenDidUnlock: " << screen_type;
   connection_finder_.reset();
+  connection_.reset();

[msarda, 2015/05/05 11:56:15]

Destroy the connection before the finder. Just in case the connection may need
to talk with the finder.

I would also call connection_->Disconnect() to make it clear that the connection
is actually closed here.

[sacomoto, 2015/05/06 13:47:59]

Done.

 };
 
 void ProximityAuthBleSystem::OnFocusedUserChanged(const std::string& user_id) {
   VLOG(1) << "OnFocusedUserChanged: " << user_id;
 };
 
 void ProximityAuthBleSystem::OnConnectionFound(
-    scoped_ptr<device::BluetoothGattConnection> connection) {
+    scoped_ptr<Connection> connection) {
   VLOG(1) << "Connection found. Unlock.";
 
-  // Close the connection as it it no longer needed.
-  connection_finder_->CloseConnection(connection.Pass());
+  connection_ = connection.Pass();

[msarda, 2015/05/05 11:56:15]

We should close the connection when the screen in unlocked.

[sacomoto, 2015/05/06 13:47:59]

There is no need to do it here. When the screen is unlocked
ProximityAuthBleSystem::OnScreenDidUnlock is called, and the connection is
destroyed.

 
   // Unlock the screen when a connection is found.
   //
   // Note that this magically unlocks Chrome (no user interaction is needed).
   // This user experience for this operation will be greately improved once
   // the Proximity Auth Unlock Manager migration to C++ is done.
   screenlock_bridge_->Unlock(browser_context_);
 }
 
 }  // namespace proximity_auth