Thomas Zimmermann [:tzimmermann] [:tdz] Assignee
	Description • 11 years ago

The session id in NFC messages is missing. The related error messages look like

> I/Gecko   (   46): Nfc Worker: Received DOM message {"requestId":"aWR7NWY0NDU0NzUtMWVjNS00YjNlLThmOTMtOWFjODljNTQzMjIwfQ==","sessionToken":"{959a7d96-b55b-4b53-9545-1d258a99f1f2}","techType":10,"type":"connect"}
> I/Gecko   (   46): Nfc Worker: New outgoing parcel of type 1
> I/Gecko   (   46): -*- Nfc: Received 'NFC:Connect' message from content process
> I/Gecko   (   46): Nfc Worker: Outgoing parcel: 0,0,0,12,1,0,0,0,0,0,0,0,10,0,0,0
> E/GeckoConsole(   46): [JavaScript Warning: "ReferenceError: reference to undefined property message.sessionId" {file: "resource://gre/modules/nfc_worker.js" line: 289}]

	Yoshi Cheng-Hao Huang [:allstars.chh][:allstarschh][:yoshi]
	Comment 2 • 11 years ago

Comment on attachment 8389209 [details] [diff] [review]
[01] Bug 982126: Set session ID in NFC messages

Review of attachment 8389209 [details] [diff] [review]:
-----------------------------------------------------------------

Nice catch, sorry I didn't notice this error before.

	Thomas Zimmermann [:tzimmermann] [:tdz] Assignee
	Comment 3 • 11 years ago

No problem, thanks for the quick review.

https://hg.mozilla.org/integration/b2g-inbound/rev/ebe5eb5b8d2a

https://tbpl.mozilla.org/?tree=B2g-Inbound&rev=ebe5eb5b8d2a

	Wes Kocher (:KWierso) (Not reading bugmail; email directly if needed)
	Comment 4 • 11 years ago

https://hg.mozilla.org/mozilla-central/rev/ebe5eb5b8d2a