1108973 - Check nfc-ndef-discovered MozActivity

Status: RESOLVED INVALID

(Firefox OS Graveyard :: NFC, defect)

Description

[Yoshi Cheng-Hao Huang [:allstars.chh][:allstarschh][:yoshi]]

nfc_manager.js in System app will fire a MozActivity whose name is nfc-ndef-discovered.
However I think the naming is inaccurate because that's no NDEF inside this MozActivity.

Comment 1

[Krzysztof Mioduszewski[:tauzen]]

nfc-ndef-discovered is fired if two conditions are met:
1. There is at least 1 NDEF record present in the system message received from Gecko
2. The 1st NDEF record in the message does not contain payload which is handled by gaia apps [1]

[1] - https://github.com/mozilla-b2g/gaia/blob/master/apps/system/js/nfc_manager.js#L488

Comment 2

[Krzysztof Mioduszewski[:tauzen]]

(Sorry submitted comment 1 by mistake, this is its continuation)

nfc-ndef-discovered contains decoded data of the first NDEF record as well as an array of all NDEF records (raw bytes). So it actually contains NDEF inside. 

Do you have some other name in mind for the activity? W3C in the latest spec introduces NfcMessageData [1], but they are considering situation in which the data will be decoded by the OS also from some proprietary data format, not only NDEF. In our case we handle only NDEF.

[1] http://w3c.github.io/nfc/index.html#the-nfcmessagedata-interface

Comment 3

[Yoshi Cheng-Hao Huang [:allstars.chh][:allstarschh][:yoshi]]

I probably didn't sleep well on that day. :/