Bump Bluetooth max filter name length from 29 bytes to 240 bytes

content/browser/bluetooth/bluetooth_device_chooser_controller.cc

Index: content/browser/bluetooth/bluetooth_device_chooser_controller.cc
diff --git a/content/browser/bluetooth/bluetooth_device_chooser_controller.cc b/content/browser/bluetooth/bluetooth_device_chooser_controller.cc
index 19cad9b7c8ea02382046bf52fcc3db903a22a406..569c3e7e5e7af970731bd3d166972559054abda5 100644
--- a/content/browser/bluetooth/bluetooth_device_chooser_controller.cc
+++ b/content/browser/bluetooth/bluetooth_device_chooser_controller.cc
@@ -79,8 +79,8 @@ namespace content {
 bool BluetoothDeviceChooserController::use_test_scan_duration_ = false;
 
 namespace {
-constexpr size_t kMaxLengthForDeviceName =
-    29;  // max length of device name in filter.
+// Max length of device name in filter, limited to the max EIR packet size.
+constexpr size_t kMaxLengthForDeviceName = 240;

[jeffcarp, 2017/01/09 18:45:20]

In my head it made more sense to limit it to 240 bytes (not the 512 that was
discussed in the issue) since there's no way the name could be longer than 240.
Let me know if you prefer 512.

[scheib, 2017/01/09 19:26:49]

I think this is OK too, practically I doubt devices will be hitting this limit.
Please cite the spec sections and concepts that brought us to this number,
though.

 
 // The duration of a Bluetooth Scan in seconds.
 constexpr int kScanDuration = 60;