Closed Bug 1021213 Opened 10 years ago Closed 10 years ago

[B2G][NFC] The browser will remain in landscape mode after bumping with another NFC phone

Categories

(Firefox OS Graveyard :: NFC, defect)

ARM
Gonk (Firefox OS)
defect
Not set
normal

Tracking

(blocking-b2g:2.0+, b2g-v1.4 unaffected, b2g-v2.0 verified, b2g-v2.1 verified)

VERIFIED FIXED
2.0 S4 (20june)
blocking-b2g 2.0+
Tracking Status
b2g-v1.4 --- unaffected
b2g-v2.0 --- verified
b2g-v2.1 --- verified

People

(Reporter: KTucker, Assigned: allstars.chh)

References

Details

(Whiteboard: [2.0-flame-test-run-1][p=1])

Attachments

(3 files)

Attached file Flamelogcat.txt (deleted) —
Description: If the user changes the phone to landscape mode, bumps with another NFC device and then pulls the phone away, the browser will remain in landscape mode. Repro Steps: 1) Updated Flame to Build ID: 20140605040202 2) Connect two phones using Bluetooth. 3) Enable NFC on both phones. 4) Open the browser on Device A. 5) Bump the phones together so vibration is felt. 6) Flip Device A to landscape mode. 7) Pull devices apart and flip device A back to portrait mode. Actual: The browser will be stuck in landscape mode and will not revert back to portrait mode. Expected: The browser will change orientation without issue. Environmental Variables Device: Flame v 2.0 Build ID: 20140605040202 Gecko: https://hg.mozilla.org/mozilla-central/rev/51b428be6213 Gaia: d2cfef555dabab415085e548ed44c48a99be5c32 Platform Version: 32.0a1 Firmware Version: v10G-2 User Agent: Mozilla/5.0 (Mobile; rv:32.0) Gecko/32.0 Firefox/32.0 Notes: Repro frequency: 100% Link to failed test case: See attached: video clip, logcat Please note there is an issue with the Flame logs https://bugzilla.mozilla.org/show_bug.cgi?id=1010993 Unfortunately, the Buri and Open C phones do not have NFC so i cannot pull a log from those devices.
This bug could fail this test case https://moztrap.mozilla.org/manage/case/11495/ if the user takes it a couple steps further by moving the phone away and then rotating the phone to portrait mode. This issue does not reproduce on the Flame 1.4: 1.4 Environmental Variables: Device: Flame 1.4 BuildID: 20140530000202 Gaia: fe612fd21389193a8e593aa718831602e5086a62 Gecko: 25011f9a8f26 Version: 30.0 Firmware Version: v10G-2 User Agent: Mozilla/5.0 (Mobile; rv:30.0) Gecko/30.0 Firefox/30.0 The browser rotates back to portrait correctly without issue after bumping the phones in landscape mode. The issue does not reproduce on the Flame base v10G-2: 1.3 Environmental Variables: Device: Flame 1.3 BuildID: 20140520094859 Gaia: a73235d23685e9898f40647cebd83b3fcbfd0117 Gecko: Unknown Version: 28.0 Firmware Version: v10G-2 User Agent: Mozilla/5.0 (Mobile; rv:28.0) Gecko/28.0 Firefox/28.0 The phones will vibrate when bumped with another NFC phone but the UI does not shrink to indicate the user can share a URL and the browser does not get stuck in landscape mode. This issue does not reproduce on the Open C 2.0 or the Buri 2.0: Open C 2.0 2.0 Environmental Variables: Device: Open C 2.0 BuildID: 20140530040207 Gaia: 26d8fcab9b61f46451600f39c51e03 87ef3c4f88 Gecko: e6f113c83095 Version: 32.0a1 Firmware Version: P821A10V1.0.0B06_LOG_DL User Agent: Mozilla/5.0 (Mobile; rv:32.0) Gecko/32.0 Firefox/32.0 Buri 2.0 2.0 Environmental Variables: Device: Buri 2.0 MOZ BuildID: 20140530040207 Gaia: 26d8fcab9b61f46451600f39c51e0387ef3c4f88 Gecko: e6f113c83095 Version: 32.0a1 Firmware Version: v1.2-device.cfg User Agent: Mozilla/5.0 (Mobile; rv:32.0) Gecko/32.0 Firefox/32.0 NFC is not an available option on the Open C phone and the Buri phone is not NFC capable.
Youtube video can be found here: http://youtu.be/i8x7raIqlzg
Alison - Can you triage this & weigh in on the severity of this bug from a NFC QA perspective?
Flags: needinfo?(ashiue)
I don't think I'd call this is a regression, since NFC is only enabled on 2.0 or later.
Keywords: regression
Not only browser but also gallery have the same problem. I think this is a P2 issue would be fixed in 2.0.
blocking-b2g: --- → 2.0?
Flags: needinfo?(ashiue)
Assignee: nobody → allstars.chh
Attached file PullRequest (deleted) —
Attachment #8437588 - Flags: review?(alive)
From Product perspective, we need to fix this issue in 2.0 as the orientation seems to get stuck even after the NFC use case completes. Users will see this after sharing any content even once.
Comment on attachment 8437588 [details] PullRequest This is incorrect. You should let orientationManager to reset the orientation after the shrinking-stop.
Attachment #8437588 - Flags: review?(alive) → review-
Comment on attachment 8437588 [details] PullRequest Find out orientation_manager didn't listen to shrinking-rejected event.
Attachment #8437588 - Flags: review- → review?(alive)
Comment on attachment 8437588 [details] PullRequest r+ if you have unit test.
Attachment #8437588 - Flags: review?(alive) → review+
Comment on attachment 8437588 [details] PullRequest test case added, will merge when travis is green.
blocking-b2g: 2.0? → 2.0+
Verified on Gaia a3a5322692578e0a577fb7fa08e32144b2b05ba3 Gecko https://hg.mozilla.org/releases/mozilla-aurora/rev/0293597de41f BuildID 20140612160201 Version 32.0a2
Status: RESOLVED → VERIFIED
Whiteboard: [2.0-flame-test-run-1] → [2.0-flame-test-run-1][p=1
Whiteboard: [2.0-flame-test-run-1][p=1 → [2.0-flame-test-run-1][p=1]
This issue has been verified successfully on Flame 2.1. See attachment: 1645.MP4 Reproducing rate: 0/5 Steps: 1. Connect two phones using Bluetooth. 2. Enable NFC on both phones. 3. Open the browser on Device A. 4. Bump the phones together so vibration is felt. 5. Flip Device A to landscape mode. 6. Pull devices apart and flip device A back to portrait mode. Actual: 7. The browser change to portrait mode successfully. Flame 2.1 version: Gaia-Rev 38e17b0219cbc50a4ad6f51101898f89e513a552 Gecko-Rev https://hg.mozilla.org/releases/mozilla-2g34_v2_1/rev/8b92c4b8f59a Build-ID 20141205001201 Version 34.0 Device-Name flame FW-Release 4.4.2 FW-Incremental eng.cltbld.20141205.035305 FW-Date Fri Dec 5 03:53:16 EST 2014 Bootloader L1TC00011880
Attached video 1645.MP4 (deleted) —
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: