bluetooth: Separate requestDevice tests

third_party/WebKit/LayoutTests/resources/bluetooth/bluetooth-helpers.js

 'use strict';
 
 // Bluetooth UUID constants:
 // Services:
 var blacklist_test_service_uuid = "611c954a-263b-4f4a-aab6-01ddb953f985";
 // Characteristics:
 var blacklist_exclude_reads_characteristic_uuid =
   "bad1c9a2-9a5b-4015-8b60-1579bbbf2135";
 
 // Sometimes we need to test that using either the name, alias, or UUID
@@ skipping 287 matching lines @@
   constructor(object, event) {
     this.eventFired = false;
     let event_listener = e => {
       object.removeEventListener(event, event_listener);
       this.eventFired = true;
     }
     object.addEventListener(event, event_listener);
   }
 }
 
+function generateRequestDeviceArgsWithServices(services) {
+  services = (services === undefined) ? ['heart_rate'] : services;

[Jeffrey Yasskin, 2016/05/09 22:09:58]

Make this an if statement:

if (services === undefined) {
  services = ['heart_rate'];
}

[ortuno, 2016/05/11 19:29:39]

Used default argument instead.

[Jeffrey Yasskin, 2016/05/11 20:27:17]

Oh, cool. Yay for new Javascript features.

+  return [{
+    filters: [{ services: services}]

[Jeffrey Yasskin, 2016/05/09 22:09:58]

Nit: Be symmetric with the spaces inside {}s: probably include 1 space on both
sides.

[ortuno, 2016/05/11 19:29:39]

Done.

+  }, {
+    filters: [{ services: services, name: 'Name'}]
+  }, {
+    filters: [{ services: services, namePrefix: 'Pre'}]
+  }, {
+    filters: [{ services: services, name: 'Name', namePrefix: 'Pre'}]
+  }, {
+    filters: [{ services: services}],
+    optionalServices: ['heart_rate']
+  }, {
+    filters: [{ services: services, name: 'Name'}],
+    optionalServices: ['heart_rate']
+  }, {
+    filters: [{ services: services, namePrefix: 'Pre'}],
+    optionalServices: ['heart_rate']
+  }, {
+    filters: [{ services: services, name: 'Name', namePrefix: 'Pre'}],
+    optionalServices: ['heart_rate']
+  }];
+}
+
 // Bluetooth tests sometimes have left-over state that could leak into the
 // next test. add_result_callback which is exposed by testharness.js allows us
-// to clean up this state after each test. In the future we will split tests
-// into separate files so that we don't have to add this callback ourselves.
-// TODO(ortuno): Split tests into separate files.
-// https://crbug.com/554240
+// to clean up this state after each test. Once the move to Mojo is complete

[Jeffrey Yasskin, 2016/05/09 22:09:58]

If this is the last change to split up tests, remove this at the same time?

[ortuno, 2016/05/11 19:29:39]

We can't get rid of the adapter one yet because our service doesn't clean up the
adapter after each navigation yet. The next patch will remove the call to
setBluetoothFakeAdapter('').

[Jeffrey Yasskin, 2016/05/11 20:27:17]

SG

+// we will no longer need to clean up the state manually.
+// https://crbug.com/508771
 add_result_callback(() => {
-  // At the end of each test we clean up all the leftover data in the browser,
-  // including revoking permissions. This happens before the test document is
-  // detached. Once the document is detached any device that connected tries
-  // to disconnect but by then the document no longer has permission to
-  // interact with the device. So before we clean up the browser data
-  // we change the visibility which results in all devices disconnecing.
-  // TODO(ortuno): Remove setPageVisibility hack. In the future, the browser
-  // will notify the renderer that the device disconnected so we won't need
-  // this hack.
-  // https://crbug.com/581855
-  testRunner.setBluetoothManualChooser(false);
   setBluetoothFakeAdapter('');
 });