975649 - [B2G][SMS] Taking a new picture through SMS attachment then saving the message as a draft and trying to view the picture later results in an error message

Status: RESOLVED WORKSFORME

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

Description

[Rachel Pribble [:rpribble]]

Attachment: ErrorMessage

Description:
Creating a new SMS and taking a new picture to attach to it, then saving the SMS as a draft results in user not being able to go back in to the SMS late and view the picture. User is shown the error message 'There is no application available to open this file type'. This only occurs when the photo attached is a new picture taken with a camera, attaching a previously taken image from the gallery is able to be viewed as expected in draft form.

Repro Steps:
1) Updated buri to BuildID: 20140221040202
2) Tap the SMS icon
3) Tap the button in the upper right to create a new SMS
4) Tap the paperclip to add an attachment without adding a recipient or text
5) Tap the option to use the amera
6) Take a photo and tap select to use the photo
7) Tap the back arrow in the upper left corner
8) Tap 'Save as draft'
9) After being returned to the main SMS list, tap the newly saved draft
10) Tap the image attached to the draft
11) Select 'View' to see the image
12) Observe error message appear

Actual:
User unable to view a newly taken image in an SMS when saved as a draft.

Expected:
User always able to view elements of the SMS.

Environmental Variables:
Device: buri v14 moz
BuildID: 20140221040202
Gaia: 35365feace970bfc51276428f40a477c9c86b7bb
Gecko: 7010ab83a06e
Version: 30.0a1
Firmware Version: v1.2-device.cfg

Notes: Unable to check in v1.3, no option when backing out of a SMS to 'Save as draft'.

Repro frequency: 100%
See attached: Screenshot

Comment 1

[Jason Smith [:jsmith]]

Joe - Are we still planning to ship drafts support in 1.4, pref it off in 1.4, or back it out in 1.4?

Comment 2

[Julien Wajsberg [:julienw]]

Backing it out would be difficult, preffing it off would be certainly doable, but I'd bet it's less risky to fix the few remaining bugs.

Plus, this bug looks a lot like bug 949202 and might be unrelated to drafts.

Comment 3

[Joe Cheng [:jcheng] (please needinfo)]

draft mode has landed for quite some time, i don't believe we need to back it out at this moment. Also agreeing with comment 2, let's identify the remaining bugs and fix them (unless the identified bugs are way too risky to fix)

Comment 4

[Jason Smith [:jsmith]]

David - Is the same as bug 949202?

Comment 5

[David Flanagan [:djf]]

It doesn't sound like exactly the same issue to me.  But certainly could be related.  I'd recommend seeing if the patch on bug 949941 (which makes camera return a file-backed blob) fixes it.

Comment 6

[Jason Smith [:jsmith]]

QA Wanted for a retest on a 3/18 1.4 build or later, as that will have bug 949941 included with it.

Comment 7

[Josh Schmitt [Joshs]]

In the latest 1.4 Buri build (3/18) after selecting the 'View' button a black overlay appears and no error message is shown, I am not able to view the picture attached.

Environmental Variables:
Device: Buri 1.4 MOZ
BuildID: 20140318000203
Gaia: c03a6af9028c4b74a84b5a98085bbb0c07261175
Gecko: 3776f72f1967
Version: 30.0a2
Firmware Version: V1.2-device.cfg

Comment 8

[Jason Smith [:jsmith]]

Can you provide a screenshot of what you are seeing in comment 7?

Comment 9

[Josh Schmitt [Joshs]]

Attachment: Overlay.png

Screenshot attached, overlay disappears after a few seconds.

Comment 10

[Jason Smith [:jsmith]]

That implies the original bug no longer reproduces, but we've got a new bug here instead. Can you open a separate bug for this?

Comment 11

[Josh Schmitt [Joshs]]

(In reply to Jason Smith [:jsmith] from comment #10)
> That implies the original bug no longer reproduces, but we've got a new bug
> here instead. Can you open a separate bug for this?

Filed a new bug 985167