900874 - [System] Register new event handlers of BluetoothAdapter

Status: RESOLVED FIXED

(Firefox OS Graveyard :: Gaia::System, defect)

Description

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

Several system messages are going to be removed. Instead of binding message handler, new event handlers should be registered.

The following cases should be covered:
* bluetooth-hfp-status-changed -> adapter.onhfpstatuschanged
* bluetooth-sco-status-changed -> adapter.onscostatuschanged

The event consist of two properties:
- event.address
- event.status

Comment 1

[Arthur Chen [:arthurcc][inactive after 6/30]]

Attachment: link to https://github.com/mozilla-b2g/gaia/pull/11423

Evelyn, could you help review the change? I've done some basic tests and it worked as expected. Thanks!

Comment 2

[Evelyn Hung]

Comment on attachment 787427 [details]
link to https://github.com/mozilla-b2g/gaia/pull/11423

r=me with a nit addressed. Thanks!

Comment 3

[Arthur Chen [:arthurcc][inactive after 6/30]]

Comment addressed. Thanks for reviewing!

master: https://github.com/mozilla-b2g/gaia/commit/f401f46388588aff4d259b4f2559d1f4cb9aa6f5

Comment 5

[Nicolas B. Pierron [:nbp]]

Attachment: part 0.1 - Fix typo in nsScriptLoader::StartLoad.

Comment 6

[Nicolas B. Pierron [:nbp]]

Attachment: part 0.2 - Rename nsScriptLoadHandler::TryDecodeRawData to DecodeRawData.

As yoda would say: “do or do not, there is no try.”

Comment 7

[Nicolas B. Pierron [:nbp]]

Comment on attachment 8800627 [details] [diff] [review]
part 0.1 - Fix typo in nsScriptLoader::StartLoad.

(wrong bug)

Comment 8

[Nicolas B. Pierron [:nbp]]

Comment on attachment 8800631 [details] [diff] [review]
part 0.2 - Rename nsScriptLoadHandler::TryDecodeRawData to DecodeRawData.

(wrong bug)