bluetooth: Implement acceptAllDevices

third_party/WebKit/public/platform/modules/bluetooth/WebRequestDeviceOptions.h

Index: third_party/WebKit/public/platform/modules/bluetooth/WebRequestDeviceOptions.h
diff --git a/third_party/WebKit/public/platform/modules/bluetooth/WebRequestDeviceOptions.h b/third_party/WebKit/public/platform/modules/bluetooth/WebRequestDeviceOptions.h
index 27cf9aaab32b1a1204aebe5e4fdc0a57a654519f..c7a96d3c8fba90a9b6254260e2c33b70fee60db3 100644
--- a/third_party/WebKit/public/platform/modules/bluetooth/WebRequestDeviceOptions.h
+++ b/third_party/WebKit/public/platform/modules/bluetooth/WebRequestDeviceOptions.h
@@ -30,7 +30,9 @@ struct WebRequestDeviceOptions {
   WebRequestDeviceOptions() {}
 
   WebVector<WebBluetoothScanFilter> filters;
+  bool hasFilters;

[scheib, 2016/11/22 21:22:44]

I don't think we need hasFilters in WebRequestDeviceOptions, just use
acceptAllDevices.

[ortuno, 2016/11/23 06:03:08]

hmm I was trying to make things in bluetooth_type_converters.cc more explicit.
If we remove hasFilters bluetooth_type_converters.cc becomes:

if (!options->accept_all_devices) {
  options->filters = mojo::Array<...>::From(web_options.filters);
}

as opposed to:

if (web_options.hasFilters) {
  options->filters = mojo::Array<...>::From(web_options.filters);
}

Both WebRequestDeviceOptions.cc and bluetooth_type_converters.cc are going away
with Onion Soup so I don't mind either way. Up to you.

   WebVector<WebString> optionalServices;
+  bool acceptAllDevices;
 };
 
 }  // namespace blink