bluetooth: Implement acceptAllDevices

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

 // Copyright 2015 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 WebRequestDeviceOptions_h
 #define WebRequestDeviceOptions_h
 
 #include "public/platform/WebString.h"
 #include "public/platform/WebVector.h"
 
@@ skipping 12 matching lines @@
   WebString name;
   WebString namePrefix;
 };
 
 // Contains members corresponding to RequestDeviceOptions members as
 // specified in the IDL.
 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
 
 #endif  // WebRequestDeviceOptions_h