1120354 - [Flame][Email]When you select two mails, the title displays as "Edit" and only one mail is selected.

Status: RESOLVED WORKSFORME

(Firefox OS Graveyard :: Gaia::E-Mail, defect)

Description

[Shine(Leave from Mozilla)]

[1.Description]:
[Flame][v2.2][Email]When refreshing mail, enter edit mode screen and select two mails, the title will be displayed as "Edit" and only one mail had been selected.
See attachment:logcat_0359.txt & video3.MP4
Found time: 03:59


[2.Testing Steps]: 
1. Launch Mail and log in a mail account.
2. Refresh mail.
3. Tap edit button on right bottom corner.
4. Select one mail.
5. Select another mail.

[3.Expected Result]: 
5. The title could display "2 selected" and the two mails should been selected.

[4.Actual Result]: 
5. The title displays as "Edit" and only one mail had been selected.

[5.Reproduction build]: 
Flame 2.1 build:
ia-Rev        64db236bea9a0510567ab7ced2f2b4688737123c
Gecko-Rev       https://hg.mozilla.org/releases/mozilla-b2g34_v2_1/rev/273f24a1d1fe
Build-ID        20150111001202
Version         34.0

Flame 2.2 build:
Gaia-Rev        f5e481d4caf9ffa561720a6fc9cf521a28bd8439
Gecko-Rev       https://hg.mozilla.org/mozilla-central/rev/bb8d6034f5f2
Build-ID        20150111010223
Version         37.0a1

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

[7.TCID]: 
Free test

Comment 1

[Shine(Leave from Mozilla)]

Attachment: logcat

Comment 2

[Shine(Leave from Mozilla)]

Attachment: video

Comment 3

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

I think what must be happening here is that edit mode is being entered before the backend has actually told us about the headers (or something very bad happened leading to the same result), so we're using the HTML cookie-cache provided HTML and prior to the "message" expandos being set.  So the first click does selectedMessages.push(undefined) and then the second click's indexOf test for its own undefined message thinks it's already in the set and so splices it out and drives us back down to zero.

I'm having trouble matching up the video with the logcat based on the visible clock value and the timestamps in the log; maybe the logcat wrapped?  Specifically, this error:
01-13 04:00:22.752 I/GeckoDump( 1384): ERR: onerror reporting: TypeError: o is undefined @ app://email.gaiamobile.org/js/mail_app.js : 2149
could maybe be something... but the timestamp seems too late.  I'm also unable to map that line error for either v2.1 or v2.2 trunk right now.

Comment 4

[Shine(Leave from Mozilla)]

This problem is verified not to happen on latest build of Flame 2.2. if anyone can repro it, reopen again please.
Rate: 0/10

Flame 2.2 build: 
Gaia-Rev        f5b3d1b6cfa3e702033f613915ae637cb735cbfb
Gecko-Rev       https://hg.mozilla.org/releases/mozilla-b2g37_v2_2/rev/5d7497ce4cc7
Build-ID        20150120002507
Version         37.0a2
Device-Name     flame
FW-Release      4.4.2
FW-Incremental  eng.cltbld.20150120.035540
FW-Date         Tue Jan 20 03:55:51 EST 2015
Bootloader      L1TC000118D0