783934 - [b2g-bluetooth] Add events onconnected and ondisconnected in BluetoothDevice

Status: RESOLVED FIXED

(Core :: DOM: Device Interfaces, defect)

Description

[Gina Yeh [:gyeh] [:ginayeh]]

As discussed in Bug778640, we'd like to create onConnected and onDisconnected events via event generator. The difference here is that, these events are implemented in BluetoothDevice rather than BluetoothApapter because only bluetooth device addresses are maintained in BluetoothAdapter. For Gaia side, we have to call getPairedDevices() first and an array of BluetoothDevices will be returned back, and then listen to onConnected and onDisconnected events.

Comment 1

[Gina Yeh [:gyeh] [:ginayeh]]

Attachment: v1: Add events onconnected and ondisconnected in BluetoothDevice

For gaia side, getPairedDevices() first and then listen every device for event onconnected and event ondisconnected.

Comment 2

[Gina Yeh [:gyeh] [:ginayeh]]

Attachment: Final version: Add events onconnected and ondisconnected in BluetoothDevice, r=qdot

Comment 3

[Blake Kaplan (:mrbkap) (inactive)]

Comment on attachment 654050 [details] [diff] [review]
Final version: Add events onconnected and ondisconnected  in BluetoothDevice, r=qdot

Review of attachment 654050 [details] [diff] [review]:
-----------------------------------------------------------------

::: dom/bluetooth/nsIDOMBluetoothDevice.idl
@@ +16,5 @@
>    readonly attribute bool connected;
>    readonly attribute bool paired;
>    attribute nsIDOMEventListener onpropertychanged;
> +  attribute nsIDOMEventListener onconnected;
> +  attribute nsIDOMEventListener ondisconnected;

Need to bump the IID for this interface.

Comment 4

[Gina Yeh [:gyeh] [:ginayeh]]

Attachment: Final version: Add events onconnected and ondisconnected in BluetoothDevice, r=qdot, sr=mrbkap

Comment 5

[Eric Chou [:ericchou] [:echou]]

https://hg.mozilla.org/integration/mozilla-inbound/rev/167fdd4db264

Comment 6

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/mozilla-central/rev/167fdd4db264