bluetooth: Implement simulateGATTConnectionResponse()

third_party/WebKit/LayoutTests/bluetooth/server/connect/garbage-collection-ran-during-error.html

Index: third_party/WebKit/LayoutTests/bluetooth/server/connect/garbage-collection-ran-during-error.html
diff --git a/third_party/WebKit/LayoutTests/bluetooth/server/connect/garbage-collection-ran-during-error.html b/third_party/WebKit/LayoutTests/bluetooth/server/connect/garbage-collection-ran-during-error.html
index 9aa12e37f081da191561ab58776152625bbb0f8e..c11beaa7413a21f04184c3a5149f9cc75996ec6a 100644
--- a/third_party/WebKit/LayoutTests/bluetooth/server/connect/garbage-collection-ran-during-error.html
+++ b/third_party/WebKit/LayoutTests/bluetooth/server/connect/garbage-collection-ran-during-error.html
@@ -2,16 +2,20 @@
 <script src="../../../resources/testharness.js"></script>
 <script src="../../../resources/testharnessreport.js"></script>
 <script src="../../../resources/bluetooth/bluetooth-helpers.js"></script>
+<script src="../../../resources/bluetooth/web-bluetooth-test.js"></script>
+<script src="../../../resources/mojo-helpers.js"></script>
 <script>
 'use strict';
 promise_test(t => {
-  return setBluetoothFakeAdapter('FailingConnectionsAdapter')
-    .then(() => requestDeviceWithKeyDown({
-      filters: [{services: [errorUUID(0x1) /* in progress error */]}]}))
-    .then(device => {
-      promise_rejects(t, 'NetworkError', device.gatt.connect());
-    })
-    .then(runGarbageCollection);
+  return setUpPreconnectedDevice({}).then(fake_peripheral => {
+    return requestDeviceWithKeyDown({acceptAllDevices: true})
+      .then(device => {
+        promise_rejects(t, 'NetworkError', device.gatt.connect());
+      })
+      .then(runGarbageCollection)

[scheib, 2017/05/12 05:43:56]

We should file an issue if not already done on 'runGarbageCollection'

[ortuno, 2017/05/15 07:01:31]

Do we need one? I wasn't planning on usptreaming this. I think we would get
quite a bit of push back on it; I've been told in the past that we should not be
testing this through layout tests but rather with unit tests.

+      .then(() => fake_peripheral.simulateGATTConnectionResponse(
+        {code: HCI_CONNECTION_TIMEOUT}));
+  });
 }, 'Garbage collection ran during a connect call that fails. ' +
    'Should not crash.');
 </script>