Closed
Bug 933678
Opened 11 years ago
Closed 11 years ago
B2G NFC: Add exposedProps for those objects passed to Gaia.
Categories
(Firefox OS Graveyard :: NFC, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
1.4 S4 (28mar)
People
(Reporter: allstars.chh, Assigned: allstars.chh)
References
Details
Attachments
(1 file)
(deleted),
patch
|
dimi
:
review+
|
Details | Diff | Splinter Review |
In Bug 674741 we discussed there should be 'exposedProps' for those objects passed to Gaia, although on device it seems working well.
https://bugzilla.mozilla.org/show_bug.cgi?id=674741#c364
This bug is filed to discuss if we need to add exposedProps back to those objects.
Updated•11 years ago
|
Blocks: backlog-RIL/Net/Conn
Assignee | ||
Updated•11 years ago
|
Assignee: nobody → allstars.chh
Assignee | ||
Comment 1•11 years ago
|
||
Right now there will be two responses will return object,
one is readNDEF, and the other is getDetailsNDEF.
The object returned in readNDEF is MozNDEFRecord, which is wrapped by XrayWrapper, so we don't need __exposedProps__ here.
However the object returned in getDetailsNDEF is a plain JS object, hence __exposedProps__ is needed here.
Assignee | ||
Comment 2•11 years ago
|
||
Assignee | ||
Updated•11 years ago
|
Attachment #8378909 -
Flags: review?(dlee)
Updated•11 years ago
|
Attachment #8378909 -
Flags: review?(dlee) → review+
Assignee | ||
Comment 3•11 years ago
|
||
ni? to myself to push this after 2/28
Flags: needinfo?(allstars.chh)
Assignee | ||
Comment 4•11 years ago
|
||
Flags: needinfo?(allstars.chh)
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Target Milestone: --- → 1.4 S4 (28mar)
You need to log in
before you can comment on or make changes to this bug.
Description
•