834126 - Choppy animation when 'EMERGENCY CALL' is touched in lockscreen passcode pad.

Status: RESOLVED WONTFIX

(Firefox OS Graveyard :: Gaia::System::Lockscreen, defect)

Description

[Leo]

1. Title : Choppy animation when 'EMERGENCY CALL' is selected in lockscreen passcode pad.
2. Precondition : Enable passcode lock, then enter the lockscreen passcode pad.
3. Tester's Action : Select 'EMERGENCY CALL' button.
4. Detailed Symptom (ENG.) : When 'EMERGENCY CALL' button is selected, the dial pad page appears with choppy animation.
5. Detailed Symptom (KOR.) : 
6. Expected : Smooth/normal page switching animation is expected.
7. Reproducibility: Y
1)Frequency Rate : 100%
8.Comparison Results : 
1)Model Comparing : 
9. Attached files: 
1)Log : 
2)Test Contents : 
3)Video file :

Comment 1

[Anthony Ricaud (:rik)]

I'm only seeing the choppy animation when going from the emergency call panel to the security code panel.

Going from security code panel to emergency panel does not exhibit the choppy animation.

Comment 2

[Leo]

Attachment: Recording

Comment 3

[Leo]

Hi Anthony, 

I have tried it on two different devices. On one device, going from Security code panel to Emergency call panel is worce than the other way. And on the other deivce, it's the opposite. Anyhow, there seems to be a problem on the animation on both ways.

Comment 4

[Daniel Zorro]

(In reply to Anthony Ricaud (:rik) from comment #1)
> I'm only seeing the choppy animation when going from the emergency call
> panel to the security code panel.
> 
> Going from security code panel to emergency panel does not exhibit the
> choppy animation.

I'm test with Keon(1.1 build 20130802051218) and Alcatel(1.1 retail build movistar/alcatel), the choppy animation is when exit from emergency panel

Comment 5

[Greg Weng [:snowmantw][:gweng][:λ]]

It seems duplicated.

Comment 6

[Greg Weng [:snowmantw][:gweng][:λ]]

:(

Sorry, I just closed this bug incorrectly. I now reopen it.

Comment 7

[Greg Weng [:snowmantw][:gweng][:λ]]

On Unagi

Gecko 28.0a2
Build: 20131218004002
Gaia: 6f0e50db

The animation played (almost) as the video shows. But I don't know whether this is a high priority bug and need to be fixed at this moment, or the performance is tolerable (compare to other performance issues). So I NI Tim to make the decision.

Comment 8

[Tim Guan-tin Chien [:timdream] (please needinfo)]

I don't make priority decisions. We make a decision collectively in triage meetings.

Comment 9

[Tim Guan-tin Chien [:timdream] (please needinfo)]

Triage: this is likely a perf bug we would like perf team to take over. However we should find out if we will be touching this part of Gaia code during v1.4 for bug 960915.

Comment 10

[Greg Weng [:snowmantw][:gweng][:λ]]

The emergency call, according to Rob's opinion, will come with different style, which is the same with the current secure camera represents (fade-in/out). So if there is any other performance issue we need to handle, it should be the performance issues of the new animation.

However I'm not started the Bug 960915, which now is delayed by Bug 960901, the instantiable lockscreen bug. The later one almost kill me because of the different results from the local environment and Travis. I'm still do debugging inefficiently (make change & print log, push ,wait ~30 mins and see what the Travis output; repeat again...)

Comment 11

[Tim Guan-tin Chien [:timdream] (please needinfo)]

Since the transition to be updated in bug 960915, let's mark this as WONTFIX and deal with the performance issue of the new transition (if there is any).