Closed Bug 996522 Opened 11 years ago Closed 11 years ago

[NFC] Home screen become black with wording "Swipe to share" when objects which can be shared (such as browser, video, or image) running on background

Categories

(Firefox OS Graveyard :: NFC, defect)

Product:
Firefox OS Graveyard
Component:
NFC
Platform:
ARM
Gonk (Firefox OS)
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

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

Status:
VERIFIED FIXED
Milestone:
2.0 S1 (9may)
Project Flags:
blocking-b2g 2.0+
Tracking Flags:
Tracking Status
b2g-v2.0 --- fixed

People

(Reporter: ashiue, Assigned: allstars.chh)

References

Details

Attachments

(2 files)

S__335944.jpg
(deleted), image/jpeg
Details
Pull Request
(deleted), text/x-github-pull-request
alive
: review+
Details
Attached image S__335944.jpg (deleted) —
Using most up-to-date pvt build (2014/4/14) to test Two phones(Device A, Device B) with NFC STR: 1. Flash up-to-date pvt build (2014/4/14) 2. Enable NFC in Settings 3. Device A open browser, and click home button to let browser running on background 4. Touch two phones together 5. Check Device A Expected result: Nothing different Actual result: Home screen become black with wording "Swipe to share"
Seems duplicated from Bug 959059, except this bug is in Homescreen.
Greg, would you take this bug?
blocking-b2g: --- → 2.0+
Flags: needinfo?(gweng)
I think this is duplicated as Yoshi said. The root cause should be the same.
Flags: needinfo?(gweng)
(In reply to Greg Weng [:snowmantw][:gweng][:λ] from comment #3) > I think this is duplicated as Yoshi said. The root cause should be the same. Set it as a duplicate of Bug 959059.
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → DUPLICATE
Checking again, this is a different bug from Bug 959059, which is about the shrinking-stop event seems to be ignored. This bug is why Shrinking UI will receive the 'shrinking-start' event from Homescreen app. I've reproduced this bug and discussed with Greg, this is a bug in the Shrinking UI. When the browser is launched, shrinking_ui.js will get 'appopen' event, and stored the evt.detail.manifestURL to this.currentAppURL. Then when Home key is pressed, the 'home' event will be passed, however the this.currentAppURL hasn't been updated. Now another NFC device is in proximity, nfc_manager will received tech-discovered event, and send 'check-p2p-registration-for-active-app' to shrinking_ui.js, then shrinking_ui will call checkP2PRegistration, 'with this.currentAppURL', which is wrong here because this.currentAppURL is still the manifest of the Browser app, instead of Homescreen app. After discussed with Greg, he or other staff will fix this, assign this to him first.
Assignee: nobody → gweng
Status: RESOLVED → REOPENED
Resolution: DUPLICATE → ---
Component: NFC → Gaia::System
I have a WIP patch in https://github.com/mozilla-b2g/gaia/pull/18740
Assignee: gweng → allstars.chh
Attached file Pull Request (deleted) —
Attachment #8414164 - Flags: review?(alive)
Status: REOPENED → ASSIGNED
Component: Gaia::System → NFC
Comment on attachment 8414164 [details] Pull Request Please listen to homescreenopened event and do switch
Attachment #8414164 - Flags: review?(alive)
Target Milestone: --- → 2.0 S1 (9may)
Comment on attachment 8414164 [details] Pull Request use homescreenopened event.
Attachment #8414164 - Flags: review?(alive)
Attachment #8414164 - Flags: review?(alive) → review+
I strongly feel that unit test is necessary, filing a bug for shrinking ui test.
https://github.com/mozilla-b2g/gaia/commit/4c136121216eef32064aa8dcf341f843a35cb2ec
Status: ASSIGNED → RESOLVED
Closed: 11 years ago → 11 years ago
Resolution: --- → FIXED
Verified on Gaia f55fc5c507312c7aac51ec9bb73061fd4ed5c5fb Gecko https://hg.mozilla.org/mozilla-central/rev/3285e030d9c0 BuildID 20140504160202 Version 32.0a1
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: