913340 - Provide mechanism to control which app (and child process) is using NFC

Status: RESOLVED FIXED

(Firefox OS Graveyard :: NFC, defect)

Description

[Paul Theriault [:pauljt] (no longer reading bugmail)]

Web NFC currently has a concept of sessions, but the session id broadcast to all child processes. As such apps with NFC permission can probably interfere with other apps that might be using it. This is a known issue, I am just raising this to track its resolution.

Comment 1

[Garner Lee]

I'm looking for a place to upload a patch for system/js/nfc_manager.js Gaia code. This a good place?

Or an existing "catch-all" bug, where it currently is: Bug 860910

-Garner

Comment 2

[Ken Chang[:kenkai]]

(In reply to Garner Lee from comment #1)
> I'm looking for a place to upload a patch for system/js/nfc_manager.js Gaia
> code. This a good place?
> 
> Or an existing "catch-all" bug, where it currently is: Bug 860910
> 
> -Garner
We can discuss this in next Monday meeting. The nfc manage may be a little different from current design.

Comment 3

[Kai-Chih Hu [:khu]]

Ken, is there any update that we can share / post here from your meeting? Thanks! 
And, who could be the best owner for this bug?

Comment 4

[Ken Chang[:kenkai]]

Garner is the right owner.

Comment 5

[Ken Chang[:kenkai]]

It seems better to finish this bug before sprint5. If you think it isn't reasonable, please update target milestone.

Comment 6

[Garner Lee]

The current landed DOM provides this functionality, via [ChromeOnly] setSessionToken(token) calls in MozNFCPeer and MozNFCTag. The app passes the same token to GetNFCPeer or GetNFCTag received from the Activity, and the call will register and associate that token with the app until it is invalid (NFC Tag presense is lost).

Close?

Comment 7

[Paul Theriault [:pauljt] (no longer reading bugmail)]

Sounds good to me - the point of this bug was to make it so one app at time can use NFC, and from our email discussion sounds like this is the case so nothing more to be done here I think.

I'll look at this code a bit deeper next week when I revisit the NFC review but this can be closed now I think.