B2G telephony: audio isn't switched on for incoming calls and never switched off once switched on

RESOLVED FIXED in mozilla12

Status

()

defect
RESOLVED FIXED
8 years ago
8 years ago

People

(Reporter: philikon, Assigned: philikon)

Tracking

unspecified
mozilla12
ARM
Gonk (Firefox OS)
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment, 1 obsolete attachment)

Another small regression from bug 674726: if we receive an incoming call, we never turn on audio and leave the audio system forever in the dial tone state, it seems. Also, if the first call we make is an outgoing call and the audio system is turned on, we never seem to turn it off again.
Assignee: nobody → philipp
Posted patch v1 (obsolete) — Splinter Review
Attachment #587864 - Flags: review?(bent.mozilla)
Comment on attachment 587864 [details] [diff] [review]
v1

>+  activeCall: null,

Let's make this '_activeCall'

>+  updateCallAudioState: function updateCallAudioState() {
>+    if (!this.activeCall ||
>+        this.activeCall.state == nsITelephone.CALL_STATE_DISCONNECTED) {

This second condition isn't possible, is it? I'd remove it.
Attachment #587864 - Flags: review?(bent.mozilla) → review+
Posted patch v2Splinter Review
comments addressed
Attachment #587864 - Attachment is obsolete: true
https://hg.mozilla.org/mozilla-central/rev/289576c50802
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla12
Depends on: 723372
You need to log in before you can comment on or make changes to this bug.