Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(690)

Unified Diff: third_party/WebKit/Source/modules/bluetooth/BluetoothRemoteGATTCharacteristic.cpp

Issue 2615093002: Typemap WebBluetoothDeviceId to WTF::String (Closed)
Patch Set: merge master Created 3 years, 11 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: third_party/WebKit/Source/modules/bluetooth/BluetoothRemoteGATTCharacteristic.cpp
diff --git a/third_party/WebKit/Source/modules/bluetooth/BluetoothRemoteGATTCharacteristic.cpp b/third_party/WebKit/Source/modules/bluetooth/BluetoothRemoteGATTCharacteristic.cpp
index a1614922a580ceaeb9131f26fa615683cbbd51c5..9a557735190a9c677f2436db682a1c7318dae5a2 100644
--- a/third_party/WebKit/Source/modules/bluetooth/BluetoothRemoteGATTCharacteristic.cpp
+++ b/third_party/WebKit/Source/modules/bluetooth/BluetoothRemoteGATTCharacteristic.cpp
@@ -16,7 +16,7 @@
#include "modules/bluetooth/BluetoothDevice.h"
#include "modules/bluetooth/BluetoothError.h"
#include "modules/bluetooth/BluetoothRemoteGATTService.h"
-#include <memory>
+#include <utility>
namespace blink {
@@ -42,35 +42,28 @@ DOMDataView* ConvertWTFVectorToDataView(const Vector<uint8_t>& wtfVector) {
BluetoothRemoteGATTCharacteristic::BluetoothRemoteGATTCharacteristic(
ExecutionContext* context,
- const String& characteristicInstanceId,
const String& serviceInstanceId,
- const String& uuid,
- uint32_t characteristicProperties,
+ mojom::blink::WebBluetoothRemoteGATTCharacteristicPtr characteristic,
BluetoothRemoteGATTService* service,
BluetoothDevice* device)
: ContextLifecycleObserver(context),
- m_characteristicInstanceId(characteristicInstanceId),
m_serviceInstanceId(serviceInstanceId),
- m_uuid(uuid),
- m_characteristicProperties(characteristicProperties),
+ m_characteristic(std::move(characteristic)),
m_service(service),
m_stopped(false),
m_device(device) {
m_properties =
- BluetoothCharacteristicProperties::create(m_characteristicProperties);
+ BluetoothCharacteristicProperties::create(m_characteristic->properties);
}
BluetoothRemoteGATTCharacteristic* BluetoothRemoteGATTCharacteristic::create(
ExecutionContext* context,
- const String& characteristicInstanceId,
const String& serviceInstanceId,
- const String& uuid,
- uint32_t characteristicProperties,
+ mojom::blink::WebBluetoothRemoteGATTCharacteristicPtr characteristic,
BluetoothRemoteGATTService* service,
BluetoothDevice* device) {
return new BluetoothRemoteGATTCharacteristic(
- context, characteristicInstanceId, serviceInstanceId, uuid,
- characteristicProperties, service, device);
+ context, serviceInstanceId, std::move(characteristic), service, device);
}
void BluetoothRemoteGATTCharacteristic::setValue(DOMDataView* domDataView) {
@@ -95,7 +88,7 @@ void BluetoothRemoteGATTCharacteristic::notifyCharacteristicObjectRemoved() {
if (!m_stopped) {
m_stopped = true;
m_device->bluetooth()->characteristicObjectRemoved(
- m_characteristicInstanceId);
+ m_characteristic->instance_id);
}
}
@@ -117,7 +110,7 @@ void BluetoothRemoteGATTCharacteristic::addedEventListener(
// listeners have been removed. See http://crbug.com/541390
if (eventType == EventTypeNames::characteristicvaluechanged) {
m_device->bluetooth()->registerCharacteristicObject(
- m_characteristicInstanceId, this);
+ m_characteristic->instance_id, this);
}
}
@@ -156,7 +149,7 @@ ScriptPromise BluetoothRemoteGATTCharacteristic::readValue(
DOMException::create(NetworkError, kGATTServerNotConnected));
}
- if (!gatt()->device()->isValidCharacteristic(m_characteristicInstanceId)) {
+ if (!gatt()->device()->isValidCharacteristic(m_characteristic->instance_id)) {
return ScriptPromise::rejectWithDOMException(
scriptState,
DOMException::create(InvalidStateError, kInvalidCharacteristic));
@@ -168,7 +161,7 @@ ScriptPromise BluetoothRemoteGATTCharacteristic::readValue(
mojom::blink::WebBluetoothService* service = m_device->bluetooth()->service();
service->RemoteCharacteristicReadValue(
- m_characteristicInstanceId,
+ m_characteristic->instance_id,
convertToBaseCallback(
WTF::bind(&BluetoothRemoteGATTCharacteristic::ReadValueCallback,
wrapPersistent(this), wrapPersistent(resolver))));
@@ -210,7 +203,7 @@ ScriptPromise BluetoothRemoteGATTCharacteristic::writeValue(
DOMException::create(NetworkError, kGATTServerNotConnected));
}
- if (!gatt()->device()->isValidCharacteristic(m_characteristicInstanceId)) {
+ if (!gatt()->device()->isValidCharacteristic(m_characteristic->instance_id)) {
return ScriptPromise::rejectWithDOMException(
scriptState,
DOMException::create(InvalidStateError, kInvalidCharacteristic));
@@ -237,7 +230,7 @@ ScriptPromise BluetoothRemoteGATTCharacteristic::writeValue(
mojom::blink::WebBluetoothService* service = m_device->bluetooth()->service();
service->RemoteCharacteristicWriteValue(
- m_characteristicInstanceId, valueVector,
+ m_characteristic->instance_id, valueVector,
convertToBaseCallback(WTF::bind(
&BluetoothRemoteGATTCharacteristic::WriteValueCallback,
wrapPersistent(this), wrapPersistent(resolver), valueVector)));
@@ -276,7 +269,7 @@ ScriptPromise BluetoothRemoteGATTCharacteristic::startNotifications(
DOMException::create(NetworkError, kGATTServerNotConnected));
}
- if (!gatt()->device()->isValidCharacteristic(m_characteristicInstanceId)) {
+ if (!gatt()->device()->isValidCharacteristic(m_characteristic->instance_id)) {
return ScriptPromise::rejectWithDOMException(
scriptState,
DOMException::create(InvalidStateError, kInvalidCharacteristic));
@@ -288,7 +281,7 @@ ScriptPromise BluetoothRemoteGATTCharacteristic::startNotifications(
mojom::blink::WebBluetoothService* service = m_device->bluetooth()->service();
service->RemoteCharacteristicStartNotifications(
- m_characteristicInstanceId,
+ m_characteristic->instance_id,
convertToBaseCallback(
WTF::bind(&BluetoothRemoteGATTCharacteristic::NotificationsCallback,
wrapPersistent(this), wrapPersistent(resolver))));
@@ -305,7 +298,7 @@ ScriptPromise BluetoothRemoteGATTCharacteristic::stopNotifications(
DOMException::create(NetworkError, kGATTServerNotConnected));
}
- if (!gatt()->device()->isValidCharacteristic(m_characteristicInstanceId)) {
+ if (!gatt()->device()->isValidCharacteristic(m_characteristic->instance_id)) {
return ScriptPromise::rejectWithDOMException(
scriptState,
DOMException::create(InvalidStateError, kInvalidCharacteristic));
@@ -317,7 +310,7 @@ ScriptPromise BluetoothRemoteGATTCharacteristic::stopNotifications(
mojom::blink::WebBluetoothService* service = m_device->bluetooth()->service();
service->RemoteCharacteristicStopNotifications(
- m_characteristicInstanceId,
+ m_characteristic->instance_id,
convertToBaseCallback(
WTF::bind(&BluetoothRemoteGATTCharacteristic::NotificationsCallback,
wrapPersistent(this), wrapPersistent(resolver),

Powered by Google App Engine
This is Rietveld 408576698