977480 - [Sora][Call][Voicemail] The "Response time" is too long when user press and hold the number 1 to call voicemail number.

Status: RESOLVED DUPLICATE of bug 990003

(Firefox OS Graveyard :: Gaia::Dialer, defect, P2)

Description

[sync-1]

Firefox OS v1.3
  Mozilla build ID: 20140208004002
 
 DEFECT DESCRIPTION:
 The "Response time" is too long when user press and hold the number 1 to call voicemail number.
 
  REPRODUCING PROCEDURES: 
  1. Idle -> Settings -> Call Settings -> Voicemail -> Add a Voicemail Number
  2. Idle -> Open phone app -> long press the number 1 -> Dial the voicemail out, but the "Response time" is too long(K.O.)
 
  EXPECTED BEHAVIOUR:
  The "Response time" is appropriate when user press and hold the number 1 to call voicemail number.
 
  ASSOCIATE SPECIFICATION:
 
  TEST PLAN REFERENCE:
 
  TOOLS AND PLATFORMS USED:
 
  USER IMPACT:
 
  REPRODUCING RATE: 100%
 
  For FT PR, PleaseInsert list reference mobile's behavior:
  It is NOK on Beetle Lite FF.
 
  If you have any questions,please dial 7526.

Comment 1

[Mason Chang [Inactive] [:mchang]]

How long is too long? And how much time is expected until we get the voicemail number?

Comment 2

[Mingming ZHAO]

Now the Response time is almost 3 seconds(2.8). Maybe 1~2 seconds is expected.

Comment 3

[:Eli Perelman]

Attachment: Cleopatra Profile

Profiling dialing voicemail. It appears this issue stems from the usage of window.open to open the call screen. Voicemail currently has a 1.5 second delay configured from the start of the long-hold to when the call screen window should be shown, but in my testing is delayed anywhere from 500-750ms. This delay occurs within the implementation of window.open. I experimented with loading a test page instead of the full call screen using window.open, but I still experienced the 500ms delay.

I have attached a profile to help look into something lower level than the Gaia layer. The window.open in question is located here:

https://github.com/mozilla-b2g/gaia/blob/master/apps/communications/dialer/js/dialer.js#L314

Comment 4

[Etienne Segonzac (:etienne)]

Our best bet here would be the wip mentioned here: https://bugzilla.mozilla.org/show_bug.cgi?id=987022#c26

But it's a big/risky patch.