Gaia-Rev 237008137f6d72b9cad25ff4faff14ff2c40ac63 Gecko-Rev https://hg.mozilla.org/releases/mozilla-b2g37_v2_2/rev/be24dd482a83 Build-ID 20150122162504 Version 37.0a2 Device-Name flame FW-Release 4.4.2 FW-Incremental eng.cltbld.20150122.194331 FW-Date Thu Jan 22 19:43:41 EST 2015 Bootloader L1TC000118D0 A phone with NFC, and a NFC-capable bluetooth earphone STR: 1. Enable NFC and Bluetooth in Settings 2. Tap the phone with NFC-capable bluetooth earphone 3. Check the confirmation dialog Expect result: Show "Are you sure you want to connect to XXXX?" Actual result: Show "Turn on Bluetooth and connect to XXXX?"
Please refer page 6.
Summary: [NFC] Incorrect BT confirm dialog when NFC headset detected while BT initial off → [NFC] Incorrect BT confirm dialog when NFC headset detected while BT initial ON
blocking-b2g: 2.2? → 2.2+
I will check this bug first. Thanks!
Assignee: nobody → shuang
Hmm...I cannot reproduce this bug. I did not see any extra pairing confirmation log.
This would be a gaia bug, and this issue is also reproducible on the master. |Bluetooth| module doesn't have |name| attribute , so |Service| doesn't have a correct mapping after |registerState()| , and |enabled| in |ncsd_setMessage()| is |undefined| as a result.  https://github.com/mozilla-b2g/gaia/blob/master/apps/system/js/bluetooth.js#L5  https://github.com/mozilla-b2g/gaia/blob/master/apps/system/js/bluetooth.js#L142  https://github.com/mozilla-b2g/gaia/blob/master/apps/system/js/service.js#L180  https://github.com/mozilla-b2g/gaia/blob/master/apps/system/js/system_nfc_connect_dialog.js#L21
Probably it is needed to use |BaseModule|  to implement |Bluetooth| module like other modules as well.  https://github.com/mozilla-b2g/gaia/blob/master/apps/system/js/base_module.js#L336
Hi Ian, would you mind having a look on it?
Assignee: shuang → nobody
Fred, could you please take a look the issue? Per comment 5,  was changed for decoupling `bluetooth`, `nfc_handler_manager` before. Looks like some regression might be happened here.
Flags: needinfo?(iliu) → needinfo?(flin)
Thanks bruce for tracing the issue. add `name = 'Bluetooth',` below https://github.com/mozilla-b2g/gaia/blob/master/apps/system/js/bluetooth.js#L5 might work. I'll get the NFC headset to test.
sorry I mean `name: 'Bluetooth',`
add name attribute to make Service.registerState works correctly. The related messages shows correctly with this patch (thanks Alison's help)
Attachment #8556317 - Flags: review?(alive)
Attachment #8556317 - Flags: review?(alive) → review+
Component: NFC → Gaia::System
Comment on attachment 8556317 [details] [review] pull request redirect to github [Approval Request Comment] [Bug caused by] (feature/regressing bug #): regress of Bug 1092893 [User impact] if declined: nfc headset with wrong message [Testing completed]: manual test passed [Risk to taking this patch] (and alternatives if risky): none [String changes made]: none
Attachment #8556317 - Flags: approval-gaia-v2.2?
this issue is also fixed by Bug 1092894 in master. But the patch is needed to uplift to 2.2
Status: NEW → RESOLVED
Last Resolved: 4 years ago
Resolution: --- → FIXED
No need for checkin-needed on uplifts. It'll show up on the right radar once it's approved.
That said, can you please post a 2.2-specific PR for this (or update the original one)? I don't know of any easy way to redirect an existing PR to a different branch than it was originally intended for.
Attachment #8556317 - Flags: approval-gaia-v2.2? → approval-gaia-v2.2+
Verified on [2.2] Gaia-Rev 21cce750c095a3f815275fe5439fa9dbfe3dfc6b Gecko-Rev https://hg.mozilla.org/releases/mozilla-b2g37_v2_2/rev/41ccc5328740 Build-ID 20150209162506 Version 37.0a2 Device-Name flame FW-Release 4.4.2 FW-Incremental eng.cltbld.20150209.194557 FW-Date Mon Feb 9 19:46:08 EST 2015 Bootloader L1TC000118D0 [3.0] Gaia-Rev 0cf517083f7eb5fc269e1236edba50534f65e3cd Gecko-Rev https://hg.mozilla.org/mozilla-central/rev/2cb22c058add Build-ID 20150209160204 Version 38.0a1 Device-Name flame FW-Release 4.4.2 FW-Incremental eng.cltbld.20150209.193059 FW-Date Mon Feb 9 19:31:10 EST 2015 Bootloader L1TC000118D0
You need to log in before you can comment on or make changes to this bug.