bluetooth: Add tests for when frame reconnects during readValue

device/bluetooth/test/mock_bluetooth_device.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_TEST_MOCK_BLUETOOTH_DEVICE_H_
 #define DEVICE_BLUETOOTH_TEST_MOCK_BLUETOOTH_DEVICE_H_
 
 #include <stdint.h>
 
 #include <memory>
@@ skipping 98 matching lines @@
   // ON_CALL(*mock_device, GetGattServices))
   //   .WillByDefault(Invoke(*mock_device,
   //                         &MockBluetoothDevice::GetMockServices));
   void AddMockService(std::unique_ptr<MockBluetoothGattService> mock_device);
   std::vector<BluetoothRemoteGattService*> GetMockServices() const;
   BluetoothRemoteGattService* GetMockService(
       const std::string& identifier) const;
 
   void AddUUID(const BluetoothUUID& uuid) { uuids_.insert(uuid); }
 
+  // Functions to save and retrieve callbacks from this device. Useful when
+  // trying to run callbacks in response to other actions e.g. run a read
+  // value callback in response to a connection request.
+  // Appends callback to the end of the callbacks queue.
+  void PushPendingCallback(const base::Closure& callback);
+  // Returns true if there are any callbacks in the queue; false otherwise.
+  bool HasPendingCallback();
+  // Pops and returns the first callback in the queue.
+  base::Closure PopPendingCallback();

[Jeffrey Yasskin, 2016/10/22 01:02:06]

Consider making this RunAllCallbacks() instead. If you need a way to run one at
a time, then Pop makes sense, but I don't see it in this patch.

[ortuno, 2016/10/24 01:52:33]

Done. I also removed the HasPendingCallback function since it is no longer
needed.

+
   void SetConnected(bool connected) { connected_ = connected; }
 
  private:
   uint32_t bluetooth_class_;
   base::Optional<std::string> name_;
   std::string address_;
   BluetoothDevice::UUIDSet uuids_;
   bool connected_;
 
+  // Used by tests to save callbacks that will be run in the future.
+  std::queue<base::Closure> pending_callbacks_;
+
   ScopedVector<MockBluetoothGattService> mock_services_;
 };
 
 }  // namespace device
 
 #endif  // DEVICE_BLUETOOTH_TEST_MOCK_BLUETOOTH_DEVICE_H_