914959 - [Buri][MMS]It will popup crash when view the 3GP file in mms

Status: RESOLVED WORKSFORME

(Firefox OS Graveyard :: Gaia::Video, defect, P1)

Description

[sync-1]

Mozilla build ID:20130902041201
 
 Created an attachment (id=514910)
 PIC
 
 DEFECT DESCRIPTION:
 It will popup crash when view the 3GP file in mms
 
  REPRODUCING PROCEDURES:
 1.Creat a mms, tap attachment button and choose video
 2.Add a 3GP video to mms
 3.View the video, it will popup warning --KO
 
  EXPECTED BEHAVIOUR:
 KO:can view the video successfully
 
  ASSOCIATE SPECIFICATION:
 
  TEST PLAN REFERENCE:
 
  TOOLS AND PLATFORMS USED:
 
  USER IMPACT:High
 
  REPRODUCING RATE:5/5
 
  For FT PR, Please list reference mobile's behavior:

Comment 1

[sync-1]

Clone from brother

Comment 2

[sync-1]

Attachment: LOG

Comment 3

[sync-1]

Clone from brother

Comment 4

[sync-1]

Attachment: 3gp file

Comment 5

[sync-1]

Clone from brother

Comment 6

[sync-1]

Attachment: PIC

Comment 7

[Naoki Hirata :nhirata (please use needinfo instead of cc)]

Can you add the crash id please?  You can find the crash id by:
adb shell ls -al /data/b2g/mozilla/Crash\ Reports/submitted

I crashed when I tried to send something via the MMS, so I think that this might be an issue with sending in general w/ MMS.

Comment 8

[xiupinglong]

The crash id is:

bp-900b40ad-9932-4032-9500-0473e2130911

Comment 9

[Julien Wajsberg [:julienw]]

This specific bug is about the video app.

Naoki, the crash you're experiencing is probably something else (and probably comm ril related ?)

Comment 10

[Naoki Hirata :nhirata (please use needinfo instead of cc)]

Looking at the crash stack, it had to do with IPC/graphics layer.  I filed a different bug for my crash.  

The crash stack for the crash id for this bug unfortunately does not have a symbol associated with it.

Comment 11

[David Scravaglieri [:scravag]]

crash seems to be something we should fix but before that could we check if this is reproduced with any 3GP files ?

Comment 12

[Naoki Hirata :nhirata (please use needinfo instead of cc)]

I tried to reproduce this crash and cannot do so in today's build with a sample 3gp file from Apple ( http://support.apple.com/kb/ht1425 ) nor with the attached video file.

Gaia   a0079597d510ce8ea0b9cbb02c506030510b9eeb
Gecko c4bcef90cef9
BuildID 20130916040205
Version 26.0a1
Buri

Can you please try to reproduce the crash with the latest build and state which build/device you used please?

Comment 13

[xiupinglong]

I can not reproduce it ethier. And I found the code has changed with the patch for Bug 895726.
Bug 895726 - [Buri][Monitor][MMS] Unable to save attachement from Camera(with conlicts fixing)

      // Make sure media is openable and savable even if:
      //   - Blob mimetype is unsupported but file extension is valid.
      //   - File extenion is missing or invalid but mimetype is supported.

      var mimetype =
        MimeMapper.guessTypeFromFileProperties(this.name, this.blob.type);
      var filename = MimeMapper.ensureFilenameMatchesType(this.name, mimetype);
      var activity = new MozActivity({
        name: 'open',
        data: {
          type: mimetype,
          filename: filename,
          blob: this.blob,
          allowSave: options && options.allowSave
        }
      });

Comment 14

[John Hu [:johnhu][:johu][:醬糊小弟]]

xiupinglong, 

This patch is only modifying the filename to meet the filename examination of device storage. That doesn't change anything about video playing.

I had tried this bug with my Inari under version 20130902041201. I can't reproduce it with both normal start-up of video app and view activity of video app. I had tested with the attached 3gp file.

Comment 15

[Julien Wajsberg [:julienw]]

Can we verify this does not reproduce on 1.1 too ?

Comment 16

[sarsenyev]

Verified on Leo 1.1 MOZ RIL, the issue doesn't reproduce
All files are opening for view and not crash appears

Build ID: 20130917041200
Gecko: http://hg.mozilla.org/releases/mozilla-b2g18/rev/b459ac3ebe24
Gaia: 862b5761825e010e3cc5968c096377e1ea54bbf2
Platform Version: 18.1

Comment 17

[Julien Wajsberg [:julienw]]

closing then
thanks !