We need to have a clear definition of what does NFCTag.connect() and NFCTag.close() should do precisely. What does connect/close do? and what happened if we don't call them correctly. And should we really need them ? connect() maybe still needed as it is used to switch RF interface. But what about close? should we rename it to disconnect() ? If the app calls NFCTag.close(), actually NFC library will close the physical location to that tag, however since Tag is still in proximity another tech-discovered will be fired immediately, this will cause a bad user experience. And what will happen if we calls two connect()? like : tag.connect(NFC_A); // didn't call close before calling next connect() tag.connect(NFC_B);
(In reply to Yoshi Huang[:allstars.chh] from comment #0) > What does connect/close do? and what happened if we don't call them > correctly. > And should we really need them ? > > connect() maybe still needed as it is used to switch RF interface. > connect is used to switch to IsoDep or Frame RF interfaces. Currently we don't support IsoDep yet. See Bug 916428 > But what about close? should we rename it to disconnect() ? From Android http://developer.android.com/reference/android/nfc/tech/TagTechnology.html#close() This function actually calls NfcService.reconnect(); So back to our WebNFC API: Should we rename close to reconnect? And under what circumstance we will need this 'reconnect' ? > And what will happen if we calls two connect()? > like : > > tag.connect(NFC_A); // didn't call close before calling next connect() > tag.connect(NFC_B); In the example above, no error should be thrown, as NFC_A and NFC_B use the same Frame RF interface. However what should happen if tag.connect(NFC_A); tag.connect(ISO_DEP); Should the second connect could switch to IsoDep RF interface successfully? or it should throw because not calling close() before ?
I think it makes sense to track (re)connect/close() states, and throw an JS exception if it isn't closed before switching technologies so it can potentially also properly finish whatever it was doing. It may also encourage proper application cleanup (close ICC channels, etc.), for future feature development. This happen whether or not the application remembers whether it is connected or not.
(In reply to Garner Lee from comment #2) > I think it makes sense to track (re)connect/close() states, I think you mean 'connect/close(reconnect)' > and throw an JS > exception if it isn't closed before switching technologies Yeah, again, define 'close'. I think you're still talking 'reconnect' from my previous comment. However I agree that we need to throw error at this case. We need some TagTechnology support first, like NfcA or IsoDep(Bug 916428). But before they are supported, I think NFCTag.connect/close are not neccesary.
Depends on: 916428
yeah, typo. sorry for my fat finger. :P
These functions are actually useless now, since when the tag is discovered nfcd already connects to it. So in this bug I'll remove these two interfaces(connect/close) from MozNFCTag.webidl, but still keep the internal impl because we might still need this when we support multi-protocol tags.
Summary: B2G NFC: Have a clear definition with NFCTag.connect and NFCTag.close → B2G NFC: remove NFCTag.connect and NFCTag.close
NFCTechType will still be used in Bug 1057264, so I don't remove them for now.
Created attachment 8477275 [details] [diff] [review] Patch
Attachment #8477275 - Flags: review?(dlee)
Comment on attachment 8477275 [details] [diff] [review] Patch Hi, Smaug Jonas has asked the purpose of these two interfaces earlier this year. These two interfaces are added when WebNFC firstly landed Bug 674741, and actually it uses the same interface from Android. See  After investigation we found that these interfaces are used to switch RF interface for different Technologies. However right now we didn't support any NFC Tag Technology yet, and nfc daemon already doing the RF-connect when the tag is discovered.(Like when a MozNFCPeer is discoverd, nfcd will connect to it by itself). So actully the two interaces are dummy, so we propose to remove these two. : http://developer.android.com/reference/android/nfc/tech/TagTechnology.html#connect()
Attachment #8477275 - Flags: review?(bugs)
Attachment #8477275 - Flags: review?(bugs) → review+
Created attachment 8479842 [details] [diff] [review] Patch Add r=smaug, dimi
add checkin-needed for b2g-inbound is close.
Status: NEW → RESOLVED
Last Resolved: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → 2.1 S3 (29aug)
blocking-b2g: backlog → ---
tracking-b2g: --- → backlog
You need to log in before you can comment on or make changes to this bug.