Closed Bug 1111523 Opened 7 years ago Closed 7 years ago

isP2P and sessionToken shouldn't exist in nfc-manager-tech-discover wrapper

Categories

(Firefox OS Graveyard :: NFC, defect)

ARM
Gonk (Firefox OS)
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED
2.2 S2 (19dec)

People

(Reporter: allstars.chh, Assigned: allstars.chh)

References

Details

(Whiteboard: [p=1])

Attachments

(1 file)

Since we use 'peer' property in nfc-manager-tech-discovered message wrapper, I think isP2P and sessionToken properties should be required.

Also I notice a warning message
12-15 02:23:00.970: W/GeckoConsole(1790): [JavaScript Warning: "ReferenceError: reference to undefined property message.records" {file: "jar:file:///system/b2g/omni.ja!/components/Nfc.js" line: 481}]

will also fix this.
Attachment #8536457 - Flags: review?(dlee) → review+
https://hg.mozilla.org/integration/b2g-inbound/rev/b89d2bdab0e4
Whiteboard: [p=1]
Target Milestone: --- → 2.2 S2 (19dec)
https://hg.mozilla.org/mozilla-central/rev/b89d2bdab0e4
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.