1060216 - [Bluedroid] BluetoothAdapter.discovering wouldn't be updated after calling startDisocovery or stopDisocovery.

Status: RESOLVED FIXED

(Firefox OS Graveyard :: Bluetooth, defect)

Description

[Jamin Liu [:jaliu][:Jamin]]

BluetoothAdapter.discovering wouldn't be updated after calling startDisocovery or stopDisocovery with Bluedroid stack.
This creates failure on verifying API test cases.

Comment 1

[Jamin Liu [:jaliu][:Jamin]]

Attachment: (for 2.1) Distribute "PropertyChanged" signal to notify BluetoothAdapter when discovery state changed. (v1)

Comment 2

[Jamin Liu [:jaliu][:Jamin]]

Attachment: (for 2.0 & 1.4) Distribute "PropertyChanged" signal to notify BluetoothAdapter when discovery state changed. (v1)

Comment 3

[Jamin Liu [:jaliu][:Jamin]]

Attachment: (for 2.1) Distribute "PropertyChanged" signal to notify BluetoothAdapter when discovery state changed. (v1), r=shuang

Comment 4

[Jamin Liu [:jaliu][:Jamin]]

Attachment: (for 2.0 & 1.4) Distribute "PropertyChanged" signal to notify BluetoothAdapter when discovery state changed. (v1), r=shuang

Comment 5

[Jamin Liu [:jaliu][:Jamin]]

Try server:
https://tbpl.mozilla.org/?tree=Try&rev=f89fae5c1209

Comment 6

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/integration/mozilla-inbound/rev/803b392f6972

Comment 7

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/mozilla-central/rev/803b392f6972

Comment 8

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/releases/mozilla-b2g30_v1_4/rev/50895ee22e5e

Please request b2g32 approval on the patch if it needs uplifting to v2.0 as well.

Comment 9

[Jamin Liu [:jaliu][:Jamin]]

Comment on attachment 8481162 [details] [diff] [review]
(for 2.0 & 1.4) Distribute "PropertyChanged" signal to notify BluetoothAdapter when discovery state changed. (v1), r=shuang

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

Request for b2g32 approval to uplift this patch to v2.0.

This patch was landed to v2.1 and v1.4 recently since MCTS (Mozilla FirefoxOS Certification Test Suite) would fail on KK-based device without this patch.
It's required on v2.0 for passing MCTS on KK-based device which uses Bluedroid stack. 
As far as I know, we will need to run MCTS on v2.0 product in future.
Therefore, I'd like to request for b2g32 approval.

Comment 10

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/releases/mozilla-b2g32_v2_0/rev/cfe694790dec

Comment 11

[Kai-Zhen Li [:seinlin][:kli]]

http://hg.mozilla.org/releases/mozilla-b2g32_v2_0m/rev/06bd8d34010b