Bluetooth: Add a "connectable" property to the BluetoothDevice.

device/bluetooth/bluetooth_device_chromeos.h

 // Copyright (c) 2012 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.
 
 #ifndef DEVICE_BLUETOOTH_BLUETOOTH_DEVICE_CHROMEOS_H_
 #define DEVICE_BLUETOOTH_BLUETOOTH_DEVICE_CHROMEOS_H_
 
 #include <string>
 #include <vector>
 
@@ skipping 107 matching lines @@
   // Called by BluetoothAdapterClient when a call to DiscoverServices()
   // completes.  |callback| and |error_callback| are the callbacks provided to
   // GetServiceRecords.
   void CollectServiceRecordsCallback(
       const ServiceRecordsCallback& callback,
       const ErrorCallback& error_callback,
       const dbus::ObjectPath& device_path,
       const BluetoothDeviceClient::ServiceMap& service_map,
       bool success);
 
+  // Called by CollectServiceRecordsCallback() every time the |service_records_|

[keybuk, 2013/02/22 23:50:03]

nit: drop the "the", it's cleaner

[deymo, 2013/02/23 00:43:16]

Done.

+  // is updated. |OnServiceRecordsChanged| updates the derivated properties that

[keybuk, 2013/02/22 23:50:03]

nit: "derived" ... "derivated" is not a real word ;-)

[deymo, 2013/02/23 00:43:16]

Done.

+  // depend on |service_records_|.
+  void OnServiceRecordsChanged(void);
+
   // Called by BluetoothProperty when the call to Set() for the Trusted
   // property completes. |success| indicates whether or not the request
-  // succeeded, |callback| and |error_callback| are the callbacks provided to
-  // Connect().
-  void OnSetTrusted(const base::Closure& callback,
-                    const ErrorCallback& error_callback,
-                    bool success);
+  // succeeded.
+  void OnSetTrusted(bool success);
+
+  // Called by BluetoothAdapterClient when a call to DiscoverServices()

[keybuk, 2013/02/22 23:50:03]

nit: GetServiceRecords()

[deymo, 2013/02/23 00:43:16]

Done.

+  // fails.  |callback| and |error_callback| are the callbacks provided to
+  // GetServiceRecords.

[keybuk, 2013/02/22 23:50:03]

nit: needs a ()

[deymo, 2013/02/23 00:43:16]

Done.

+  void OnGetServiceRecordsError(const ServiceRecordsCallback& callback,
+                                const ErrorCallback& error_callback);
+
+  // Called by BluetoothAdapterClient when the initial call to
+  // DiscoverServices() after pairing completes. |callback| and |error_callback|
+  // are the callbacks provided to Connect().
+  void OnInitialGetServiceRecords(const base::Closure& callback,
+                                  const ConnectErrorCallback& error_callback,
+                                  const ServiceRecordList& list);
+
+  // Called by BluetoothAdapterClient when the initial call to
+  // DiscoverServices() after pairing fails. |callback| and |error_callback|
+  // are the callbacks provided to Connect().
+  void OnInitialGetServiceRecordsError(
+      const base::Closure& callback,
+      const ConnectErrorCallback& error_callback);
 
   // Connect application-level protocols of the device to the system, called
   // on a successful connection or to reconnect to a device that is already
   // paired or previously connected. |error_callback| is called on failure.
   // Otherwise, |callback| is called when the request is complete.
   void ConnectApplications(const base::Closure& callback,
                            const ConnectErrorCallback& error_callback);
 
   // Called by IntrospectableClient when a call to Introspect() completes.
   // |success| indicates whether or not the request succeeded, |callback| and
@@ skipping 216 matching lines @@
   // During pairing these callbacks are set to those provided by method calls
   // made on us by |agent_| and are called by our own method calls such as
   // SetPinCode() and SetPasskey().
   PinCodeCallback pincode_callback_;
   PasskeyCallback passkey_callback_;
   ConfirmationCallback confirmation_callback_;
 
   // Used to keep track of pending application connection requests.
   int connecting_applications_counter_;
 
+  // A service records cache.
+  ServiceRecordList service_records_;
+
+  // This says wether the |service_records_| cache is initialized. Note that an

[keybuk, 2013/02/22 23:50:03]

nit: whether

[deymo, 2013/02/23 00:43:16]

Done.

+  // empty |service_records_| list can be a valid list.
+  bool service_records_loaded_;
+
   // Note: This should remain the last member so it'll be destroyed and
   // invalidate its weak pointers before any other members are destroyed.
   base::WeakPtrFactory<BluetoothDeviceChromeOS> weak_ptr_factory_;
 
   DISALLOW_COPY_AND_ASSIGN(BluetoothDeviceChromeOS);
 };
 
 }  // namespace chromeos
 
 #endif  // DEVICE_BLUETOOTH_BLUETOOTH_DEVICE_CHROMEOS_H_