Alison Shiue Reporter
	Description • 11 years ago

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"

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

Seems duplicated from Bug 959059, except this bug is in Homescreen.

	Wesley Huang [:wesley_huang] (EPM) (NI me)
	Comment 2 • 11 years ago

Greg, would you take this bug?

	Greg Weng [:snowmantw][:gweng][:λ]
	Comment 3 • 11 years ago

I think this is duplicated as Yoshi said. The root cause should be the same.

	Ken Chang[:kenkai]
	Comment 4 • 11 years ago

(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.

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

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.

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

I have a WIP patch in https://github.com/mozilla-b2g/gaia/pull/18740

	(inactive after 6/18) Alive Kuo [:alive]
	Comment 8 • 11 years ago

Comment on attachment 8414164 [details]
Pull Request

Please listen to homescreenopened event and do switch

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

Comment on attachment 8414164 [details]
Pull Request

use homescreenopened event.

	(inactive after 6/18) Alive Kuo [:alive]
	Comment 10 • 11 years ago

I strongly feel that unit test is necessary, filing a bug for shrinking ui test.

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

https://github.com/mozilla-b2g/gaia/commit/4c136121216eef32064aa8dcf341f843a35cb2ec

	Alison Shiue Reporter
	Comment 12 • 11 years ago

Verified on
Gaia      f55fc5c507312c7aac51ec9bb73061fd4ed5c5fb
Gecko     https://hg.mozilla.org/mozilla-central/rev/3285e030d9c0
BuildID   20140504160202
Version   32.0a1