Closed Bug 717434 Opened 11 years ago Closed 11 years ago

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

Categories

(Core :: DOM: Device Interfaces, defect)

ARM
Gonk (Firefox OS)
defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla12

People

(Reporter: philikon, Assigned: philikon)

References

Details

Attachments

(1 file, 1 obsolete file)

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
Attached 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+
Attached patch v2Splinter Review
comments addressed
Attachment #587864 - Attachment is obsolete: true
https://hg.mozilla.org/mozilla-central/rev/289576c50802
Status: NEW → RESOLVED
Closed: 11 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.