1006750 - [B2G][Gallery][Share]Sharing an image from gallery via MMS, user can see contacts list and simultaneously scroll through gallery

Status: RESOLVED FIXED

(Firefox OS Graveyard :: Gaia::System::Window Mgmt, defect)

Description

[Ross Kuhlman ([e-mail])]

Attachment: ScrollPicturesAndContacts.txt

User is able to enter a state in which the  contacts list and gallery are simultaneously visible and interactible.
This issue can only be reproduced by sharing from the gallery with the keyboard visible.


Repro Steps:
1) Update a Open C to BuildID: 20140506040204
2) Launch Gallery app.
3) Tap on an image.
4) Tap the Share button and select 'Messages'.
5) Tap on any textbox to summon the keyboard.
6) Tap on the '+' to see list of contacts.

Actual:
The contacts list is visible, but the viewspace previously occupied by the keyboard is now displaying the gallery. Images in gallery can be scrolled in this state.

Expected:
List of contacts is displays and occupies entire screen

Master M-C Environmental Variables:
Device: Open C Master M-C
BuildID: 20140506040204
Gaia: 4af716f09747edfbea637f5b60f7fd7d0183f19b
Gecko: 81651ad5e43c
Version: 32.0a1
Firmware Version: FFOS_US_EBAY_P821A10V1.0.0B06_LOG_DL

Notes:


Repro frequency: 100%
See attached: screenshot, logcat

Comment 1

[Ross Kuhlman ([e-mail])]

Attachment: HalfContactHalfGallery.png

Comment 2

[Natalya Kot [:nkot]]

'qawanted' to test on Buri trunk and Open_C 1.4

Comment 3

[Brogan Zumwalt [Inactive]]

Issue does NOT occur on 1.4 Open C PVT build.

Result: List of contacts is displays and occupies entire screen

Environmental Variables
Device: Open_C v1.4 Mozilla RIL
Build ID: 20140506000202
Gecko: https://hg.mozilla.org/releases/mozilla-b2g30_v1_4/rev/fe4080728c60
Gaia: b1242f33981024de59b8b4c26bacff8b876211b1
Platform Version: 30.0
Firmware Version: FFOS_US_EBAY_P821A10V1.0.0B06_LOG_DL


Issue DOES occur on 2.0 Buri PVT build.

Result: The contacts list is visible, but the viewspace previously occupied by the keyboard is now displaying the gallery. Images in gallery can be scrolled in this state.

Environmental Variables
Device: Buri v2.0 Master M-C Mozilla RIL
Build ID: 20140506040204
Gecko: https://hg.mozilla.org/mozilla-central/rev/81651ad5e43c
Gaia: 4af716f09747edfbea637f5b60f7fd7d0183f19b
Platform Version: 32.0a1
Firmware Version: v1.2-device.cfg

Comment 4

[Jason Smith [:jsmith]]

This looks like a keyboard regression.

Comment 5

[Harvey]

This window uses b2g-inbound production builds, as engineering builds from this time were blocked from testing by Bug 1000617.

B2G Inbound Regression Window

Last Working:
Environmental Variables
Device: Buri
BuildID: 20140424053001
Gaia: 4b6c9d4929c57005ab142e8eb3dc6783d55c427c
Gecko: c30df002e626
Version: 31.0a1
Firmware Version: v1.2-device.cfg

First Broken:
Environmental Variables
Device: Buri
BuildID: 20140424083005
Gaia: afa7142f2dc49b5b131ca48bfe818e2dfda15298
Gecko: ba6ea9f01a97
Version: 31.0a1
Firmware Version: v1.2-device.cfg

Last Working Gaia First Broken Gecko: Issue does NOT reproduce
Gaia: 4b6c9d4929c57005ab142e8eb3dc6783d55c427c
Gecko: ba6ea9f01a97

First Broken Gaia Last Working Gecko: Issue DOES reproduce
Gaia: afa7142f2dc49b5b131ca48bfe818e2dfda15298
Gecko: c30df002e626

Gaia Pushlog: https://github.com/mozilla-b2g/gaia/compare/4b6c9d4929c57005ab142e8eb3dc6783d55c427c...afa7142f2dc49b5b131ca48bfe818e2dfda15298

Comment 6

[Jason Smith [:jsmith]]

Maybe a window manager bug?

Alive - What do you think?

The relevant bugs in the range seem to be bug 936729, bug 993346, and bug 916709.

Comment 7

[(inactive after 6/18) Alive Kuo [:alive]]

Might be bug 916709's regression. Patch coming.

Comment 8

[(inactive after 6/18) Alive Kuo [:alive]]

Attachment: https://github.com/mozilla-b2g/gaia/pull/19046/files

This only happens in 3 layer inline activity chaining.
We are putting activity window on top of its caller right now, but we don't want to resize all the vertical windows. The keyboard is affecting the second window height but dismissed on third window opening which is already too late to resize second window again. Keyboard is a pain to the resize chain because we do not know which window it belongs. I hope this could be fixed in KeyboardWindow but not for now.

Also we will revisit resize strategy in https://bugzilla.mozilla.org/show_bug.cgi?id=1003870 but this patch free us from this case.

Comment 9

[(inactive after 6/18) Alive Kuo [:alive]]

Waiting travis.

Comment 10

[Tim Guan-tin Chien [:timdream] (please needinfo)]

Triage: regression.

Comment 11

[(inactive after 6/18) Alive Kuo [:alive]]

master
https://github.com/mozilla-b2g/gaia/pull/19046