[NFC] NFC sharing does not work

VERIFIED FIXED in 2.0 S3 (6june)

Status

defect
VERIFIED FIXED
5 years ago
5 years ago

People

(Reporter: ashiue, Assigned: dimi)

Tracking

unspecified
2.0 S3 (6june)
ARM
Gonk (Firefox OS)
Dependency tree / graph

Firefox Tracking Flags

(blocking-b2g:2.0+, b2g-v2.0 fixed)

Details

(Whiteboard: [p=2])

Attachments

(2 attachments)

Posted file nfc_url_sharing.log
Gaia      b669dd2cc321f37cebc7081a79b968cac36b4200
Gecko     https://hg.mozilla.org/mozilla-central/rev/b85b57f05fda
BuildID   20140529160204
Version   32.0a1

STR:
1. enable NFC on both phones
2. open URL (or contact, files)
3. tap two phone, slide the shrinking UI to share

Expected result:
receiver can receive what sender sharing

Actual result:
receiver receive nothing
It worked before, right? When doesn't it work.
Yes, this problem should start from the following build:
Gaia      4142df90c71abdc1e3a87cd37dff1a22d5e38b34
Gecko     https://hg.mozilla.org/mozilla-central/rev/bc995dc1ea09
BuildID   20140528160204
Version   32.0a1
blocking-b2g: --- → 2.0+
Dimi, Do you find anything?
Flags: needinfo?(dlee)
I was looking into this while doing builds for my new flame device. I've made a build mentioned in Comment 2. Few findings:
- NDEF receiving works well. I've tested with Android device and could share contact and url from Android to Flame.
- There is a problem with sending NDEF. In the attached log you can see that NFC:WriteNDEF has wrong records without type and payload. While doing share from flame to Android it results in Unsupported tag message.

At first glance it seems like a problem with MozNDERRecord. I've started adding additional loging in the browser app and it seems MozNDEFRecord is empty immediately after being created, but need to confirm this after looking closely.
This issue happened because of TypedArray is changed in changeset:
http://hg.mozilla.org/mozilla-central/log/57b0932e2f06

I'll provide patch to fix it.
Flags: needinfo?(dlee)
Hi Jeff,
  This issue seems related to changeset mentioned in Comment 5.
Could you help review this patch ? Thanks!
Attachment #8433272 - Flags: review?(jwalden+bmo)
Comment on attachment 8433272 [details] [diff] [review]
Bug_1018068_v1.patch

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

Yeah this is fine.

Is this file only built in b2g configurations or something?  I had patchwork that renamed the Length/Data accessors, to be sure I'd adjusted every caller, that built fine for me locally (but which I never intended to land, of course).
Attachment #8433272 - Flags: review?(jwalden+bmo) → review+
From moz.build, yes:

if CONFIG['MOZ_NFC']

> Is this file only built in b2g configurations or something?
To be more precise, not every platform has NFC compiled in. gecko/configure.in has the conditional config per general platform, and device/<mfg>/<device name>/device.mk for runtime enabling of nfc for platforms that have NFC hardware:

PRODUCT_PROPERTY_OVERRIDES += \
        ro.moz.nfc.enabled=true
Hm. Its crashing on the new .ComputeLengthAndData() for me (type field): master head: 691bac9c467fe1385dbd2d1318f1b4be4e23d2c8
(In reply to Garner Lee from comment #10)
> Hm. Its crashing on the new .ComputeLengthAndData() for me (type field):
> master head: 691bac9c467fe1385dbd2d1318f1b4be4e23d2c8

Hi Garner,
  How did you reproduce this bug ? It works fine in my environment with latest master.
Flags: needinfo?(dgarnerlee)
Works ok on my flame with latest gecko master (cbe7e9c7e767ee6ce0459b519b4de2292b3cbd60).
Whiteboard: [p=2]
Target Milestone: --- → 2.0 S3 (6june)
> Hi Garner,
>   How did you reproduce this bug ? It works fine in my environment with
> latest master.

Yes, but only in a feature I'm working on. There's a sec bug open on it (I'll add you to it).
Flags: needinfo?(dgarnerlee)
https://hg.mozilla.org/mozilla-central/rev/7661bebd6d07
Status: NEW → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
Verified on
Device    Flame (Nexus 4)
Gaia      2e5636e852a9354a5f8072b179cf16f72647cfd6
Gecko     https://hg.mozilla.org/mozilla-central/rev/8bd92dc9ef59
BuildID   20140608160201
Version   32.0a1
Status: RESOLVED → VERIFIED
Doesn't this need backporting to all branches bug 991981 was checked in on?
Flags: needinfo?(dlee)
Those branches would be b2g30, b2g28, and b2g26, to be clear.
NFC feature is enabled only after 2.0, so we don't have to backporting for this issue.
Flags: needinfo?(dlee)
Partial NFC has been also applied into gecko v1.4(b2g30) branch. (I don't know how partial it's applied)
And browser app provided by gaia v1.4 is also supporting NFC feature.
So, this issue not only occur in v1.4 branch(b2g30) but also should be fixed.
You need to log in before you can comment on or make changes to this bug.