891408 - [B2G][Email] The contacts list cannot by displayed in full view after the keyboard has been invoked

Status: RESOLVED FIXED

(Firefox OS Graveyard :: Gaia::Contacts, defect)

Description

[Natalya Kot [:nkot]]

Attachment: screenshot

Description:
When the user is attempting to add "cc" or "bcc" contact from the contacts list after the keyboard has been invoked (typing in the subject or email body), the contacts list is displayed partially, lower half of the screen is white out

Prerequisites:
have contacts list available on the test device

Repro Steps:
1) Updated to Leo Build ID: 20130708070214
2) Open Email app
3) Select to create a new message
4) Type in the subject line 
5) Add "bcc" recipient from the contacts list ("+")
6) Scroll down the contacts list

Actual:
 Lower part of the screen is white out, contacts list displays only at the top

Expected:
 Contacts list opens up in full view

Environmental Variables
Gecko: http://hg.mozilla.org/releases/mozilla-b2g18/rev/8c0127cc7be3
Gaia: 7c40bdaeaffae708342fc773926dcfac5389348e
Platform Version: 18.1

Notes:
Repro frequency: 100%
See attached screenshot

Comment 1

[Natalya Kot [:nkot]]

Attachment: logcat

Comment 2

[Andrew Sutherland [:asuth] (he/him)]

The contacts list is provided by the contacts app, triggered via webactivity.  This seems like a contacts regression or system regression interfering with contacts.

Comment 3

[Ben Kelly [:bkelly, not reviewing]]

I just tried with a recent gaia master and gaia v1-train using a mozilla-central gecko build.  The problem did not reproduce in either case.

I'm building a b2g18 image to see if that triggers the issue.

Comment 4

[Ben Kelly [:bkelly, not reviewing]]

I was able to reproduce by flashing b2g18 gecko.

It seems like there is a compositing issue or something going on.  I am also getting weird flashing in settings app, scrolling the contacts list, etc.

Comment 5

[Ben Kelly [:bkelly, not reviewing]]

My apologies.  This does appear to be a v1-train gaia issue.  I had forgotten to update my local tracking branch.

Comment 6

[Ben Kelly [:bkelly, not reviewing]]

Bisecting shows that this was introduced by:

  https://github.com/mozilla-b2g/gaia/commit/5d7a0da528dce626b563abafc135eb828599d3c0

Rex, any thoughts on what is happening here?

Comment 7

[KM Lee [:rexboy] (inactivate after Sep. 22)]

Seems the newly added |appWindow.resize()| failed on calculating window size of inline activities.. Investigating.

Comment 8

[KM Lee [:rexboy] (inactivate after Sep. 22)]

Attachment: v1-train patch

Seems like a regression that I didn't discover in bug 877903..

I can reproduce it only in v1-train, but seems the code in master is just kind of "lucky hit" which is not using the correct way, so I'll patch both side.

Alive, would you help reviewing this?
Thanks a lot!

Comment 9

[KM Lee [:rexboy] (inactivate after Sep. 22)]

Attachment: master patch

Comment 10

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

Comment on attachment 773130 [details]
v1-train patch

There'd be chances to publish an instance event from app window so please consider to create a publish function, see the url for reference. You don't need to borrow all codes from my branch.

Comment 11

[KM Lee [:rexboy] (inactivate after Sep. 22)]

master
https://github.com/mozilla-b2g/gaia/commit/b3ef9da9c967e8f33b7b750d986d395e1aa38c95

v1-train
https://github.com/mozilla-b2g/gaia/commit/b2cb483bd6f89a7bdd4c47df256bea003bebcfba

Comment 12

[dkumar]

Varified fix on leo 1.1 Mozilla RIL
Environmental Variables
Build ID: 20130716070204
Gecko: http://hg.mozilla.org/releases/mozilla-b2g18/rev/629020cf576b
Gaia: fb9362d34260771d4a00b9a0e10a6bbad397bd3b
Platform Version: 18.1
RIL Version: 01.01.00.019.138 

The user is able to see  Contacts list up in full view.

Comment 13

[John Ford [:jhford] CET/CEST Berlin Time]

v1.1.0hd: b2cb483bd6f89a7bdd4c47df256bea003bebcfba
v1.1.0hd: 5ec8152a93acdad6319fd09446c99c1324bb059f