Chromium Code Reviews| Index: public/platform/modules/bluetooth/WebBluetooth.h |
| diff --git a/public/platform/modules/bluetooth/WebBluetooth.h b/public/platform/modules/bluetooth/WebBluetooth.h |
| index ef959057e3abaf28907dbcdb46269235bbb0c937..5d9a9f730f1c11ae39a59051e23fcce3b0e8c7ac 100644 |
| --- a/public/platform/modules/bluetooth/WebBluetooth.h |
| +++ b/public/platform/modules/bluetooth/WebBluetooth.h |
| @@ -16,6 +16,7 @@ struct WebBluetoothError; |
| struct WebBluetoothGATTCharacteristic; |
| struct WebBluetoothGATTRemoteServer; |
| struct WebBluetoothGATTService; |
| +struct WebRequestDeviceOptions; |
| // Success and failure callbacks for requestDevice. |
| // WebBluetoothDevice and WebBluetoothError object ownership is transferred. |
| @@ -44,7 +45,10 @@ public: |
| // BluetoothDiscovery Methods: |
| // See https://webbluetoothcg.github.io/web-bluetooth/#idl-def-bluetoothdiscovery |
| // WebBluetoothRequestDeviceCallbacks ownership transferred to the client. |
| - virtual void requestDevice(WebBluetoothRequestDeviceCallbacks*) = 0; |
| + virtual void requestDevice(const WebRequestDeviceOptions&, WebBluetoothRequestDeviceCallbacks* callbacks) { requestDevice(callbacks); } |
| + // TODO(jyasskin): Remove this overload and make the other abstract after |
|
scheib
2015/06/15 18:08:11
It's OK to leave the default implementations. I us
Jeffrey Yasskin
2015/06/15 19:19:19
I definitely want to remove the second overload, a
scheib
2015/06/15 20:15:15
Not a strong preference, but as you can see the ne
|
| + // https://codereview.chromium.org/1172853004/ lands. |
| + virtual void requestDevice(WebBluetoothRequestDeviceCallbacks*) { } |
| // BluetoothDevice methods: |
| // See https://webbluetoothcg.github.io/web-bluetooth/#idl-def-bluetoothdevice |