bluetooth: Browser-side implementation of readValue

content/browser/bluetooth/bluetooth_dispatcher_host.cc

 // Copyright 2014 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.
 
 #include "content/browser/bluetooth/bluetooth_dispatcher_host.h"
 
 #include "base/strings/utf_string_conversions.h"
 #include "content/browser/bad_message.h"
 #include "content/common/bluetooth/bluetooth_messages.h"
 #include "device/bluetooth/bluetooth_adapter.h"
@@ skipping 40 matching lines @@
 }
 
 bool BluetoothDispatcherHost::OnMessageReceived(const IPC::Message& message) {
   DCHECK_CURRENTLY_ON(BrowserThread::UI);
   bool handled = true;
   IPC_BEGIN_MESSAGE_MAP(BluetoothDispatcherHost, message)
   IPC_MESSAGE_HANDLER(BluetoothHostMsg_RequestDevice, OnRequestDevice)
   IPC_MESSAGE_HANDLER(BluetoothHostMsg_ConnectGATT, OnConnectGATT)
   IPC_MESSAGE_HANDLER(BluetoothHostMsg_GetPrimaryService, OnGetPrimaryService)
   IPC_MESSAGE_HANDLER(BluetoothHostMsg_GetCharacteristic, OnGetCharacteristic)
+  IPC_MESSAGE_HANDLER(BluetoothHostMsg_ReadValue, OnReadValue)
   IPC_MESSAGE_UNHANDLED(handled = false)
   IPC_END_MESSAGE_MAP()
   return handled;
 }
 
 void BluetoothDispatcherHost::SetBluetoothAdapterForTesting(
     scoped_refptr<device::BluetoothAdapter> mock_adapter) {
   DCHECK_CURRENTLY_ON(BrowserThread::UI);
   current_delay_time_ = kTestingDelayTime;
   set_adapter(mock_adapter.Pass());
@@ skipping 116 matching lines @@
       device->GetGattService(service_instance_id);
   if (!service) {
     Send(new BluetoothMsg_GetCharacteristicError(
         thread_id, request_id, BluetoothError::NETWORK_ERROR));
     return;
   }
 
   for (BluetoothGattCharacteristic* characteristic :
        service->GetCharacteristics()) {
     if (characteristic->GetUUID().canonical_value() == characteristic_uuid) {
+      const std::string& characteristic_instance_id =
+          characteristic->GetIdentifier();
+
+      auto insert_result = characteristic_to_service_.insert(
+          make_pair(characteristic_instance_id, service_instance_id));
+
+      // If the characteristic existed already check that service_instance_id is
+      // the same.
+      if (!insert_result.second)
+        DCHECK(insert_result.first->second == service_instance_id);
+
       // TODO(ortuno): Use generated instance ID instead.
       // https://crbug.com/495379
       Send(new BluetoothMsg_GetCharacteristicSuccess(
-          thread_id, request_id, characteristic->GetIdentifier()));
+          thread_id, request_id, characteristic_instance_id));
       return;
     }
   }
   Send(new BluetoothMsg_GetCharacteristicError(thread_id, request_id,
                                                BluetoothError::NOT_FOUND));
 }
 
+void BluetoothDispatcherHost::OnReadValue(
+    int thread_id,
+    int request_id,
+    const std::string& characteristic_instance_id) {
+  DCHECK_CURRENTLY_ON(BrowserThread::UI);
+
+  auto characteristic_iter =
+      characteristic_to_service_.find(characteristic_instance_id);
+  // A characteristic_instance_id not in the map implies a hostile renderer
+  // because a renderer obtains the characteristic id from this class and
+  // it will be added to the map at that time.
+  if (characteristic_iter == characteristic_to_service_.end()) {
+    // Kill the renderer
+    bad_message::ReceivedBadMessage(this,
+                                    bad_message::BDH_INVALID_CHARACTERISTIC_ID);
+    return;
+  }
+  const std::string& service_instance_id = characteristic_iter->second;
+
+  auto device_iter = service_to_device_.find(service_instance_id);
+  // A service_instance_id not in the map implies a hostile renderer
+  // because a renderer obtains the service id from this class and
+  // it will be added to the map at that time.
+  if (device_iter == service_to_device_.end()) {
+    // Kill the renderer
+    bad_message::ReceivedBadMessage(this, bad_message::BDH_INVALID_SERVICE_ID);
+    return;
+  }
+
+  device::BluetoothDevice* device =
+      adapter_->GetDevice(device_iter->second /* device_instance_id */);
+  if (device == nullptr) {
+    Send(new BluetoothMsg_ReadCharacteristicValueError(
+        thread_id, request_id, BluetoothError::NETWORK_ERROR));
+    return;
+  }
+
+  BluetoothGattService* service = device->GetGattService(service_instance_id);
+  if (service == nullptr) {
+    Send(new BluetoothMsg_ReadCharacteristicValueError(
+        thread_id, request_id, BluetoothError::NETWORK_ERROR));

[Jeffrey Yasskin, 2015/06/10 21:03:45]

Hm, I'm not sure this is notionally a network error. The device can drop some
services, or change the characteristics in a service, while a connection is
still there. We'd report that to the webpage with a serviceremoved or
servicechanged event, but we should also give a good error if the webpage misses
that event. I've filed
https://github.com/WebBluetoothCG/web-bluetooth/issues/130 to specify this.

This is also a place where we could really use the elaboration error string.

[ortuno, 2015/06/10 22:03:01]

Added a TODO for when the error is implemented.
Yes! That's happening in the next CL: http://crrev.com/1177613002 (not ready for
review).

+    return;
+  }
+
+  BluetoothGattCharacteristic* characteristic =
+      service->GetCharacteristic(characteristic_instance_id);
+  if (characteristic == nullptr) {
+    Send(new BluetoothMsg_ReadCharacteristicValueError(
+        thread_id, request_id, BluetoothError::NETWORK_ERROR));
+    return;
+  }
+
+  characteristic->ReadRemoteCharacteristic(
+      base::Bind(&BluetoothDispatcherHost::OnCharacteristicValueRead,
+                 weak_ptr_factory_.GetWeakPtr(), thread_id, request_id),
+      base::Bind(&BluetoothDispatcherHost::OnCharacteristicReadValueError,
+                 weak_ptr_factory_.GetWeakPtr(), thread_id, request_id));
+}
+
 void BluetoothDispatcherHost::OnDiscoverySessionStarted(
     int thread_id,
     int request_id,
     scoped_ptr<device::BluetoothDiscoverySession> discovery_session) {
   DCHECK_CURRENTLY_ON(BrowserThread::UI);
   BrowserThread::PostDelayedTask(
       BrowserThread::UI, FROM_HERE,
       base::Bind(&BluetoothDispatcherHost::StopDiscoverySession,
                  weak_ptr_factory_.GetWeakPtr(), thread_id, request_id,
                  base::Passed(&discovery_session)),
@@ skipping 104 matching lines @@
 
       Send(new BluetoothMsg_GetPrimaryServiceSuccess(thread_id, request_id,
                                                      service_identifier));
       return;
     }
   }
   Send(new BluetoothMsg_GetPrimaryServiceError(thread_id, request_id,
                                                BluetoothError::NOT_FOUND));
 }
 
+void BluetoothDispatcherHost::OnCharacteristicValueRead(
+    int thread_id,
+    int request_id,
+    const std::vector<uint8>& value) {
+  Send(new BluetoothMsg_ReadCharacteristicValueSuccess(thread_id, request_id,
+                                                       value));
+}
+
+void BluetoothDispatcherHost::OnCharacteristicReadValueError(
+    int thread_id,
+    int request_id,
+    device::BluetoothGattService::GattErrorCode) {
+  Send(new BluetoothMsg_ReadCharacteristicValueError(
+      thread_id, request_id, BluetoothError::NETWORK_ERROR));

[Jeffrey Yasskin, 2015/06/10 21:03:45]

https://webbluetoothcg.github.io/web-bluetooth/#error-handling says how to map
GATT errors to Javascript errors, and they're not all NetworkErrors.

[ortuno, 2015/06/10 22:03:01]

I added a TODO here as well. The CL that introduces more detailed error messages
 (http://crrev.com/1177613002) will have a function to get the message and the
error based on the
GattError code. Sounds good? 

[Jeffrey Yasskin, 2015/06/10 22:55:49]

Sure.

+}
+
 }  // namespace content