Firefox OS/Comms/Dialer/Sprint/v2.1-S7/20141024-StandupNotes
From MozillaWiki
< Firefox OS | Comms | Dialer | Sprint | v2.1-S7
Contents
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.
- Landed. Waiting for approval for uplift.
- 2.1+ bug 1079810 - Empty sound writes into speaker, cause serious power consumption
- Assigned to Doug.
- Landed.
- 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.
- It landed in master.
- Uplifting to 2.1 requested.
- 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.
- Landed.
drs
- Admin:
- We got all of our 2.1+ blockers landed! Now we just have to get them approved for uplift and then do it.
- 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.
- MVC proposal.
- Patches:
- bug 1079810 - Empty sound writes into speaker, cause serious power consumption
- Landed.
- bug 1083124 - [Call Log] Call log is not updated when user delete contact
- Going to investigate this.
- bug 1070050 - Dialer doesn't show you which # a contact called you on
- Got review comments from Anthony. Going to iterate.
- bug 1079810 - Empty sound writes into speaker, cause serious power consumption
- Reviews:
- bug 834530 - Emergency dialer DTMF tones aren't regulated by the volume button (and are different volume from Dialer App's DTMF tones)
- f+
- bug 1086528 - telephony_helper_test.js incorrectly uses Promises as a way to delay a test assertion
- r+
- bug 1082139 - JavascriptException: JavascriptException: TypeError: window.getComputedStyle(...) is null at: app://callscreen.gaiamobile.org/gaia_build_defer_index.js line: 146
- f+
- bug 1072064 - [ partner confidential ]
- r?
- bug 1070481 - [ partner confidential ]
- f?
- bug 834530 - Emergency dialer DTMF tones aren't regulated by the volume button (and are different volume from Dialer App's DTMF tones)
- Other:
- In a discussion with a guy from Intel about the Telephony API.
gtorodelvalle
- Admin:
- As it is usual on Fridays, I won’t be able to joint the standup meeting due to family matters.
- 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
- Currently investigating why Hsin-Yi, Johan and myself are getting distinct event notification and data flows, even when Johan and myself are using exactly the same build.
- After fixing an issue which didn’t let me get logs from the RIL (the updates to the pref.js file were not taken into consideration), I managed to modify the RIL constants file and to get these logs. I provided them to Hsin-Yi so she could try to shred some light on the issue.
- 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.
- I have not been able to test it since I am still observing an event flow which does not correspond with the ones Hsin-Yi and Johan are observing, although using the same as Johan.
- Michael Schwartz is supposed to be having a look at it. I we do not get any reply, I’ll ping him on Monday.
- bug 977588 - [Dialer] The Active call screen does not have the option to place calls on hold.
- Helping Paco with this bug.
- New version of the patch available hiding the put-on-hold and merge buttons for CDMA networks since these networks seem not to support these capabilities.
- bug 1082139 - JavascriptException: JavascriptException: TypeError: window.getComputedStyle(...) is null at: app://callscreen.gaiamobile.org/gaia_build_defer_index.js line: 146
thills
- bug 1075699 - T-Mobile Voicemail Bug. This is flagged with checkin-needed and has a fresh try run that is completed. Once this is checked in, I will land the gaia portion (though they can really be landed separately)
- 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 - I’m currently trying to repro the black screen that drs is seeing. I’ve tried on Hamachi and on flame and can’t seem to repro as of yet.
- bug 1081854 - Conference Call hangup.. The Gaia side has landed on m-c. I don’t see approval request for the gecko side to land on 2.1.
- bug 1082193 - Review for German. We gave r+ on this one.
- bug 1088690 - Filed this one. Narrowed it down to between 10/19 and 10/24 this occurs.
jlorenzo
- bug 1083731: pending feedback from QA in the US to run the test with AT&T and T-Mobile.
- bug 1086269: waiting for merge!
- bug 1086676: waiting for second reviews
- bug 1087926: i’m creating a test to check that the contact photo appears on the callscreen.