1093002 - [Flame][v2.1][Contact]The title name of contact list changes after you lock and unlock the screen.

Status: VERIFIED FIXED

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

Description

[Paladin]

Attachment: select_contact.MP4

[1.Description]:
It returns to contact list， but "Select contact" displays as title.
Attachment: logcat_flame_1242.txt and select_contact.MP4
Occurrence time: 00:42

[2.Testing Steps]: 
1. Open contacts app 
2. Press settings button 
3. Press export contacts button 
4. Select "To Media Card" option 
5. Tap the power button to Lock the phone and then tap it again to wake the phone up.
6. Press Close button 

[3.Expected Result]: 
The title display is Contact.

[4.Actual Result]: 
The user return to contact list and the title display is Select contact.

[5.Reproduction build]: 
Gaia-Rev        36d9f52883b420f90b759303609b2d27d6f94f2c
Gecko-Rev       https://hg.mozilla.org/releases/mozilla-b2g34_v2_1/rev/97fd482ec87a
Build-ID        20141102161201
Version         34.0

[6.Reproduction Frequency]: 
Always Recurrence,5/5

Comment 1

[Paladin]

Attachment: logcat_flame_0042.txt

Comment 2

[Francisco Jordano [:arcturus] [:francisco]]

Nice catch, let's try to take a quick look in next sprint.

Comment 3

[Francisco Jordano [:arcturus] [:francisco]]

Attachment: Pointer to PR 26113

This PR ends up removing a lot of code. Before we had the proper edit mode with the header antimating and so on, we were changing the normal header, but that's not needed anymore.

We had this problem probably since 2.0

Comment 4

[Jose Manuel Cantera]

Comment on attachment 8522214 [details] [review]
Pointer to PR 26113

Francisco,

the proposed patch introduces at least one regression. When you go to the pick contact activity the title of the app is 'Contacts' instead of 'Select Contact'. 

thanks!

Comment 5

[Francisco Jordano [:arcturus] [:francisco]]

(In reply to Jose Manuel Cantera from comment #4)
> Comment on attachment 8522214 [details] [review]
> Pointer to PR 26113
> 
> Francisco,
> 
> the proposed patch introduces at least one regression. When you go to the
> pick contact activity the title of the app is 'Contacts' instead of 'Select
> Contact'. 
> 
> thanks!

If I'm not wrong we had that behavior already in 2.0 (just showing 'Contacts' in a pick activity), that's why I decided to remove the code for changing the header title.

ni Carrie, do you still want the title changed 'Select contact' in a pick activity or are you ok just with 'Contacts'?

Comment 6

[Carrie Wang [New account: [e-mail]]]

Hi, 

For the Contact picker, I'd like to display the header with "Select Contact" and keep the Contact APP header with the title "Contacts". Thanks!

Comment 7

[Francisco Jordano [:arcturus] [:francisco]]

Comment on attachment 8522214 [details] [review]
Pointer to PR 26113

Jose, second round?

Comment 8

[Jose Manuel Cantera]

(In reply to Francisco Jordano [:arcturus] [:francisco] from comment #7)
> Comment on attachment 8522214 [details] [review]
> Pointer to PR 26113
> 
> Jose, second round?

The patch works properly, but I have realised that we need also to have the same title 'Select Contact' when we pick an ICE Contact.

Comment 9

[Francisco Jordano [:arcturus] [:francisco]]

Hei Jose, just added to the PR the chance for selecting ice contacts.

Comment 11

[Jose Manuel Cantera]

Comment on attachment 8522214 [details] [review]
Pointer to PR 26113

works perfectly

thanks, Francisco

Comment 12

[Francisco Jordano [:arcturus] [:francisco]]

Landed in master:

https://github.com/mozilla-b2g/gaia/commit/072b91a55ce625c8c7681777e0ecd4a4f536bb0e

Comment 13

[Lance（leave from Mozilla）]

Attachment: contacts.mp4

Comment 14

[Lance（leave from Mozilla）]

The problem is verified not happen on latest Flame 2.1build.

Steps:
1) Open contacts app 
2) Press settings button 
3) Press export contacts button 
4) Select "To Media Card" option 
5) Tap the power button to Lock the phone and then tap it again to wake the phone up.
6) Press Close button 

  
Actual:
The title display is Contact.
Fail rate:0/5
See attachment:contacts.mp4

Flame 2.1 version:
Gaia-Rev        d98bbe9d2bfdb53e80dc1ab1572bd05938a85526
Gecko-Rev       https://hg.mozilla.org/releases/mozilla-b2g34_v2_1/rev/c694578ff69e
Build-ID        20150128001258
Version         34.0
Device-Name     flame
FW-Release      4.4.2
FW-Incremental  eng.cltbld.20150128.034753
FW-Date         Wed Jan 28 03:48:03 EST 2015
Bootloader      L1TC000118D0