Closed Bug 1042003 Opened 10 years ago Closed 10 years ago
Audio Offload Playback is resuming from the beginning when the device is switched from headset to BT
[Blocking Requested - why for this release]: Steps to Reproduce: 1. Open music app and start playing an offload capable music clip (mp3) 2. Go to Settings and Turnon Bluetooth. 3. Pair and connect to a BT headset. 4. Once BT headset is connected playback resumes from the beginning of the clip. Root cause: SeekTarget takes time in int64_t usecs whereas we pass double seconds in 2 places. Attached patch fixes this issue.
Attachment #8460143 - Flags: review?(roc)
Component: General → Video/Audio
Product: Firefox OS → Core
Attachment #8460143 - Flags: review?(roc) → review+
Whiteboard: [CR 693996] → [caf priority: p2][CR 693996]
Thanks Robert! Submitted my first Try run https://tbpl.mozilla.org/?tree=Try&rev=3765c7f8fd08
Try run looks successful. Starred the only error. Adding checkin-needed
Assignee: nobody → vasanth
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla34
QA Contact: alexandra.lucinet
I see that the affected platform here is Firefox OS. Is there something to be done on Firefox Desktop as well? Thanks in advance!
(In reply to Alexandra Lucinet, QA Mentor [:adalucinet] from comment #6) > Is there something to be done on Firefox Desktop as well? Thanks in advance! I guess part of this change in MediaDecoder.cpp  should fix if this issue comes up in Firefox desktop. May be we can check MediaDecoder::SeekTarget is initialized with int64_t micro seconds instead of double seconds everywhere, just to make sure same issue doesn't exist in desktop specific code. Forwarding ni to author of SeekTarget as he might know better than me.  https://bugzilla.mozilla.org/attachment.cgi?id=8460143&action=diff#a/content/media/MediaDecoder.cpp_sec2
Flags: needinfo?(vasanth) → needinfo?(cpearce)
Desktop Firefox doesn't (yet) make decoders dormant, so Desktop Firefox should not be affected this issue.
This will need to be covered by a new test case.
QA Whiteboard: [QAnalyst-Triage?]
Test case added to moztrap: https://moztrap.mozilla.org/manage/case/14378/
QA Whiteboard: [QAnalyst-Triage?] → [QAnalyst-Triage+]
(In reply to vasanth from comment #0) > Steps to Reproduce: > 1. Open music app and start playing an offload capable music clip (mp3) Hi Vasanth, Could you please tell me what audio format the offload capable music clip (mp3) is？
You need to log in before you can comment on or make changes to this bug.