[B2G][NFC] Handle unformatted NFC Tags in NFC Manager

RESOLVED FIXED in 1.4 S1 (14feb)

Status

RESOLVED FIXED
5 years ago
5 years ago

People

(Reporter: dgarnerlee, Assigned: dgarnerlee)

Tracking

unspecified
1.4 S1 (14feb)
x86_64
Linux
Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment, 1 obsolete attachment)

(Assignee)

Description

5 years ago
NFC (daemon, manager, etc). Doesn't gracefully handle completely unformatted tags currently, and relies on the app to know what to do given no feedback besides vibrating.

Why there's no record:
When nfcd sends technology discovered with no records, it indicates that the NDEF start bit "MB" is missing, meaning, it either isn't NDEF, or is unformatted. This is different from "Empty", which is an NDEF Record type.

This is primarily a writeNDEF use case: empty, unformatted tags. There is nothing to read on the tag. This also requires foreground dispatch (Bug 937430) to trigger a new NDEF message write.
(Assignee)

Updated

5 years ago
Blocks: 860906
Depends on: 937430
(Assignee)

Comment 1

5 years ago
Created attachment 8372702 [details] [diff] [review]
WIP: 0001-Update-NFC-Manager-to-handle-Tag-Discovered.patch

Handle generic "Tags" case better.
Blocks: 933640
No longer blocks: 860906
(Assignee)

Updated

5 years ago
Blocks: 963556
(Assignee)

Comment 2

5 years ago
Created attachment 8374502 [details] [review]
(v1) Update NFC Manager to handle Tag Discovered, empty tags, and unformatted tags r=alive
Assignee: nobody → dgarnerlee
Attachment #8372702 - Attachment is obsolete: true
Status: NEW → ASSIGNED
(Assignee)

Updated

5 years ago
Attachment #8374502 - Flags: review?(alive)
Attachment #8374502 - Flags: review?(alive) → review+
(Assignee)

Updated

5 years ago
Attachment #8374502 - Attachment description: (v1) Update NFC Manager to handle Tag Discovered, empty tags, and unformatted tags → (v1) Update NFC Manager to handle Tag Discovered, empty tags, and unformatted tags r=alive
(Assignee)

Comment 3

5 years ago
Travis fail on something in unrelated calendar sizing, giving it one more go...
(Assignee)

Comment 4

5 years ago
green!
Keywords: checkin-needed
Master: 9ceebbcda0a82e7b9205de9900ad14c74aa6350c
Status: ASSIGNED → RESOLVED
Last Resolved: 5 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → 1.4 S1 (14feb)
You need to log in before you can comment on or make changes to this bug.