bluetooth: Test Web Bluetooth getCharacteristic calls against blacklist.

content/browser/bluetooth/bluetooth_dispatcher_host.cc

Index: content/browser/bluetooth/bluetooth_dispatcher_host.cc
diff --git a/content/browser/bluetooth/bluetooth_dispatcher_host.cc b/content/browser/bluetooth/bluetooth_dispatcher_host.cc
index c4fe0562a92f8f147fdd61ae4439257845369cd6..2ed8471f3ed01409cfac8327abdfe1d71758bbca 100644
--- a/content/browser/bluetooth/bluetooth_dispatcher_host.cc
+++ b/content/browser/bluetooth/bluetooth_dispatcher_host.cc
@@ -792,6 +792,16 @@ void BluetoothDispatcherHost::OnGetCharacteristic(
   RecordWebBluetoothFunctionCall(UMAWebBluetoothFunction::GET_CHARACTERISTIC);
   RecordGetCharacteristicCharacteristic(characteristic_uuid);
 
+  // Check Blacklist for characteristic_uuid.
+  if (BluetoothBlacklist::Get().IsExcluded(

[ortuno, 2016/02/23 22:37:58]

You probably want to crash the renderer in QueryCacheForCharacteristic if the
UUID is blacklisted.

[scheib, 2016/02/24 19:51:28]

We discussed in person, and concluded that the initial blacklist checks will
result in service and characteristics not being found in respective lists and
also resulting in renderer crashes in QueryCache* methods if a blacklisted
resource is attempted to be accessed.

No code change made.

+          BluetoothUUID(characteristic_uuid))) {
+    RecordGetCharacteristicOutcome(UMAGetCharacteristicOutcome::BLACKLISTED);
+    Send(new BluetoothMsg_GetCharacteristicError(
+        thread_id, request_id,
+        WebBluetoothError::GetCharacteristicWithBlacklistedUUID));
+    return;
+  }
+
   const CacheQueryResult query_result =
       QueryCacheForService(GetOrigin(frame_routing_id), service_instance_id);