959104 - [NFC] NFC doesn't work anymore when draw near and back two devices with one opened and one locked.

Status: VERIFIED FIXED

(Firefox OS Graveyard :: NFC, defect)

Description

[John Hu [:johnhu][:johu][:醬糊小弟]]

Reproduce prerequisite: two firefox os devices with NFC enabled

Summary:
   The behavior of NFC is not correct. After the test, the NFC of one device is no longer working and need to re-enable NFC through settings app.

STR:
1. enable NFC on both devices
2. lock one device with power button. The screen should be all black, not in the lock screen mode.
3. touch these two devices <=== Error: the opened device vibrates forever but the locked device not.
4. draw back two devices
5. touch two devices again <=== Error or Correct: The opened device vibrates once but the locked device not. This is the correct behavior when we have two opened devices, but not sure with this case.
6. draw back two devices
7. touch two devices again <=== Error: no more vibration.

After those 7 STR, the NFC is no long working at the locked device even if we unlock it. The opened device only vibrates when NFC tag leaving but not vibrates when tag touched.

Comment 1

[Yoshi Cheng-Hao Huang [:allstars.chh][:allstarschh][:yoshi]]

Depend on Bug 959109, we need to disable nfc when screen is off.

Comment 2

[John Hu [:johnhu][:johu][:醬糊小弟]]

(In reply to Yoshi Huang[:allstars.chh][:yoshi] from comment #1)
> Depend on Bug 959109, we need to disable nfc when screen is off.

If we disable NFC when screen is off, I believe this bug will be gone.

Comment 3

[Garner Lee]

Attachment: Check screen and lock state at the same time. r=alive

Moved from Bug 959109.

Comment 4

[Garner Lee]

Comment on attachment 8364289 [details] [review]
Check screen and lock state at the same time. r=alive

This is the same patch, moved from 959109 to 959104, since it directly fixes the problems in this particular bug, with additional edge case fixes coming from 959109.

Comment 5

[Garner Lee]

Travis tests passed.

Comment 6

[Ryan VanderMeulen [:RyanVM]]

Master: cc10bd50f93bbf7fdf40195c70ffa38eb075e6ec