Test that the right events are sent to the Bluetooth chooser.

LayoutTests/bluetooth/resources/bluetooth-helpers.js

Index: LayoutTests/bluetooth/resources/bluetooth-helpers.js
diff --git a/LayoutTests/bluetooth/resources/bluetooth-helpers.js b/LayoutTests/bluetooth/resources/bluetooth-helpers.js
index 064880493c829a79d49516cd89c63e9b8beff53e..f226699c16c1451ad821b4e1fdf539b989f87c28 100644
--- a/LayoutTests/bluetooth/resources/bluetooth-helpers.js
+++ b/LayoutTests/bluetooth/resources/bluetooth-helpers.js
@@ -57,6 +57,23 @@ function requestDeviceWithKeyDown() {
   return callWithKeyDown(() => navigator.bluetooth.requestDevice.apply(navigator.bluetooth, args));
 }
 
+// Calls testRunner.getBluetoothManualChooserEvents() until it's returned
+// |expected_count| events. Or just once if |expected_count| is undefined.
+function getBluetoothManualChooserEvents(expected_count) {
+  return new Promise((resolve, reject) => {
+    let events = [];
+    let accumulate_events = new_events => {
+      events.push(...new_events);
+      if (expected_count === undefined || events.length >= expected_count) {

[ortuno, 2015/09/22 19:00:02]

Would it be better if the first line was:
expected_count = expected_count !== undefined ? expected_count : 0;

and then this could be simplified to:
if (events.length >= expected_count)

That way when we get default function parameters we can just remove the first
line.

nit: you shouldn't compare to undefined:
http://stackoverflow.com/questions/27509/detecting-an-undefined-object-proper...

[Jeffrey Yasskin, 2015/09/22 20:01:34]

Good idea. Done.
This isn't true anymore. Trying to overwrite 'undefined' has no effect.

+        resolve(events);
+      } else {
+        testRunner.getBluetoothManualChooserEvents(accumulate_events);
+      }
+    };
+    testRunner.getBluetoothManualChooserEvents(accumulate_events);
+  });
+}
+
 // errorUUID(alias) returns a UUID with the top 32 bits of
 // '00000000-97e5-4cd7-b9f1-f5a427670c59' replaced with the bits of |alias|.
 // For example, errorUUID(0xDEADBEEF) returns
@@ -85,3 +102,24 @@ function assert_promise_rejects_with_message(promise, expected, description) {
     }
   });
 }
+
+// Parses add-device(name)=id lines in
+// testRunner.getBluetoothManualChooserEvents() output, and exposes the name->id
+// mapping.
+class AddDeviceEventSet {
+  constructor() {
+    this._idsByName = new Map();
+    this._addDeviceRegex = /^add-device\(([^)]+)\)=(.+)$/;
+  }
+  assert_add_device_event(event, description) {
+    let match = this._addDeviceRegex.exec(event);
+    assert_true(!!match, event + "isn't an add-device event: " + description);
+    this._idsByName.set(match[1], match[2]);
+  }
+  has(name) {
+    return this._idsByName.has(name);
+  }
+  get(name) {
+    return this._idsByName.get(name);
+  }
+}