1091442 - [Midori 2.0][Camera][Contacts]Can add a video for contact head portrait

Status: RESOLVED WORKSFORME

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

Description

[sync-1]

FireFox OS v2.0
 
 Created an attachment (id=976169)
 jrdlog
 
 DEFECT DESCRIPTION:
 Can add a video for contact head portrait
 
 REPRODUCING PROCEDURES:
 1.Launch messages->new a message->add a attachment by camera->click the camcorder icon->click home key
 2.Launch contacts->new a contact->add contacts head portrait by camera;
 3.Click the camcorder icon to switch to camcorder->click the shutter key to take video,can take video for contact head portrait->KO1
 4.Click the shutter key to stop record video->click the select button->it will still in a state of to set as contact head portrait->KO2
 5.Click the home button and launch contacts again->click the retake button->click the "X"button to close camera->can not close camera->KO3
 
 Comments:
 EXPECTED BEHAVIOR:
 KO:The camcorder icon should not display in camera interface,if solve this issues, other problems will not exist
 
 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]

Attachment: screenshot

Comment 2

[sync-1]

Attachment: jrdlog

Comment 3

[Wesly Huang (EPM)]

reproducible in my Flame with v188 base image. Indeed a problem to user in STR step#3/#4 but the key is to have step#1 in front of everything (otherwise no problem), so I think overall it's not a huge impact to user due to unusual step is required. de-nom candidate in coming Triage.

Comment 4

[Jack Liu]

(In reply to Wesly Huang from comment #3)
> reproducible in my Flame with v188 base image. Indeed a problem to user in
> STR step#3/#4 but the key is to have step#1 in front of everything
> (otherwise no problem), so I think overall it's not a huge impact to user
> due to unusual step is required. de-nom candidate in coming Triage.

Hi Wesly,

The severity of this issue is high and I don't think it's a problem only happened on core apps, you know many 3rd party apps including social apps like loop, line, fb etc. are very important to users, users need to take picture and video while chatting as they wish. It's impossible to iterate all usage scenarios, but it's sure the same issue as this definitely blocks normal usage. 

There will be strong demands from carriers and end users on fixing this bug ever since they found the defect, why not do it earlier?

Comment 5

[Wesly Huang (EPM)]

Hi Jack, thanks for your comment here, let me bring this to Triage for more discussion internally.

Comment 6

[Wesly Huang (EPM)]

[Triage] tag for 2.0.

Hi Evelyn: it seems fixed in 2.1 already w/ Wayne's phone, would you help on it to 2.0? Thank you.

Comment 7

[Evelyn Hung]

Gary, so comment 6 said there is a fix on 2.1, could you find the bug?

Comment 8

[GaryChen [:GaryChen][:PYChen][:gchen][:陳柏宇]]

(In reply to sync-1 from comment #0)
> FireFox OS v2.0
>  
>  Created an attachment (id=976169)
>  jrdlog
>  
>  DEFECT DESCRIPTION:
>  Can add a video for contact head portrait
>  
>  REPRODUCING PROCEDURES:
>  1.Launch messages->new a message->add a attachment by camera->click the
> camcorder icon->click home key
>  2.Launch contacts->new a contact->add contacts head portrait by camera;
>  3.Click the camcorder icon to switch to camcorder->click the shutter key to
> take video,can take video for contact head portrait->KO1
This step is not doable on currently v2.0 branch, since there is no switch icon anymore.
Here is my build info
----
Gaia-Rev        fe2167fa5314c7e71c143a590914cbf3771905a8
Gecko-Rev       https://hg.mozilla.org/releases/mozilla-b2g32_v2_0/rev/241e51806687
Build-ID        20141103160206
Version         32.0
Device-Name     flame
FW-Release      4.4.2
FW-Incremental  27
FW-Date         Thu Sep  4 14:59:02 CST 2014
Bootloader      L1TC10011800

>  4.Click the shutter key to stop record video->click the select button->it
> will still in a state of to set as contact head portrait->KO2
>  5.Click the home button and launch contacts again->click the retake
> button->click the "X"button to close camera->can not close camera->KO3
>  
>  Comments:
>  EXPECTED BEHAVIOR:
>  KO:The camcorder icon should not display in camera interface,if solve this
> issues, other problems will not exist
>  
>  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 9

[Joshua Mitchell (Inactive)]

Gary - what was the purpose of adding the qa-wanted tag? You did not make any request or ask any question so we don't know what you want done here.

Comment 10

[Hema Koka [:hema]]

(In reply to Joshua Mitchell [:Joshua_M] from comment #9)
> Gary - what was the purpose of adding the qa-wanted tag? You did not make
> any request or ask any question so we don't know what you want done here.

Could QA help reproduce this on 2.0 on Flame and put in the clear STR. I think what Gary meant is he does see switch icon for Step 3 in original STR.

Comment 11

[David Flanagan [:djf]]

Taking this to investigate.

Comment 12

[No-Jun Park [:njpark]]

(In reply to Hema Koka [:hema] from comment #10)
> (In reply to Joshua Mitchell [:Joshua_M] from comment #9)
> > Gary - what was the purpose of adding the qa-wanted tag? You did not make
> > any request or ask any question so we don't know what you want done here.
> 
> Could QA help reproduce this on 2.0 on Flame and put in the clear STR. I
> think what Gary meant is he does see switch icon for Step 3 in original STR.

I just tested 2.0 build, and in step 2, camcorder option is no longer visible, hence the bug isn't reproducible.  If I understand it correctly, 2.0 has the bug fixed now I suppose?  Joshua, could you confirm what I'm seeing as well?  Thanks!

Comment 13

[David Flanagan [:djf]]

Like Gary and No-Jun, I cannot reproduce this bug on a 2.0 Flame with 1024mb of memory.

The STR include two apps both launching pick activities for the camera app at the same time. From those STR it sounds like the Contacts app is getting the same instance of the pick activity that was created for the Messages app.

But I don't see that. When the contacts app launches the camera, the pick activity that is pending for the Messages app is cancelled.  

We need more info here:

Alive: have there been any recent window manager changes in 2.0 recently that would affect this case where two apps both launch the camera to handle a pick activity?

Wesly: can you still reproduce this? If so, could you explain how, please? What did you mean in comment #3 when you wrote "the key is to have step #1 in front of everything". Did you just mean that you have to do step 1 first, or do you mean that somehow you're bringing the messaging app back to the foreground?

Jack: can you still reproduce this bug, and is your 2.0 tree fully in sync with Mozilla's v2.0 tree?

Comment 14

[David Flanagan [:djf]]

I also tried reproducing this on my Flame (v188) with a nightly 2.0 build from 10/28, two days before this bug was filed. I still can't reproduce. So that makes me think that the STR are unclear or that our partners are testing with an out-of-date version of the v2.0 tree.

Unassigning myself since I can't work on the bug if I can't reproduce it.

Comment 15

[Pi Wei Cheng [:piwei] (inactive)]

The bug is reproducible on an earlier Flame 2.0 build, but it is no longer reproducible on today's Flame 2.0.

Issue reproducible in the following build:

Device: Flame 2.0
BuildID: 20141023124833
Gaia: 86d83f4b4111ca45ebc92ca779348cc966f43cff
Gecko: f8432250efb7
Version: 32.0 (2.0)
Firmware: V188-1
User Agent: Mozilla/5.0 (Mobile; rv:32.0) Gecko/32.0 Firefox/32.0


Issue NOT reproducible in the following build:
Device: Flame 2.0
BuildID: 20141104053644
Gaia: fe2167fa5314c7e71c143a590914cbf3771905a8
Gecko: 093de6b632c5
Version: 32.0 (2.0)
Firmware: V188-1
User Agent: Mozilla/5.0 (Mobile; rv:32.0) Gecko/32.0 Firefox/32.0

As mentioned at comment 12 and 8, at step 2 while in Contacts app there isn't an option to switch to Camcorder anymore.

Comment 16

[(inactive after 6/18) Alive Kuo [:alive]]

(In reply to David Flanagan [:djf] from comment #13)
> Like Gary and No-Jun, I cannot reproduce this bug on a 2.0 Flame with 1024mb
> of memory.
> 
> The STR include two apps both launching pick activities for the camera app
> at the same time. From those STR it sounds like the Contacts app is getting
> the same instance of the pick activity that was created for the Messages app.
> 
> But I don't see that. When the contacts app launches the camera, the pick
> activity that is pending for the Messages app is cancelled.  
> 
> We need more info here:
> 
> Alive: have there been any recent window manager changes in 2.0 recently
> that would affect this case where two apps both launch the camera to handle
> a pick activity?

Looks like https://bugzilla.mozilla.org/show_bug.cgi?id=1032068 is uplifted to v2.0.
With that we are able keep inline activity - even pressing home button - until the next activity request comes,
but in your case, the second activity request should drop the first camera activity.

I don't know what happens, and since we are not able to reproduce, there might be something unclear in the STR.

Comment 17

[Hema Koka [:hema]]

Wesly/Jack: If this is no longer an issue with latest 2.0 for you, please close as works for me. Thanks!

Comment 18

[hanp]

We merged the patch and found this problem resolved, thanx

Comment 19

[Wesly Huang (EPM)]

Agree to close, like in comment#15 it's fixed in later Flame 2.0 SW I believe.

Comment 20

[Jack Liu]

(In reply to Hema Koka [:hema] from comment #17)
> Wesly/Jack: If this is no longer an issue with latest 2.0 for you, please
> close as works for me. Thanks!

Hi,

Would you please tell us which patch to take, you know it's impossible for us to pull latest code from mozilla directly. 

Thanks.