Closed
Bug 830657
Opened 11 years ago
Closed 11 years ago
[FM Radio] No sound on the second launch of the application
Categories
(Firefox OS Graveyard :: General, defect)
Tracking
(blocking-b2g:tef+, b2g18 fixed, b2g18-v1.0.0 fixed, b2g18-v1.0.1 fixed)
VERIFIED
FIXED
blocking-b2g | tef+ |
People
(Reporter: dscravaglieri, Assigned: rlin)
References
Details
(Whiteboard: [EU_TPE_TRIAGED][triage:1/16])
Attachments
(1 file)
2.34 KB,
text/plain
|
Details |
gecko-b2g18: f2460bf17811 gaia: c70e6b82b9d208153766a1111c3bdc36a6fcb479 kernel: Dec. 5 STR: 1 - launch the App 2 - plug the Headset Expected: Can ear sound from radio station Actual: No sound.
Reporter | ||
Updated•11 years ago
|
blocking-b2g: --- → tef?
Reporter | ||
Updated•11 years ago
|
blocking-b2g: tef? → tef+
Whiteboard: [EU_TPE_TRIAGED]
Updated•11 years ago
|
Assignee: nobody → rlin
Comment 1•11 years ago
|
||
It seems like this happens on every second launch of the application. I have this error in the console when the application is closed manually in the cards view: E/GeckoConsole( 107): [JavaScript Error: "FMRadio message DOMFMRadio:getPowerState from a content process with no 'fmradio' privileges." {file: "resource://gre/modules/DOMFMRadioParent.jsm" line: 400}]
Severity: normal → major
Component: Gaia::FMRadio → General
QA Contact: jshih
Summary: [FM Radio] No sound if headset plugged after launching the App → [FM Radio] No sound on the second launch of the application
Assignee | ||
Comment 2•11 years ago
|
||
Do you press the play button when after 1. open the fm app 2. plug-in headset I found I need to press the play button and the fm radio works well. test on 1/15 mozilla-b2g18-unagi-eng 15-Jan-2013 08:01 build.
Flags: needinfo?(dscravaglieri)
Assignee | ||
Comment 3•11 years ago
|
||
Hi philipp, Could you help check this? I back out this change. changeset: 118173:00ab761c3545 user: Philipp von Weitershausen <philipp@weitershausen.de> date: Mon Jan 14 19:46:32 2013 -0800 summary: Bug 823798 - Route FM radio through analog path. r=mwu a=blocking-b2g and this problem was gone.
Flags: needinfo?(dscravaglieri)
Updated•11 years ago
|
Flags: needinfo?(philipp)
Whiteboard: [EU_TPE_TRIAGED] → [EU_TPE_TRIAGED][triage:1/16]
Comment 5•11 years ago
|
||
In Bug 823798, it also modified full_unagi.mk for adding "ro.moz.fm.noAnalog=true". This will disable FM in analog mode in default so this bug should not be happened. And this new property is stored at /system/build.prop so if you just update your phone via OTA then it just for gecko only. Please flash an entire newest system.img for that new property.
Reproduced this issue on OTA Unagi to build 20130117070201, Kernel Dec 5. Was able to hear FM Radio the 1st time, then unplugged headset, then plugged it back in and reopened FM Radio and could not hear anything. Gecko 35f645b83289 Gaia 2794faeec9da
Assignee | ||
Comment 7•11 years ago
|
||
Hi Michael, I found the [ro.moz.fm.noAnalog]: isn't include in b2g-18 nightly build on (17-Jan-2013 07:55) As test with mchen and me, it seems need to clean build to fix this problem. Could you help to check?
Flags: needinfo?(philipp)
Comment 8•11 years ago
|
||
Randy, it looks like bug 823798 has landed the pref on b2g18. Philikon, can you lend a hand here?
Flags: needinfo?(philipp)
Assignee | ||
Comment 9•11 years ago
|
||
compare with the releases.mozilla.com latest image, the [ro.moz.fm.noAnalog]: [true] attribute is setting correctly and can't find this issue. But After checking the pvtbuild nightly one, the build.prop can't find this property, so as in device. Do the clean build in the pvtbuild build machine may helpful for this issue.
Comment 12•11 years ago
|
||
Chris Atlee can probably provide more info on the builds.
Flags: needinfo?(catlee)
Comment 13•11 years ago
|
||
where in the build process should this pref be getting set? I can't find it in any of our builds. can you give me the url of the build that you think has this pref set properly?
Flags: needinfo?(catlee)
Assignee | ||
Comment 14•11 years ago
|
||
1. This ROM on this url:https://releases.mozilla.com/b2g/latest/ is set correctly. Can you fine this change on the "device/qcom/unagi" git Repository? commit 6c014552d1b26bee611d9a9b23bd4cd014e392ee Author: Philipp von Weitershausen <philipp@weitershausen.de>
To note when you get an incoming call, it also stops the radio from playing. Not sure if this is related or a separate bug.
Updated•11 years ago
|
I see : 01-23 09:54:08.617: W/AudioFlinger(118): Thread AudioOut_1 cannot connect to the power manager service Just wondering, didn't the audio go on a different channel now? Or is that only for the music app?
Assignee | ||
Comment 17•11 years ago
|
||
Hi Hal, Test on <full_unagi-user 4.0.4.0.4.0.4 OPENMASTER eng.cltbld.20130124.020958 test-keys> Still can't find the "ro.moz.fm.noAnalog=true" Does the Bug 833410's fix land?
Flags: needinfo?(philipp)
Comment 18•11 years ago
|
||
If we get a new build from releng before rlin gets to the office, can you verify the fix, Naoki? qawanted: verify analog path pref fixes this bug
Flags: needinfo?(hwine) → needinfo?(nhirata.bugzilla)
Keywords: qawanted
Feed me! ;)
Flags: needinfo?(nhirata.bugzilla)
Oh and mark this needinfo on me again... I'll look at it asap.
Updated•11 years ago
|
Flags: needinfo?(overholt)
Comment 21•11 years ago
|
||
I can still reproduce this issue using the latest nightly, Gaia: 67a5b7f79b73 Gecko: 49c25ca40020
Comment 22•11 years ago
|
||
(In reply to Marcia Knous [:marcia] from comment #21) > I can still reproduce this issue using the latest nightly, > > Gaia: 67a5b7f79b73 > Gecko: 49c25ca40020 Correct - the change was only landed at 2013-01-24 12:57:06 PT (see bug 833410 comment 7) Only builds started after that will use the new gonk with the requested changeset.
Updated•11 years ago
|
Flags: needinfo?(nhirata.bugzilla)
Hrm. This is fixed now with the build: tried: - not having headset in and playing fm radio : message (as expected) - plugging in headset and hit play : played radio through headset - stopping fm radio : stopped - started fm radio again : started up again ( slight delay ) ** - plugged out headset : stopped radio - plugged in headset : started radio again - gave a call : ringing sounded not through headset but through speaker; voice was through headset ** need to report a bug - ended call : fm radio played - quit fmradio through task manager : fm radio stopped - started fmradio : fm radio played again (didn't memorize last location) - placed radio in background : conintued to play - unplugged headset : fmradio stopped - plugged in headset : fmradio played - loaded music, played a song : fmradio stopped, music played - stopped music : fmradio started playing - played music again, waited til it hit the next song : ** music didn't go to the next song, fmradio played. - played video : fm radio stopped while video played, fm radio started when it ended went to camera, capture video : fm radio stopped completely even after finishing recording. there was a crash that occurred not sure exactly what. ** need to report a bug. - launched browser, went to youtube, played a youtube vid: fm radio stopped, video played - stopped youtube : fm radio continued - set alarm : fm radio stopped... alarm sounds through speaker not headset... ** - closed alarm : fm radio started - set alarm again, used snooze : after hitting snooze, fm radio started again. - turned on all sounds in settings including keyboard typing, went to email, composed an email : typing sounds go through headset while fm radio is playing; no email send sound like on uxbranch... is that a bug? - snooze went off : fm radio stopped and when closing the alarm, fm radio started - went back to browser, and went to soundcloud.com : fmradio plays over soundcloud.com ** - e.me : grooveshank : fm radio plays over grooveshark at the same time ** - unplugged while both were playing : fm radio stops, grooveshark still plays.
Flags: needinfo?(nhirata.bugzilla)
That was all in about 30 minutes of testing; session based.
probably would have been faster if I wasn't documenting. :)
bug 796872 is fixed; bug 816118 maybe fixed?; bug 814054 is fixed; bug 817456 is fixed; bug 831481 seems fixed; bug 833989 seems fixed bug 825817 <- noted the pause in this bug. bug 830208 <- may have been my crash Bug 834527 - incoming call rings through speakers instead of headset. Bug 834529 - [Music] If FM Radio is playing and you play a song in the music app, music app will not go to the next song Bug 834532 - [Clock] Alarm does not play through headset the last 2 are probably the same as bug 796514 where it's the web audios that keep playing... Bug 834535 - FMradio beeps when you close it from the task manager
Comment 27•11 years ago
|
||
Andrea/Marco, can you help Naoki sort out these issues?
Flags: needinfo?(overholt) → needinfo?(amarchesini)
Assignee | ||
Comment 28•11 years ago
|
||
Test and verify on 1/27 pvt builds. The analog property is set on the ROM image correctly. Change the status to fix ready.
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Comment 29•11 years ago
|
||
(In reply to Naoki Hirata :nhirata from comment #26) bug 796872/817456/833989/825817/830208/834527/834529/834532/834535 Not related to the build issue here. bug 816118 maybe fixed?; It is invalid and not related to the build issue here. https://bugzilla.mozilla.org/show_bug.cgi?id=816118#c3 > bug 814054 is fixed; The report on comment 20 of 814054 should be fixed as Randy said here. But the original issue from 814054 is not fixed yet. > bug 831481 Fixed by the build issue here.
Updated•11 years ago
|
status-b2g18:
--- → fixed
Updated•11 years ago
|
status-b2g18-v1.0.0:
--- → fixed
Comment 30•11 years ago
|
||
The original issue in this bug has been resolved, so removing qawanted flag.
Updated•11 years ago
|
status-b2g18-v1.0.1:
--- → fixed
Comment 31•11 years ago
|
||
Verified this bug has been fixed on Unagi. Build ID: 20130215070202 Kernel Dec 5 Gecko http://hg.mozilla.org/releases/mozilla-b2g18_v1_0_1/rev/a9e4f8912607 Gaia 21ba59d933c66024cb351c2379315301d5352e0c
Status: RESOLVED → VERIFIED
Flags: needinfo?(amarchesini)
You need to log in
before you can comment on or make changes to this bug.
Description
•