1125033 - [BlueDroid] Follow up bug for Bug 1059136 - expose BluetoothAdapter.uuids to v2.1 and v2.2

Status: RESOLVED FIXED

(Firefox OS Graveyard :: Bluetooth, defect)

Description

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

[Bluedroid] Expose Adapter Uuids fields, we currently missed to add uuid into PropertyChange event at the beginning of supporting bluedroid. This creates failure on verifying API test cases. Please refer to Bug 1116878

Comment 1

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

Attachment: (for 2.1) [bluedroid] Expose adapter uuids field. (v0)

Comment 2

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

Attachment: (for 2.1) [bluedroid] Expose adapter uuids field. (v1), r=shuang

- add reviewer's name to commit message
- convert to hg format

Thank Shawn for reviewing the patch.

Comment 3

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

[Blocking Requested - why for this release]:
Request for 2.1? since this bug blocks 2.1+ bug Bug 1116878.

Mozilla QA team is working on the release of MCTS for v2.1.
Please refer to https://bugzilla.mozilla.org/show_bug.cgi?id=970615#c44

The MCTS bluetooth tests would fail on v2.1 when it attempts to access 'BluetoothAdapter.uuids'. 'BluetoothAdapter.uuids' was exposed on Bug 1059136, however, the change didn't be applied to v2.1.
It blocks the release of MCTS v2.1 (Bug 970615 and Bug 1116878)

Comment 4

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

Comment on attachment 8553583 [details] [diff] [review]
(for 2.1) [bluedroid] Expose adapter uuids field. (v1), r=shuang

NOTE: Please see https://wiki.mozilla.org/Release_Management/B2G_Landing to better understand the B2G approval process and landings.

[Approval Request Comment]
> Bug caused by (feature/regressing bug #):
Bug 1059136 - [Bluedroid] Expose Adapter Bluetooth Uuids field

> User impact if declined:
This MDN page [1] indicates that BluetoothAdapter can get Bluetooth service uuids from 'BluetoothAdapter.uuids'. However, the value of 'uuids' is invalid on FxOS v2.1 and would cause a JavascriptException: 'NS_ERROR_FAILURE'.

[1] https://developer.mozilla.org/en-US/docs/Web/API/BluetoothAdapter

Although 'BluetoothAdapter.uuids' hasn't been used by any build-in gaia app yet, it still a part of Bluetooth Web API. It means it have to be tested by MCTS (Firefox OS Certification Testsuite). Currently, it would blocks Bug 1116878.

> Testing completed: 
Use this value in setting app and print the UUID through adb logcat.

> Risk to taking this patch (and alternatives if risky): 
'BluetoothAdapter.uuids' is a rarely used property and it'll be removed from Bluetooth API v2 [2] on FxOS v3.

[2] https://wiki.mozilla.org/B2G/Bluetooth/WebBluetooth-v2/BluetoothAdapter#BluetoothAdapter

> String or UUID changes made by this patch:
no

Comment 5

[bhavana bajaj [:bajaj]]

shawn is this not needed on central/b2g37 ?

Comment 6

[Shawn Huang [:shawnjohnjr][:shuang] (as a happy gecko contributor)]

In v3.0, this API no longer be exposed, but still need to patch for v2.2.
Jamin, can you confirm the patch can go into v2.2 directly or need rebase?

Comment 7

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

Attachment: (for 2.2) [bluedroid] Expose adapter uuids field. (v1), r=shuang

Comment 8

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

Attachment: (for 2.2) [bluedroid] Expose adapter uuids field. (v1), r=shuang

- rebase

Comment 9

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

(In reply to Shawn Huang [:shawnjohnjr] from comment #6)
> In v3.0, this API no longer be exposed, but still need to patch for v2.2.
> Jamin, can you confirm the patch can go into v2.2 directly or need rebase?

v2.1 patch can be cherry-pick to v2.2.
I've verified the patch on v2.2 by checking the value of BluetoothAdapter.uuid.

Comment 10

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

Comment on attachment 8555772 [details] [diff] [review]
(for 2.2) [bluedroid] Expose adapter uuids field. (v1), r=shuang

The patch was cherry-pick from v2.1 patch.
Copy [Approval Request Comment] from #comment 4

[Approval Request Comment]
> Bug caused by (feature/regressing bug #):
Bug 1059136 - [Bluedroid] Expose Adapter Bluetooth Uuids field

> User impact if declined:
This MDN page [1] indicates that BluetoothAdapter can get Bluetooth service uuids from 'BluetoothAdapter.uuids'. However, the value of 'uuids' is invalid on FxOS v2.1 and would cause a JavascriptException: 'NS_ERROR_FAILURE'.

[1] https://developer.mozilla.org/en-US/docs/Web/API/BluetoothAdapter

Although 'BluetoothAdapter.uuids' hasn't been used by any build-in gaia app yet, it still a part of Bluetooth Web API. It means it have to be tested by MCTS (Firefox OS Certification Testsuite). Currently, it would blocks Bug 1116878.

> Testing completed: 
Use this value in setting app and print the UUID through adb logcat.

> Risk to taking this patch (and alternatives if risky): 
'BluetoothAdapter.uuids' is a rarely used property and it'll be removed from Bluetooth API v2 [2] on FxOS v3.

[2] https://wiki.mozilla.org/B2G/Bluetooth/WebBluetooth-v2/BluetoothAdapter#BluetoothAdapter

> String or UUID changes made by this patch:
no

Comment 11

[howie [:howie]]

Hi Ryan, can we land this patch? Thanks.

Comment 12

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/releases/mozilla-b2g37_v2_2/rev/f98eb7d174df
https://hg.mozilla.org/releases/mozilla-b2g34_v2_1/rev/9a4ef05c858d

In the future, please make sure the bug number in the commit message matches the bug the patch is landing under.

Comment 13

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

(In reply to Ryan VanderMeulen [:RyanVM UTC-5] from comment #12)
> https://hg.mozilla.org/releases/mozilla-b2g37_v2_2/rev/f98eb7d174df
> https://hg.mozilla.org/releases/mozilla-b2g34_v2_1/rev/9a4ef05c858d
> 
> In the future, please make sure the bug number in the commit message matches
> the bug the patch is landing under.

I'm so sorry.
I once uploaded this patch to bug 1116878 and I forgot to change the commit message when I moved it to here. It's a terrible mistake.

Sorry for any trouble caused.

Comment 14

[孟宪茂]

Dear vance and shawn,

Please help land the change to v2.1s.

Thanks！

Comment 15

[Vance Chen [:vchen][skype:vance.lucida][[e-mail]]]

Hi Steven -

Could you help to make sure this one land on 2.1s?

Thanks

Comment 16

[Vincent Liu[:vliu]]

https://hg.mozilla.org/releases/mozilla-b2g34_v2_1s/rev/9a4ef05c858d