Firefox OS/Comms/Dialer/Sprint/v2.1-S7/20141022-StandupNotes

Blockers

• 2.1+ bug 1081854 - GAIA follow-up to bug 978639 (for GCF case 31.4.2.1.3)
  • Assigned to Tamara.
  • Depends on bug 978639, which has landed. This is ready to go.
• 2.1+ bug 1079810 - Empty sound writes into speaker, cause serious power consumption
  • Assigned to Doug.
  • Dev Tools are holding a weak reference to the WebAudio context. Investigating more.
• 2.1- bug 1082876 - TypeError: context is null at: app://callscreen.gaiamobile.org/gaia_build_defer_index.js line: 84
  • Assigned to Gabriele.
  • In reality is basically a blocker, since it’s blocking a partner’s testing, even if there are no STR.
  • Cause unknown, possibly something in shared/js/dialer/tone_player.js
• 2.1+ bug 1082139 - JavascriptException: JavascriptException: TypeError: window.getComputedStyle(...) is null at: app://callscreen.gaiamobile.org/gaia_build_defer_index.js line: 146
  • Assigned to Germán.
  • We are trying to reproduce the scenario mentioned by Anshul in Telefónica creating an integration test. I have not received the results from our QA team yet. Anyhow, I hope Anshul can provide us with further information about the tests he is running.
  • It could be related to bug 548397 - window.getComputedStyle() returns null inside an iframe with display: none
  • Germán will include Tamara’s comments ASAP :)
• 2.1+ bug 1075699 - Calling voicemail by long-pressing the 1 button displays an error and then calls
  • Assigned to Tamara.
  • Ready to land a fix, but waiting for vendor to reply to a needinfo.
  • Moved to RIL component, but we should still watch this.
  • We should land this as the vendor is unlikely to answer in time.

drs

• Admin:
  • Bug triage is tomorrow. Please let me know if you’re coming.
• Next steps:
  • MVC proposal.
    • This is getting a lot further. I should have it ready in a few days. It has evolved in a “design guidelines” document that includes many things, including MVC, code styling, suggested refactors, etc.
• Patches:
  • bug 1079810 - Empty sound writes into speaker, cause serious power consumption
    • Dev Tools are holding a weak reference to the WebAudio context. Investigating more.
  • bug 1083124 - [Call Log] Call log is not updated when user delete contact
    • Going to investigate this.
• Reviews:
  • bug 1069835 - [ partner confidential ]
    • r-
  • bug 834530 - Emergency dialer DTMF tones aren't regulated by the volume button (and are different volume from Dialer App's DTMF tones)
    • r-
  • bug 1081854 - GAIA follow-up to bug 978639 (for GCF case 31.4.2.1.3)
    • r-
• Other:
  • In a discussion with a guy from Intel about the Telephony API.

gtorodelvalle

• Admin:
  • Nothing to report.
• Next steps:
  • bug 1082139 - JavascriptException: JavascriptException: TypeError: window.getComputedStyle(...) is null at: app://callscreen.gaiamobile.org/gaia_build_defer_index.js line: 146
    • Blocker (2.1+) (see the blocker section)
  • bug 977056: [Dialer] Visual discrepancies: Call ended view under conference call participants list
    • It seems we currently have an agreed sequence diagram spec including the event notification and data flow for the call scenario when a participant leaves a conference call amongst 3 parties.
    • This agreed event notification and data flow should unblock the patch for this bug.
    • I really think it would be great to have this kind of diagrams for all the available calling scenarios and that they should be published in https://developer.mozilla.org/en-US/docs/Web/API/Telephony So I’ll spend some minutes on them to start collecting them.
    • Definitely an interesting topic for our next ‘office hour’ and/or ‘bug bash’ session.
  • bug 1083402 - "X has left the conference call" when a party drops off a conference call is missing:
    • The cause of the problem is the same as the one reported in bug 977056.
    • It should be solved once the patches for “bug 1059649 - [tarako/dolphin] After we hangup any one of the two calls in a group call, we'll never success to merge conference call again” are included in the generated builds. I’ll track it and check it accordingly.
  • bug 977588 - [Dialer] The Active call screen does not have the option to place calls on hold.
    • Helping Paco with this bug.
    • We already have a new version of the patch covering the CDMA case (disabling the put-on-hold button) waiting for Carrie’s confirmation that that is the way to go.

thills

• • bug 1075699 - T-Mobile Voicemail Bug. Re-running try and retested last night to make sure nothing changed since this sat a while and we need to land it now. Also, I responded back to the thread that this cannot be repro’d on v184 with T-Mobile. It can. I reproduced this on first try.
  • bug 1079143 - BlueTooth Bug - Posted my findings on this. Still have an NI on me for this, but prioritizing this after the blocker
  • bug 1068093 - Removed legacy call screen transitions - Fixed the trivial items and then posted a question back on one of the comment that still needs sorting out.
  • bug 1081854 - Conference Call hangup. Posted a review and then got review- back and reposted the PR with the changes last night.
  • bug 1082193 - Review for German. Finished this with suggestion that we should try and have a test for this.

jlorenzo

• • bug 1083731: pending feedback from QA in the US to run the test with AT&T and T-Mobile.
  • bug 1086269: one r+, waiting for the other one.
  • bug 1086676: Add a test to check notification when you miss a call from a known contact.