[NFC] Techlist should be included with the NFC Activity

RESOLVED FIXED in 2.0 S5 (4july)

Status

Firefox OS
NFC
P3
normal
RESOLVED FIXED
4 years ago
4 years ago

People

(Reporter: Garner Lee, Assigned: tauzen)

Tracking

unspecified
2.0 S5 (4july)
x86_64
Linux
Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Reporter)

Description

4 years ago
NFC Activities are launched based on the activity type and filters on those types, and also the user selection on the Activity Picker for multiple matches. Currently only the NFC "tech" is forwarded, not the entire list.

With planned support for more NFC technology types, NFC Manager should include the entire techList it received from the NFC Daemon (nfcd) so applications can decide to interface with the tag with different compatible protocols than initially presented.

Prerequisites: WebNFC APIs must include support for those extra protocols.
(Reporter)

Updated

4 years ago
Priority: -- → P3
(Assignee)

Updated

4 years ago
Assignee: nobody → kmioduszewski
(Assignee)

Updated

4 years ago
Blocks: 933640
(Assignee)

Updated

4 years ago
Depends on: 1006375
(Assignee)

Comment 1

4 years ago
Created attachment 8448504 [details] [review]
pull-request-1017111.txt

Additionally this change is required to have the whole info about the discovered tag. When we were sending the tech with highest priority we were loosing the info that a tag is writeable (NDEF_WRITEABLE flag).

Try results: https://tbpl.mozilla.org/?tree=Gaia-Try&rev=b75b2c18934abfd45ac69a91c5b31358e43b9933

Travis has failed with two unrelated errors - clock unit tests error and marionette part 1 music player error.
Attachment #8448504 - Flags: review?(gweng)
Comment on attachment 8448504 [details] [review]
pull-request-1017111.txt

Okay, it looks good to me. Thanks.
Attachment #8448504 - Flags: review?(gweng) → review+
(Assignee)

Updated

4 years ago
Keywords: checkin-needed
master: https://github.com/mozilla-b2g/gaia/commit/b0ddb2122771e25cdd880359406a53b10cdc0507
Status: NEW → RESOLVED
Last Resolved: 4 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → 2.0 S5 (4july)
You need to log in before you can comment on or make changes to this bug.