962257 - [B2G][Email][Browser] Link navigates to blank screen when sharing an email link in browser

Status: RESOLVED FIXED

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

Description

[sarsenyev]

Attachment: Videobug.0531.MOV

Description:
When opening the iphone email link from the test page "
http://people.mozilla.com/~nhirata/html_tp/iPhoneLinks.html" (without quotes), the email app opens with a white screen

Prerequestes:
Email is set up on the device

Repro Steps:
1) Updated Buri to BuildID: 20140121004137
2) Open the link in browser "http://people.mozilla.com/~nhirata/html_tp/iPhoneLinks.html"
3) Select the first link : John Frank. 

Actual:
The email app is opens with blank page

Expected:
Email app opens with compose message

E1.3 Environmental Variables:
Device: Buri 1.3 MOZ
BuildID: 20140121004137
Gaia: 47049555282a9a01fb60d1e1421b57e2810c96f5
Gecko: 6f7dfe36ab6c
Version: 28.0a2
Firmware Version: v1.2-device.cfg

Notes:
Repro frequency: 100%
Link to failed test case: https://moztrap.mozilla.org/manage/cases/?filter-id=2031
See attached: video clip

Comment 1

[sarsenyev]

The issue doesn't reproduce on 1.2 
After tapping the link email app opens with the name in compose field

Environmental Variables:
Device: Buri 1.2 COM
BuildID: 20130820040204
Gaia: 7249b5d61b955c23efe9e18f80f0da2e78d827eb
Gecko: bb025b6949e8
Version: 26.0a1
RIL Version: 01.02.00.019.102
Firmware Version: v1.2-device.cfg

Comment 2

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

logcat? https://wiki.mozilla.org/Gaia/Email/RequiredBugInfo

Comment 3

[sarsenyev]

Attachment: email-log

Account type:  "IMAP+SMTP"
Domain: Gmail, Yahoo

Comment 4

[James Burke [:jrburke]]

Attachment: GitHub pull request

A result of bug 960282, which introduced. attachmentName.ensureNameList. That method wants to cycle through blobs to ensure names exist for all blobs. However, in this case, there are no blobs passed. So, normalizing blobs data to an array, and using empty array if none found.

As bug 960282 was applied to 1.3, this bug fix should also be applied to 1.3.

Asking :asuth for review as he has context from bug 960282.

Comment 5

[James Burke [:jrburke]]

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

from pull request:
https://github.com/mozilla-b2g/gaia/pull/15581

Comment 7

[Natalya Kot [:nkot]]

don't think we still need regression window here

Comment 9

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

Uplifted f705458617f6837aca38c1b7a7194210cfc1657f to:
v1.3: f05e8faeb0ef75d072925f65961a9069e721d348